Re: [PATCH] mm, vmpressure: Fix a signedness bug in vmpressure_register_event()

From: David Rientjes
Date: Thu Sep 26 2019 - 15:04:47 EST


On Wed, 25 Sep 2019, Dan Carpenter wrote:

> The "mode" and "level" variables are enums and in this context GCC will
> treat them as unsigned ints so the error handling is never triggered.
>
> I also removed the bogus initializer because it isn't required any more
> and it's sort of confusing.
>
> Fixes: 3cadfa2b9497 ("mm/vmpressure.c: convert to use match_string() helper")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Acked-by: David Rientjes <rientjes@xxxxxxxxxx>