Re: [PATCH] fs/binfmt_elf: remove the same condition check

2018-08-01 Thread zhong jiang
On 2018/8/2 11:04, Al Viro wrote: > On Thu, Aug 02, 2018 at 10:00:28AM +0800, zhong jiang wrote: >> dump_align is used to double check in a expression. It is redundant. >> so just remove one of them. > > What makes you think that it is redundant? > I am sorry for that. Maybe I miss something.

Re: [PATCH] fs/binfmt_elf: remove the same condition check

2018-08-01 Thread zhong jiang
On 2018/8/2 10:34, Matthew Wilcox wrote: > On Thu, Aug 02, 2018 at 10:00:28AM +0800, zhong jiang wrote: >> dump_align is used to double check in a expression. It is redundant. >> so just remove one of them. > You're wrong. Functions in C can have side-effects (and this one does). > > Ok, I miss t

Re: [PATCH] fs/binfmt_elf: remove the same condition check

2018-08-01 Thread Al Viro
On Thu, Aug 02, 2018 at 10:00:28AM +0800, zhong jiang wrote: > dump_align is used to double check in a expression. It is redundant. > so just remove one of them. What makes you think that it is redundant? > Signed-off-by: zhong jiang > --- > fs/binfmt_elf.c | 2 +- > 1 file changed, 1 inserti

Re: [PATCH] fs/binfmt_elf: remove the same condition check

2018-08-01 Thread Matthew Wilcox
On Thu, Aug 02, 2018 at 10:00:28AM +0800, zhong jiang wrote: > dump_align is used to double check in a expression. It is redundant. > so just remove one of them. You're wrong. Functions in C can have side-effects (and this one does).

[PATCH] fs/binfmt_elf: remove the same condition check

2018-08-01 Thread zhong jiang
dump_align is used to double check in a expression. It is redundant. so just remove one of them. Signed-off-by: zhong jiang --- fs/binfmt_elf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/binfmt_elf.c b/fs/binfmt_elf.c index efae2fb..b6c5b02 100644 --- a/fs/binfmt_elf.