Message ID | 1386017539-10295-1-git-send-email-santosh.shilimkar@ti.com |
---|---|
State | New |
Headers | show |
On Mon, Dec 02, 2013 at 03:52:19PM -0500, Santosh Shilimkar wrote: > Russell, > > Here is the pull request for-next so that the series gets wider testing > to catch any regression early in the cycle. This unfortunately adds a new warning to all the ARM builds: mm/nobootmem.c: In function '__free_pages_memory': mm/nobootmem.c:88:201: warning: comparison of distinct pointer types lacks a cast I don't think it's your change which is causing this, but it does introduce this new code. The problem is this: while (start < end) { order = min(MAX_ORDER - 1UL, __ffs(start)); The types of "MAX_ORDER - 1UL" and "__ffs(start)" differ, which may leads to min() having unexpected behaviour. This isn't helped by: arch/x86/include/asm/bitops.h:static inline unsigned long __ffs(unsigned long word) vs static inline int fls(int x) ... #define ffs(x) ({ unsigned long __t = (x); fls(__t & -__t); }) #define __ffs(x) (ffs(x) - 1) So on ARM, we have a different return type. We need to fix ARM's definition of __ffs() et.al. to have comparable return types to x86.
On Thursday 05 December 2013 05:33 AM, Russell King - ARM Linux wrote: > On Mon, Dec 02, 2013 at 03:52:19PM -0500, Santosh Shilimkar wrote: >> Russell, >> >> Here is the pull request for-next so that the series gets wider testing >> to catch any regression early in the cycle. > > This unfortunately adds a new warning to all the ARM builds: > > mm/nobootmem.c: In function '__free_pages_memory': > mm/nobootmem.c:88:201: warning: comparison of distinct pointer types lacks a cast > I actually sent fix for it before sending you the pull request. Forgot to capture that in message. Patch[1] is already acked by Tejun and I was assuming it would have been applied and should start showing up in next. Not sure if its good idea but we could add the patch along with the $subject series. Regards, Santosh [1] https://lkml.org/lkml/2013/11/25/455