Re: replace incorrect memcpy_user_stub code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Chris Torek <chris.torek@xxxxxxxxxxxxx>
Date: Thu, 18 Dec 2008 20:11:26 -0700

> >This is going to perform really bad ...
> 
> Yes, I went for space on the theory that it was not common...
> 
> >and it's common for
> >syscalls invoked by the kernel to trigger this case.
> 
> ... but so much for that theory. :-)

At least it used to be the case, it may not matter today and
I'll do some investigation.  If it doesn't matter, your patch
is fine and I'll apply it directly.

> >I'll have to think about this, thanks for pointing out the
> >issue.
> 
> The other obvious thing is to have a bunch of userlike memcpy
> variants, one for each *memcpy.S file.  The macros you have
> should make this straightforward.  I was mostly worried about
> breaking stuff.

I'm trending away from that because it's unnecessary bloat.

I wish memcpy() returned void, that would make so many things like
this easy to handle.  I could just add the annotations to the real
memcpy, always return the same return values as the user copies do,
and the memcpy() callers would just ignore the return value.

But things are what they are :-)
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Development]     [DCCP]     [Linux ARM Development]     [Linux]     [Photo]     [Yosemite Help]     [Linux ARM Kernel]     [Linux SCSI]     [Linux x86_64]     [Linux Hams]

  Powered by Linux