lkml.org 
[lkml]   [2016]   [Mar]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH] staging/android: change IOCTLs opcode after ABI change
    Date
    From: Gustavo Padovan <gustavo.padovan@collabora.co.uk>

    Burn the old opcode to avoid any potential old userspace running the old
    API to get weird errors. Changing the opcodes will make them fail right
    away.

    This is just a precaution, there no upstream users of these interfaces
    yet and the only user is Android, but we don't expect anyone trying to
    run android userspace and all it dependencies on top of upstream kernels.

    Moreover Android should be converted to use upstream sync_files.

    Suggested-by: Rob Clark <robdclark@gmail.com>
    Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
    ---
    drivers/staging/android/uapi/sync.h | 11 +++++++++--
    1 file changed, 9 insertions(+), 2 deletions(-)

    diff --git a/drivers/staging/android/uapi/sync.h b/drivers/staging/android/uapi/sync.h
    index 859977c..fbadb8a 100644
    --- a/drivers/staging/android/uapi/sync.h
    +++ b/drivers/staging/android/uapi/sync.h
    @@ -69,13 +69,20 @@ struct sync_file_info {
    #define SYNC_IOC_MAGIC '>'

    /**
    + * Opcodes 0, 1 and 2 were burned during a API change to avoid users of the
    + * old API to get weird errors when trying to handling sync_files. The API
    + * change happened during the de-stage of the Sync Framework when there was
    + * no upstream users available.
    + */
    +
    +/**
    * DOC: SYNC_IOC_MERGE - merge two fences
    *
    * Takes a struct sync_merge_data. Creates a new fence containing copies of
    * the sync_pts in both the calling fd and sync_merge_data.fd2. Returns the
    * new fence's fd in sync_merge_data.fence
    */
    -#define SYNC_IOC_MERGE _IOWR(SYNC_IOC_MAGIC, 1, struct sync_merge_data)
    +#define SYNC_IOC_MERGE _IOWR(SYNC_IOC_MAGIC, 3, struct sync_merge_data)

    /**
    * DOC: SYNC_IOC_FENCE_INFO - get detailed information on a fence
    @@ -88,6 +95,6 @@ struct sync_file_info {
    * pt_info is a buffer containing sync_pt_infos for every sync_pt in the fence.
    * To iterate over the sync_pt_infos, use the sync_pt_info.len field.
    */
    -#define SYNC_IOC_FILE_INFO _IOWR(SYNC_IOC_MAGIC, 2, struct sync_file_info)
    +#define SYNC_IOC_FILE_INFO _IOWR(SYNC_IOC_MAGIC, 4, struct sync_file_info)

    #endif /* _UAPI_LINUX_SYNC_H */
    --
    2.5.0
    \
     
     \ /
      Last update: 2016-03-04 00:21    [W:2.758 / U:0.388 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site