lkml.org 
[lkml]   [2013]   [Feb]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] staging: android: ashmem: fix ashmem pin/unpin interface
Hi Greg,

On 01/02/13 16:18, Greg KH wrote:
> On Fri, Feb 01, 2013 at 04:08:00PM +0000, Serban Constantinescu wrote:
>> The values exchanged between kernel and userspace through struct
>> ashmem_pin should be of type size_t. This change won't affect the
>> existing interface but will stand as the basis of 64bit compat layer.
>
> How do you define size_t with a 64bit kernel and a 32bit userspace
> properly? Doesn't this change open up a bunch of problems?

The current ashmem pin/unpin kernel interface uses __u32 to specify the
memory region and length in bytes. However these values should be of
type size_t so that they are able to represent the whole range of
possible values when compiled for a 64bit platform.

Android API uses ashmem driver through libcutils, from where I attach
the following snippet:

<aosp>/system/core/libcutils/ashmem-dev.c

75 int ashmem_pin_region(int fd, size_t offset, size_t len)
76 {
77 struct ashmem_pin pin = { offset, len };
78 return ioctl(fd, ASHMEM_PIN, &pin);
79 }

The kernel changes inline with the userspace usage and do not affect
existing 32bit Android (we have exported the new kernel header, rebuilt
and tested the interface with success).

However this change will affect any 64bit userspace using the current
faulty interface, but there is none that we know of.

Thanks,
Serban Constantinescu
`





\
 
 \ /
  Last update: 2013-02-01 18:44    [W:0.051 / U:0.576 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site