Re: [next:master 156/212] fs/binfmt_elf.c:158:18: note: in expansion of macro 'min'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 25 Jun 2014 19:30:27 +0800 Jeff Liu <jeff.liu@xxxxxxxxxx> wrote:

> 
> On 06/25/2014 18:02 PM, Fengguang Wu wrote:
> > tree:   git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > head:   30404ddcb1872c8a571fa0889935ff65677e4c78
> > commit: aef93cafef35b8830fc973be43f0745f9c16eff4 [156/212] binfmt_elf.c: use get_random_int() to fix entropy depleting
> > config: make ARCH=mn10300 asb2364_defconfig
> > 
> > All warnings:
> > 
> >    In file included from include/asm-generic/bug.h:13:0,
> >                     from arch/mn10300/include/asm/bug.h:35,
> >                     from include/linux/bug.h:4,
> >                     from include/linux/thread_info.h:11,
> >                     from include/asm-generic/preempt.h:4,
> >                     from arch/mn10300/include/generated/asm/preempt.h:1,
> >                     from include/linux/preempt.h:18,
> >                     from include/linux/spinlock.h:50,
> >                     from include/linux/seqlock.h:35,
> >                     from include/linux/time.h:5,
> >                     from include/linux/stat.h:18,
> >                     from include/linux/module.h:10,
> >                     from fs/binfmt_elf.c:12:
> >    fs/binfmt_elf.c: In function 'get_atrandom_bytes':
> >    include/linux/kernel.h:713:17: warning: comparison of distinct pointer types lacks a cast
> >      (void) (&_min1 == &_min2);  \
> >                     ^
> >>> fs/binfmt_elf.c:158:18: note: in expansion of macro 'min'
> >       size_t chunk = min(nbytes, sizeof(random_variable));
> 
> I remember we have the same report on arch mn10300 about half a year ago, but the code
> is correct. :)

We really need to do something about this patch - it's been stuck in
-mm for ever.

I have a note here that Stephan Mueller identified issues with it but I
don't recall what they were - do you? 

Maybe you could go back through the list dicussion, identify all/any
issues which were raised, update the changelog to address them then
resend it, copying people who were involved in the earlier discussion?

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]