Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-25 Thread Greg KH
On Thu, Jan 25, 2018 at 10:09:53AM +0800, Ganesh Mahendran wrote: > Hi, Martijn > > 2018-01-24 22:33 GMT+08:00 Martijn Coenen : > > On Mon, Jan 22, 2018 at 4:54 PM, Greg KH wrote: > >> Martijn and Todd, any objections to this patch? > > > > Looks

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-25 Thread Greg KH
On Thu, Jan 25, 2018 at 10:09:53AM +0800, Ganesh Mahendran wrote: > Hi, Martijn > > 2018-01-24 22:33 GMT+08:00 Martijn Coenen : > > On Mon, Jan 22, 2018 at 4:54 PM, Greg KH wrote: > >> Martijn and Todd, any objections to this patch? > > > > Looks good to me. > > Thanks for your review. > >

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-24 Thread Ganesh Mahendran
Hi, Martijn 2018-01-24 22:33 GMT+08:00 Martijn Coenen : > On Mon, Jan 22, 2018 at 4:54 PM, Greg KH wrote: >> Martijn and Todd, any objections to this patch? > > Looks good to me. Thanks for your review. Should I cherry-pick this change to aosp

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-24 Thread Ganesh Mahendran
Hi, Martijn 2018-01-24 22:33 GMT+08:00 Martijn Coenen : > On Mon, Jan 22, 2018 at 4:54 PM, Greg KH wrote: >> Martijn and Todd, any objections to this patch? > > Looks good to me. Thanks for your review. Should I cherry-pick this change to aosp kernel 3.10/3.18/4.4/4.9 now? Thanks. > >> >>

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-24 Thread Ganesh Mahendran
Hi, Todd: 2018-01-23 1:02 GMT+08:00 Todd Kjos : > On Mon, Jan 22, 2018 at 7:54 AM, Greg KH wrote: >> On Wed, Jan 10, 2018 at 10:49:05AM +0800, Ganesh Mahendran wrote: >>> VM_IOREMAP is used to access hardware through a mechanism called >>> I/O mapped

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-24 Thread Ganesh Mahendran
Hi, Todd: 2018-01-23 1:02 GMT+08:00 Todd Kjos : > On Mon, Jan 22, 2018 at 7:54 AM, Greg KH wrote: >> On Wed, Jan 10, 2018 at 10:49:05AM +0800, Ganesh Mahendran wrote: >>> VM_IOREMAP is used to access hardware through a mechanism called >>> I/O mapped memory. Android binder is a IPC machanism

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-24 Thread Martijn Coenen
On Mon, Jan 22, 2018 at 4:54 PM, Greg KH wrote: > Martijn and Todd, any objections to this patch? Looks good to me. > > thanks, > > greg k-h

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-24 Thread Martijn Coenen
On Mon, Jan 22, 2018 at 4:54 PM, Greg KH wrote: > Martijn and Todd, any objections to this patch? Looks good to me. > > thanks, > > greg k-h

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-23 Thread Ganesh Mahendran
Hi, Arve: 2018-01-23 2:55 GMT+08:00 Arve Hjønnevåg : > On Mon, Jan 22, 2018 at 9:02 AM, Todd Kjos wrote: >> On Mon, Jan 22, 2018 at 7:54 AM, Greg KH wrote: >>> On Wed, Jan 10, 2018 at 10:49:05AM +0800, Ganesh Mahendran wrote:

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-23 Thread Ganesh Mahendran
Hi, Arve: 2018-01-23 2:55 GMT+08:00 Arve Hjønnevåg : > On Mon, Jan 22, 2018 at 9:02 AM, Todd Kjos wrote: >> On Mon, Jan 22, 2018 at 7:54 AM, Greg KH wrote: >>> On Wed, Jan 10, 2018 at 10:49:05AM +0800, Ganesh Mahendran wrote: VM_IOREMAP is used to access hardware through a mechanism called

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-22 Thread Arve Hjønnevåg
On Mon, Jan 22, 2018 at 9:02 AM, Todd Kjos wrote: > On Mon, Jan 22, 2018 at 7:54 AM, Greg KH wrote: >> On Wed, Jan 10, 2018 at 10:49:05AM +0800, Ganesh Mahendran wrote: >>> VM_IOREMAP is used to access hardware through a mechanism called >>> I/O

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-22 Thread Arve Hjønnevåg
On Mon, Jan 22, 2018 at 9:02 AM, Todd Kjos wrote: > On Mon, Jan 22, 2018 at 7:54 AM, Greg KH wrote: >> On Wed, Jan 10, 2018 at 10:49:05AM +0800, Ganesh Mahendran wrote: >>> VM_IOREMAP is used to access hardware through a mechanism called >>> I/O mapped memory. Android binder is a IPC machanism

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-22 Thread Todd Kjos
On Mon, Jan 22, 2018 at 7:54 AM, Greg KH wrote: > On Wed, Jan 10, 2018 at 10:49:05AM +0800, Ganesh Mahendran wrote: >> VM_IOREMAP is used to access hardware through a mechanism called >> I/O mapped memory. Android binder is a IPC machanism which will >> not access I/O

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-22 Thread Todd Kjos
On Mon, Jan 22, 2018 at 7:54 AM, Greg KH wrote: > On Wed, Jan 10, 2018 at 10:49:05AM +0800, Ganesh Mahendran wrote: >> VM_IOREMAP is used to access hardware through a mechanism called >> I/O mapped memory. Android binder is a IPC machanism which will >> not access I/O memory. >> >> And VM_IOREMAP

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-22 Thread Greg KH
On Wed, Jan 10, 2018 at 10:49:05AM +0800, Ganesh Mahendran wrote: > VM_IOREMAP is used to access hardware through a mechanism called > I/O mapped memory. Android binder is a IPC machanism which will > not access I/O memory. > > And VM_IOREMAP has alignment requiement which may not needed in >

Re: [PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-22 Thread Greg KH
On Wed, Jan 10, 2018 at 10:49:05AM +0800, Ganesh Mahendran wrote: > VM_IOREMAP is used to access hardware through a mechanism called > I/O mapped memory. Android binder is a IPC machanism which will > not access I/O memory. > > And VM_IOREMAP has alignment requiement which may not needed in >

[PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-09 Thread Ganesh Mahendran
VM_IOREMAP is used to access hardware through a mechanism called I/O mapped memory. Android binder is a IPC machanism which will not access I/O memory. And VM_IOREMAP has alignment requiement which may not needed in binder. __get_vm_area_node() { ... if (flags & VM_IOREMAP)

[PATCH v3] android: binder: use VM_ALLOC to get vm area

2018-01-09 Thread Ganesh Mahendran
VM_IOREMAP is used to access hardware through a mechanism called I/O mapped memory. Android binder is a IPC machanism which will not access I/O memory. And VM_IOREMAP has alignment requiement which may not needed in binder. __get_vm_area_node() { ... if (flags & VM_IOREMAP)