| From: |
| Laura Abbott <labbott-AT-redhat.com> |
| To: |
| Sumit Semwal <sumit.semwal-AT-linaro.org>, John Stultz <john.stultz-AT-linaro.org>, Arve Hjønnevåg <arve-AT-android.com>, Riley Andrews <riandrews-AT-android.com> |
| Subject: |
| [PATCHv2 0/4] New Ion ioctls |
| Date: |
| Thu, 1 Sep 2016 15:40:40 -0700 |
| Message-ID: |
| <1472769644-11039-1-git-send-email-labbott@redhat.com> |
| Cc: |
| Laura Abbott <labbott-AT-redhat.com>, Daniel Vetter <daniel.vetter-AT-ffwll.ch>, linaro-mm-sig-AT-lists.linaro.org, devel-AT-driverdev.osuosl.org, linux-kernel-AT-vger.kernel.org, Eun Taik Lee <eun.taik.lee-AT-samsung.com>, Liviu Dudau <Liviu.Dudau-AT-arm.com>, Jon Medhurst <tixy-AT-linaro.org>, Mitchel Humpherys <mitchelh-AT-codeaurora.org>, Jeremy Gebben <jgebben-AT-codeaurora.org>, Bryan Huntsman <bryanh-AT-codeaurora.org>, Greg Kroah-Hartman <gregkh-AT-linuxfoundation.org>, Android Kernel Team <kernel-team-AT-android.com>, Chen Feng <puck.chen-AT-hisilicon.com>, Brian Starkey <brian.starkey-AT-arm.com> |
Hi,
This is a follow up to my previous series[1] for Ion ioctls. I've changed the
focus slightly based on the feedback. The ID remapping was less useful than I
originally thought and without that addition there isn't much benefit to have
a new alloc ioctl. The ABI check and query interface still seem beneficial.
There was some discussion on where exactly these types of ioctls would be
called. I expect the answer will depend on exactly how it's integrated.
Long term, I'd still like to fix the ABI to not be a checklist of botching
up ioctls but that focus will come later.
Changes from v1:
- Rebased
- Dropped RFC
- Dropped ID remapping and dependent logic
- Changed query logic to only need one ioctl
- Fixed alignment of query ioctl structure
[1] http://www.mail-archive.com/driverdev-devel@linuxdriverpr...
Laura Abbott (4):
staging: android: ion: Drop heap type masks
staging: android: ion: Pull out ion ioctls to a separate file
staging: android: ion: Add an ioctl for ABI checking
staging: android: ion: Add ioctl to query available heaps
drivers/staging/android/ion/Makefile | 3 +-
drivers/staging/android/ion/ion-ioctl.c | 188 ++++++++++++++++++++++++++
drivers/staging/android/ion/ion.c | 226 ++++++--------------------------
drivers/staging/android/ion/ion_priv.h | 94 +++++++++++++
drivers/staging/android/uapi/ion.h | 67 +++++++++-
5 files changed, 382 insertions(+), 196 deletions(-)
create mode 100644 drivers/staging/android/ion/ion-ioctl.c
--
2.7.4
