Re: [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-03 Thread Greg Kroah-Hartman
On Fri, Sep 02, 2016 at 01:41:48PM -0700, Laura Abbott wrote: > On 09/01/2016 11:14 PM, Greg Kroah-Hartman wrote: > > On Thu, Sep 01, 2016 at 03:40:44PM -0700, Laura Abbott wrote: > > > > > > Ion clients currently lack a good method to determine what > > > heaps are available and what ids they

Re: [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-03 Thread Greg Kroah-Hartman
On Fri, Sep 02, 2016 at 01:41:48PM -0700, Laura Abbott wrote: > On 09/01/2016 11:14 PM, Greg Kroah-Hartman wrote: > > On Thu, Sep 01, 2016 at 03:40:44PM -0700, Laura Abbott wrote: > > > > > > Ion clients currently lack a good method to determine what > > > heaps are available and what ids they

Re: [Linaro-mm-sig] [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-02 Thread Laura Abbott
On 09/02/2016 02:37 PM, Arnd Bergmann wrote: On Friday, September 2, 2016 2:27:21 PM CEST Laura Abbott wrote: All warnings (new ones prefixed by >>): drivers/staging/android/ion/ion.c: In function 'ion_query_heaps': drivers/staging/android/ion/ion.c:1181:3: warning: cast to pointer from

Re: [Linaro-mm-sig] [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-02 Thread Laura Abbott
On 09/02/2016 02:37 PM, Arnd Bergmann wrote: On Friday, September 2, 2016 2:27:21 PM CEST Laura Abbott wrote: All warnings (new ones prefixed by >>): drivers/staging/android/ion/ion.c: In function 'ion_query_heaps': drivers/staging/android/ion/ion.c:1181:3: warning: cast to pointer from

Re: [Linaro-mm-sig] [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-02 Thread Arnd Bergmann
On Friday, September 2, 2016 2:27:21 PM CEST Laura Abbott wrote: > > > > All warnings (new ones prefixed by >>): > > > >drivers/staging/android/ion/ion.c: In function 'ion_query_heaps': > >>> drivers/staging/android/ion/ion.c:1181:3: warning: cast to pointer from > >>> integer of different

Re: [Linaro-mm-sig] [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-02 Thread Arnd Bergmann
On Friday, September 2, 2016 2:27:21 PM CEST Laura Abbott wrote: > > > > All warnings (new ones prefixed by >>): > > > >drivers/staging/android/ion/ion.c: In function 'ion_query_heaps': > >>> drivers/staging/android/ion/ion.c:1181:3: warning: cast to pointer from > >>> integer of different

Re: [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-02 Thread Laura Abbott
On 09/01/2016 04:44 PM, kbuild test robot wrote: Hi Laura, [auto build test WARNING on next-20160825] [cannot apply to staging/staging-testing v4.8-rc4 v4.8-rc3 v4.8-rc2 v4.8-rc4] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use

Re: [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-02 Thread Laura Abbott
On 09/01/2016 04:44 PM, kbuild test robot wrote: Hi Laura, [auto build test WARNING on next-20160825] [cannot apply to staging/staging-testing v4.8-rc4 v4.8-rc3 v4.8-rc2 v4.8-rc4] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use

Re: [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-02 Thread Laura Abbott
On 09/01/2016 11:14 PM, Greg Kroah-Hartman wrote: On Thu, Sep 01, 2016 at 03:40:44PM -0700, Laura Abbott wrote: Ion clients currently lack a good method to determine what heaps are available and what ids they map to. This leads to tight coupling between user and kernel space and headaches. Add

Re: [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-02 Thread Laura Abbott
On 09/01/2016 11:14 PM, Greg Kroah-Hartman wrote: On Thu, Sep 01, 2016 at 03:40:44PM -0700, Laura Abbott wrote: Ion clients currently lack a good method to determine what heaps are available and what ids they map to. This leads to tight coupling between user and kernel space and headaches. Add

Re: [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-02 Thread Greg Kroah-Hartman
On Thu, Sep 01, 2016 at 03:40:44PM -0700, Laura Abbott wrote: > > Ion clients currently lack a good method to determine what > heaps are available and what ids they map to. This leads > to tight coupling between user and kernel space and headaches. > Add a query ioctl to let userspace know the

Re: [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-02 Thread Greg Kroah-Hartman
On Thu, Sep 01, 2016 at 03:40:44PM -0700, Laura Abbott wrote: > > Ion clients currently lack a good method to determine what > heaps are available and what ids they map to. This leads > to tight coupling between user and kernel space and headaches. > Add a query ioctl to let userspace know the

Re: [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-01 Thread kbuild test robot
Hi Laura, [auto build test WARNING on next-20160825] [cannot apply to staging/staging-testing v4.8-rc4 v4.8-rc3 v4.8-rc2 v4.8-rc4] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto

Re: [PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-01 Thread kbuild test robot
Hi Laura, [auto build test WARNING on next-20160825] [cannot apply to staging/staging-testing v4.8-rc4 v4.8-rc3 v4.8-rc2 v4.8-rc4] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto

[PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-01 Thread Laura Abbott
Ion clients currently lack a good method to determine what heaps are available and what ids they map to. This leads to tight coupling between user and kernel space and headaches. Add a query ioctl to let userspace know the availability of heaps. Signed-off-by: Laura Abbott

[PATCHv2 4/4] staging: android: ion: Add ioctl to query available heaps

2016-09-01 Thread Laura Abbott
Ion clients currently lack a good method to determine what heaps are available and what ids they map to. This leads to tight coupling between user and kernel space and headaches. Add a query ioctl to let userspace know the availability of heaps. Signed-off-by: Laura Abbott ---