Re: [PATCH 1/7] binder: create userspace-to-binder-buffer copy function

2019-01-29 Thread Dan Carpenter
On Tue, Jan 29, 2019 at 04:32:09PM -0800, Todd Kjos wrote: > On Tue, Jan 29, 2019 at 12:12 AM Dan Carpenter > wrote: > > > > On Mon, Jan 28, 2019 at 04:49:28PM -0800, Todd Kjos wrote: > > > +/** > > > + * binder_alloc_copy_user_to_buffer() - copy src user to tgt user > > > + * @alloc:

Re: [PATCH 1/7] binder: create userspace-to-binder-buffer copy function

2019-01-29 Thread Todd Kjos
On Tue, Jan 29, 2019 at 12:12 AM Dan Carpenter wrote: > > On Mon, Jan 28, 2019 at 04:49:28PM -0800, Todd Kjos wrote: > > +/** > > + * binder_alloc_copy_user_to_buffer() - copy src user to tgt user > > + * @alloc: binder_alloc for this proc > > + * @buffer: binder buffer to be accessed > > + *

Re: [PATCH 1/7] binder: create userspace-to-binder-buffer copy function

2019-01-29 Thread Dan Carpenter
On Mon, Jan 28, 2019 at 04:49:28PM -0800, Todd Kjos wrote: > +/** > + * binder_alloc_copy_user_to_buffer() - copy src user to tgt user > + * @alloc: binder_alloc for this proc > + * @buffer: binder buffer to be accessed > + * @buffer_offset: offset into @buffer data > + * @from: userspace pointer

[PATCH 1/7] binder: create userspace-to-binder-buffer copy function

2019-01-28 Thread Todd Kjos
The binder driver uses a vm_area to map the per-process binder buffer space. For 32-bit android devices, this is now taking too much vmalloc space. This patch removes the use of vm_area when copying the transaction data from the sender to the buffer space. Instead of using copy_from_user() for