Hi Dan, On Thu, Jan 22, 2015 at 04:30:44PM +0300, Dan Carpenter wrote: > We recently re-arranged the code in these functions and now static > checkers complain that "ret" is uninitialized. Oddly enough GCC is fine > with this code. > > Fixes: d1ebc463cf89 ('mm: page_counter: pull "-1" handling out of page_counter_memparse()') > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> This code was again re-arranged in -mm, and that ret = page_counter_memparse() is now happening unconditionally, so I think it should be fine. The latest mmots snapshot (mmots-2015-01-21-16-38) has it. Thanks! -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html