| Date | Tue, 28 Apr 2009 22:53:52 -0700 | From | Andrew Morton <> | Subject | Re: [PATCH 20/30] microblaze_mmu_v1: uaccess MMU update |
| |
On Mon, 27 Apr 2009 10:32:09 +0200 monstr@monstr.eu wrote:
> +#define __clear_user(addr, n) (memset((void *)(addr), 0, (n)), 0)
__clear_user() already takes a void* argument, so the cast is hopefully unneeded and undesirable. The same applies to many other architectures. If someone is passing a non-pointer type to __clear_user() then that's potentially a bad thing which we want to know about.
IOW, someone screwed up and everyone copied it :(
|