On Wed, 17 Dec 2014, Andrey Ryabinin wrote:

> hugetlb_treat_as_movable declared as unsigned long, but
> proc_dointvec() used for parsing it:
> 
> static struct ctl_table vm_table[] = {
> ...
>       {
>               .procname       = "hugepages_treat_as_movable",
>               .data           = &hugepages_treat_as_movable,
>               .maxlen         = sizeof(int),
>               .mode           = 0644,
>               .proc_handler   = proc_dointvec,
>       },
> 
> This seems harmless, but it's better to use int type here.
> 
> Signed-off-by: Andrey Ryabinin <[email protected]>

Acked-by: David Rientjes <[email protected]>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to