From: Gustavo Padovan <gustavo@padovan.org> To: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org, dri-devel@lists.freedesktop.org, "Daniel Stone" <daniels@collabora.com>, "Arve Hjønnevåg" <arve@android.com>, "Riley Andrews" <riandrews@android.com>, "Daniel Vetter" <daniel.vetter@ffwll.ch>, "Rob Clark" <robdclark@gmail.com>, "Greg Hackmann" <ghackmann@google.com>, "John Harrison" <John.C.Harrison@Intel.com>, "Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>, "Gustavo Padovan" <gustavo.padovan@collabora.co.uk> Subject: [PATCH v3 00/11] android sync framework: clean up IOCTLs and ABI Date: Wed, 3 Feb 2016 11:25:29 -0200 [thread overview] Message-ID: <1454505940-18094-1-git-send-email-gustavo@padovan.org> (raw) From: Gustavo Padovan <gustavo.padovan@collabora.co.uk> Hi, This patch series clean up IOCTLs and abi of sync framework and it is a follow up on the clean up series I've sent on Jan 21: http://thread.gmane.org/gmane.comp.video.dri.devel/145509 The main changes here are: * remove of SYNC_WAIT ioctl, poll() should be used instead. * rename some structs and macros to better reflect the new internal names. * clean up and improve ABI on SYNC_IOC_FILE_INFO * add flags filed to all ABI structs * make sync_file_info->len return only the size of the array of fence_infos v2: - Check flags passed to the kernel (Maarten) - Rework len to report only the size of fence_infos array (Emil) - Fix fence_info type. Use __u64 pointer type (Emil, Maarten) v3: - Rename fence_info to sync_fence_info (Maarten) - Fix check for info.name (Maarten) - Update commit description of patch 06 (Emil, Maarten) Please review! Thanks. Gustavo Padovan (11): staging/android: remove SYNC_WAIT ioctl staging/android: rename sync_pt_info to sync_fence_info staging/android: rename sync_file_info_data to sync_file_info staging/android: remove driver_data from struct sync_fence_info staging/android: remove len field from struct sync_fence_info staging/android: turn fence_info into a __u64 pointer staging/android: add num_fences field to struct sync_file_info staging/android: make info->len return only size of sync_fence_info array staging/android: rename SYNC_IOC_FENCE_INFO staging/android: add flags member to sync ioctl structs staging/android: remove redundant comments on sync_merge_data drivers/staging/android/sw_sync.c | 14 ---- drivers/staging/android/sync.c | 129 +++++++++-------------------------- drivers/staging/android/sync.h | 20 ------ drivers/staging/android/trace/sync.h | 44 ------------ drivers/staging/android/uapi/sync.h | 50 ++++++-------- 5 files changed, 56 insertions(+), 201 deletions(-) -- 2.5.0
WARNING: multiple messages have this Message-ID (diff)
From: Gustavo Padovan <gustavo@padovan.org> To: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: devel@driverdev.osuosl.org, "Daniel Stone" <daniels@collabora.com>, "Daniel Vetter" <daniel.vetter@ffwll.ch>, "Riley Andrews" <riandrews@android.com>, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, "Arve Hjønnevåg" <arve@android.com>, "Gustavo Padovan" <gustavo.padovan@collabora.co.uk>, "John Harrison" <John.C.Harrison@Intel.com> Subject: [PATCH v3 00/11] android sync framework: clean up IOCTLs and ABI Date: Wed, 3 Feb 2016 11:25:29 -0200 [thread overview] Message-ID: <1454505940-18094-1-git-send-email-gustavo@padovan.org> (raw) From: Gustavo Padovan <gustavo.padovan@collabora.co.uk> Hi, This patch series clean up IOCTLs and abi of sync framework and it is a follow up on the clean up series I've sent on Jan 21: http://thread.gmane.org/gmane.comp.video.dri.devel/145509 The main changes here are: * remove of SYNC_WAIT ioctl, poll() should be used instead. * rename some structs and macros to better reflect the new internal names. * clean up and improve ABI on SYNC_IOC_FILE_INFO * add flags filed to all ABI structs * make sync_file_info->len return only the size of the array of fence_infos v2: - Check flags passed to the kernel (Maarten) - Rework len to report only the size of fence_infos array (Emil) - Fix fence_info type. Use __u64 pointer type (Emil, Maarten) v3: - Rename fence_info to sync_fence_info (Maarten) - Fix check for info.name (Maarten) - Update commit description of patch 06 (Emil, Maarten) Please review! Thanks. Gustavo Padovan (11): staging/android: remove SYNC_WAIT ioctl staging/android: rename sync_pt_info to sync_fence_info staging/android: rename sync_file_info_data to sync_file_info staging/android: remove driver_data from struct sync_fence_info staging/android: remove len field from struct sync_fence_info staging/android: turn fence_info into a __u64 pointer staging/android: add num_fences field to struct sync_file_info staging/android: make info->len return only size of sync_fence_info array staging/android: rename SYNC_IOC_FENCE_INFO staging/android: add flags member to sync ioctl structs staging/android: remove redundant comments on sync_merge_data drivers/staging/android/sw_sync.c | 14 ---- drivers/staging/android/sync.c | 129 +++++++++-------------------------- drivers/staging/android/sync.h | 20 ------ drivers/staging/android/trace/sync.h | 44 ------------ drivers/staging/android/uapi/sync.h | 50 ++++++-------- 5 files changed, 56 insertions(+), 201 deletions(-) -- 2.5.0 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
next reply other threads:[~2016-02-03 13:25 UTC|newest] Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-02-03 13:25 Gustavo Padovan [this message] 2016-02-03 13:25 ` [PATCH v3 00/11] android sync framework: clean up IOCTLs and ABI Gustavo Padovan 2016-02-03 13:25 ` [PATCH v3 01/11] staging/android: remove SYNC_WAIT ioctl Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan 2016-02-03 13:25 ` [PATCH v3 02/11] staging/android: rename sync_pt_info to sync_fence_info Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan 2016-02-03 13:25 ` [PATCH v3 03/11] staging/android: rename sync_file_info_data to sync_file_info Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan 2016-02-03 13:25 ` [PATCH v3 04/11] staging/android: remove driver_data from struct sync_fence_info Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan 2016-02-03 13:25 ` [PATCH v3 05/11] staging/android: remove len field " Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan 2016-02-03 13:25 ` [PATCH v3 06/11] staging/android: turn fence_info into a __u64 pointer Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan 2016-02-03 14:39 ` Maarten Lankhorst 2016-02-03 14:39 ` Maarten Lankhorst 2016-02-03 20:09 ` Gustavo Padovan 2016-02-03 20:09 ` Gustavo Padovan 2016-02-04 9:57 ` Maarten Lankhorst 2016-02-04 9:57 ` Maarten Lankhorst 2016-02-04 13:05 ` Gustavo Padovan 2016-02-04 13:23 ` Maarten Lankhorst 2016-02-04 13:23 ` Maarten Lankhorst 2016-02-08 9:01 ` Daniel Vetter 2016-02-08 9:01 ` Daniel Vetter 2016-02-03 13:25 ` [PATCH v3 07/11] staging/android: add num_fences field to struct sync_file_info Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan 2016-02-03 13:25 ` [PATCH v3 08/11] staging/android: make info->len return only size of sync_fence_info array Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan 2016-02-03 13:25 ` [PATCH v3 09/11] staging/android: rename SYNC_IOC_FENCE_INFO Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan 2016-02-03 13:25 ` [PATCH v3 10/11] staging/android: add flags member to sync ioctl structs Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan 2016-02-03 13:25 ` [PATCH v3 11/11] staging/android: remove redundant comments on sync_merge_data Gustavo Padovan 2016-02-03 13:25 ` Gustavo Padovan
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1454505940-18094-1-git-send-email-gustavo@padovan.org \ --to=gustavo@padovan.org \ --cc=John.C.Harrison@Intel.com \ --cc=arve@android.com \ --cc=daniel.vetter@ffwll.ch \ --cc=daniels@collabora.com \ --cc=devel@driverdev.osuosl.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=ghackmann@google.com \ --cc=gregkh@linuxfoundation.org \ --cc=gustavo.padovan@collabora.co.uk \ --cc=linux-kernel@vger.kernel.org \ --cc=maarten.lankhorst@linux.intel.com \ --cc=riandrews@android.com \ --cc=robdclark@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.