I got confused when I see the title of patch. That's because I understood copy_to_user returns unsigned. Please, change title to describe your patch. [PATCH] proc : fix wrong usage of copy_to_user's return value. Maybe you could name it well than me who is a not native English. On Tue, Apr 6, 2010 at 7:45 PM, Dan Carpenter <error27@xxxxxxxxx> wrote: > copy_to_user() returns the number of bytes left to be copied. > > This was a typo from: d82ef020cf31504c816803b1def94eb5ff173363 > "proc: pagemap: Hold mmap_sem during page walk". > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> Reviewed-by: Minchan Kim <minchan.kim@xxxxxxxxx> -- Kind regards, Minchan Kim -- 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