lkml.org 
[lkml]   [2013]   [Apr]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v3 0/6] Android Binder IPC Fixes
Hi all,

Any feedback or comments on this patch set?

Thanks,
Serban


On 12/04/13 13:25, Serban Constantinescu wrote:
> Hi all,
>
> This set of patches will clean-up and fix some of the issues that arise
> with the current binder interface when moving to a 64bit kernel. All these
> changes will not affect the existing 32bit Android interface and are meant
> to stand as the base for the 64bit binder compat layer(kernel or userpsace).
>
> The patch set has been successfully tested with a 64bit Linux userspace and
> 64bit binder unit-tests.
>
> This patch set has been successfully tested on 32bit platforms(ARMv7 VExpress)
> and 64bit platforms(ARMv8 RTSM) running a 32bit Android userspace and an in
> kernel binder compat layer.
>
> Changes for v3:
> 1: Dropped the patch that was replacing uint32_t types with unsigned int
> 2: Dropped the patch fixing the IOCTL types(since it has been added to Greg's
> staging tree)
> 3: Split one patch into two: 'modify binder_write_read' and '64bit changes'
> 4: Modified BINDER_SET_MAX_THREADS ioctl definition accordint to Arve's review
> 5: Modified the binder command IOCTL declarations according to Arve's review
>
> Changes for v2:
> 1: 1/7: Modified the commit message according to Greg's feedback;
> 2: 3/7: Merged with the patch fixing the printk format specifiers.
>
> Serban Constantinescu (6):
> staging: android: binder: modify struct binder_write_read to use
> size_t
> staging: android: binder: fix binder interface for 64bit compat layer
> staging: android: binder: fix BINDER_SET_MAX_THREADS declaration
> staging: android: binder: fix BC_FREE_BUFFER ioctl declaration
> staging: android: binder: fix alignment issues
> staging: android: binder: replace types with portable ones
>
> drivers/staging/android/binder.c | 40 ++++++++++++++++-----------------
> drivers/staging/android/binder.h | 46 +++++++++++++++++++-------------------
> 2 files changed, 43 insertions(+), 43 deletions(-)
>



\
 
 \ /
  Last update: 2013-04-29 19:01    [W:0.079 / U:0.644 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site