All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laura Abbott <labbott@redhat.com>
To: Sumit Semwal <sumit.semwal@linaro.org>,
	Riley Andrews <riandrews@android.com>,
	arve@android.com, Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Laura Abbott <labbott@redhat.com>,
	romlem@google.com, devel@driverdev.osuosl.org,
	linux-kernel@vger.kernel.org, linaro-mm-sig@lists.linaro.org,
	linux-arm-kernel@lists.infradead.org,
	linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org,
	Brian Starkey <brian.starkey@arm.com>,
	Daniel Vetter <daniel.vetter@intel.com>,
	Mark Brown <broonie@kernel.org>,
	Benjamin Gaignard <benjamin.gaignard@linaro.org>,
	linux-mm@kvack.org,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Subject: [PATCHv4 12/12] staging/android: Update Ion TODO list
Date: Tue, 18 Apr 2017 11:27:14 -0700	[thread overview]
Message-ID: <1492540034-5466-13-git-send-email-labbott@redhat.com> (raw)
In-Reply-To: <1492540034-5466-1-git-send-email-labbott@redhat.com>

Most of the items have been taken care of by a clean up series. Remove
the completed items and add a few new ones.

Signed-off-by: Laura Abbott <labbott@redhat.com>
---
 drivers/staging/android/TODO | 21 ++++-----------------
 1 file changed, 4 insertions(+), 17 deletions(-)

diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO
index 8f3ac37..5f14247 100644
--- a/drivers/staging/android/TODO
+++ b/drivers/staging/android/TODO
@@ -7,23 +7,10 @@ TODO:
 
 
 ion/
- - Remove ION_IOC_SYNC: Flushing for devices should be purely a kernel internal
-   interface on top of dma-buf. flush_for_device needs to be added to dma-buf
-   first.
- - Remove ION_IOC_CUSTOM: Atm used for cache flushing for cpu access in some
-   vendor trees. Should be replaced with an ioctl on the dma-buf to expose the
-   begin/end_cpu_access hooks to userspace.
- - Clarify the tricks ion plays with explicitly managing coherency behind the
-   dma api's back (this is absolutely needed for high-perf gpu drivers): Add an
-   explicit coherency management mode to flush_for_device to be used by drivers
-   which want to manage caches themselves and which indicates whether cpu caches
-   need flushing.
- - With those removed there's probably no use for ION_IOC_IMPORT anymore either
-   since ion would just be the central allocator for shared buffers.
- - Add dt-binding to expose cma regions as ion heaps, with the rule that any
-   such cma regions must already be used by some device for dma. I.e. ion only
-   exposes existing cma regions and doesn't reserve unecessarily memory when
-   booting a system which doesn't use ion.
+ - Add dt-bindings for remaining heaps (chunk and carveout heaps). This would
+   involve putting appropriate bindings in a memory node for Ion to find.
+ - Split /dev/ion up into multiple nodes (e.g. /dev/ion/heap0)
+ - Better test framework (integration with VGEM was suggested)
 
 Please send patches to Greg Kroah-Hartman <greg@kroah.com> and Cc:
 Arve Hjønnevåg <arve@android.com> and Riley Andrews <riandrews@android.com>
-- 
2.7.4

WARNING: multiple messages have this Message-ID (diff)
From: Laura Abbott <labbott@redhat.com>
To: Sumit Semwal <sumit.semwal@linaro.org>,
	Riley Andrews <riandrews@android.com>,
	arve@android.com, Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Laura Abbott <labbott@redhat.com>,
	romlem@google.com, devel@driverdev.osuosl.org,
	linux-kernel@vger.kernel.org, linaro-mm-sig@lists.linaro.org,
	linux-arm-kernel@lists.infradead.org,
	linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org,
	Brian Starkey <brian.starkey@arm.com>,
	Daniel Vetter <daniel.vetter@intel.com>,
	Mark Brown <broonie@kernel.org>,
	Benjamin Gaignard <benjamin.gaignard@linaro.org>,
	linux-mm@kvack.org,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Subject: [PATCHv4 12/12] staging/android: Update Ion TODO list
Date: Tue, 18 Apr 2017 11:27:14 -0700	[thread overview]
Message-ID: <1492540034-5466-13-git-send-email-labbott@redhat.com> (raw)
In-Reply-To: <1492540034-5466-1-git-send-email-labbott@redhat.com>

Most of the items have been taken care of by a clean up series. Remove
the completed items and add a few new ones.

Signed-off-by: Laura Abbott <labbott@redhat.com>
---
 drivers/staging/android/TODO | 21 ++++-----------------
 1 file changed, 4 insertions(+), 17 deletions(-)

diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO
index 8f3ac37..5f14247 100644
--- a/drivers/staging/android/TODO
+++ b/drivers/staging/android/TODO
@@ -7,23 +7,10 @@ TODO:
 
 
 ion/
- - Remove ION_IOC_SYNC: Flushing for devices should be purely a kernel internal
-   interface on top of dma-buf. flush_for_device needs to be added to dma-buf
-   first.
- - Remove ION_IOC_CUSTOM: Atm used for cache flushing for cpu access in some
-   vendor trees. Should be replaced with an ioctl on the dma-buf to expose the
-   begin/end_cpu_access hooks to userspace.
- - Clarify the tricks ion plays with explicitly managing coherency behind the
-   dma api's back (this is absolutely needed for high-perf gpu drivers): Add an
-   explicit coherency management mode to flush_for_device to be used by drivers
-   which want to manage caches themselves and which indicates whether cpu caches
-   need flushing.
- - With those removed there's probably no use for ION_IOC_IMPORT anymore either
-   since ion would just be the central allocator for shared buffers.
- - Add dt-binding to expose cma regions as ion heaps, with the rule that any
-   such cma regions must already be used by some device for dma. I.e. ion only
-   exposes existing cma regions and doesn't reserve unecessarily memory when
-   booting a system which doesn't use ion.
+ - Add dt-bindings for remaining heaps (chunk and carveout heaps). This would
+   involve putting appropriate bindings in a memory node for Ion to find.
+ - Split /dev/ion up into multiple nodes (e.g. /dev/ion/heap0)
+ - Better test framework (integration with VGEM was suggested)
 
 Please send patches to Greg Kroah-Hartman <greg@kroah.com> and Cc:
 Arve HjA,nnevAJPYg <arve@android.com> and Riley Andrews <riandrews@android.com>
-- 
2.7.4

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: labbott@redhat.com (Laura Abbott)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv4 12/12] staging/android: Update Ion TODO list
Date: Tue, 18 Apr 2017 11:27:14 -0700	[thread overview]
Message-ID: <1492540034-5466-13-git-send-email-labbott@redhat.com> (raw)
In-Reply-To: <1492540034-5466-1-git-send-email-labbott@redhat.com>

Most of the items have been taken care of by a clean up series. Remove
the completed items and add a few new ones.

Signed-off-by: Laura Abbott <labbott@redhat.com>
---
 drivers/staging/android/TODO | 21 ++++-----------------
 1 file changed, 4 insertions(+), 17 deletions(-)

diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO
index 8f3ac37..5f14247 100644
--- a/drivers/staging/android/TODO
+++ b/drivers/staging/android/TODO
@@ -7,23 +7,10 @@ TODO:
 
 
 ion/
- - Remove ION_IOC_SYNC: Flushing for devices should be purely a kernel internal
-   interface on top of dma-buf. flush_for_device needs to be added to dma-buf
-   first.
- - Remove ION_IOC_CUSTOM: Atm used for cache flushing for cpu access in some
-   vendor trees. Should be replaced with an ioctl on the dma-buf to expose the
-   begin/end_cpu_access hooks to userspace.
- - Clarify the tricks ion plays with explicitly managing coherency behind the
-   dma api's back (this is absolutely needed for high-perf gpu drivers): Add an
-   explicit coherency management mode to flush_for_device to be used by drivers
-   which want to manage caches themselves and which indicates whether cpu caches
-   need flushing.
- - With those removed there's probably no use for ION_IOC_IMPORT anymore either
-   since ion would just be the central allocator for shared buffers.
- - Add dt-binding to expose cma regions as ion heaps, with the rule that any
-   such cma regions must already be used by some device for dma. I.e. ion only
-   exposes existing cma regions and doesn't reserve unecessarily memory when
-   booting a system which doesn't use ion.
+ - Add dt-bindings for remaining heaps (chunk and carveout heaps). This would
+   involve putting appropriate bindings in a memory node for Ion to find.
+ - Split /dev/ion up into multiple nodes (e.g. /dev/ion/heap0)
+ - Better test framework (integration with VGEM was suggested)
 
 Please send patches to Greg Kroah-Hartman <greg@kroah.com> and Cc:
 Arve Hj?nnev?g <arve@android.com> and Riley Andrews <riandrews@android.com>
-- 
2.7.4

WARNING: multiple messages have this Message-ID (diff)
From: Laura Abbott <labbott@redhat.com>
To: Sumit Semwal <sumit.semwal@linaro.org>,
	Riley Andrews <riandrews@android.com>,
	arve@android.com, Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: devel@driverdev.osuosl.org, romlem@google.com,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
	linaro-mm-sig@lists.linaro.org, linux-mm@kvack.org,
	Mark Brown <broonie@kernel.org>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Daniel Vetter <daniel.vetter@intel.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-media@vger.kernel.org
Subject: [PATCHv4 12/12] staging/android: Update Ion TODO list
Date: Tue, 18 Apr 2017 11:27:14 -0700	[thread overview]
Message-ID: <1492540034-5466-13-git-send-email-labbott@redhat.com> (raw)
In-Reply-To: <1492540034-5466-1-git-send-email-labbott@redhat.com>

Most of the items have been taken care of by a clean up series. Remove
the completed items and add a few new ones.

Signed-off-by: Laura Abbott <labbott@redhat.com>
---
 drivers/staging/android/TODO | 21 ++++-----------------
 1 file changed, 4 insertions(+), 17 deletions(-)

diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO
index 8f3ac37..5f14247 100644
--- a/drivers/staging/android/TODO
+++ b/drivers/staging/android/TODO
@@ -7,23 +7,10 @@ TODO:
 
 
 ion/
- - Remove ION_IOC_SYNC: Flushing for devices should be purely a kernel internal
-   interface on top of dma-buf. flush_for_device needs to be added to dma-buf
-   first.
- - Remove ION_IOC_CUSTOM: Atm used for cache flushing for cpu access in some
-   vendor trees. Should be replaced with an ioctl on the dma-buf to expose the
-   begin/end_cpu_access hooks to userspace.
- - Clarify the tricks ion plays with explicitly managing coherency behind the
-   dma api's back (this is absolutely needed for high-perf gpu drivers): Add an
-   explicit coherency management mode to flush_for_device to be used by drivers
-   which want to manage caches themselves and which indicates whether cpu caches
-   need flushing.
- - With those removed there's probably no use for ION_IOC_IMPORT anymore either
-   since ion would just be the central allocator for shared buffers.
- - Add dt-binding to expose cma regions as ion heaps, with the rule that any
-   such cma regions must already be used by some device for dma. I.e. ion only
-   exposes existing cma regions and doesn't reserve unecessarily memory when
-   booting a system which doesn't use ion.
+ - Add dt-bindings for remaining heaps (chunk and carveout heaps). This would
+   involve putting appropriate bindings in a memory node for Ion to find.
+ - Split /dev/ion up into multiple nodes (e.g. /dev/ion/heap0)
+ - Better test framework (integration with VGEM was suggested)
 
 Please send patches to Greg Kroah-Hartman <greg@kroah.com> and Cc:
 Arve Hjønnevåg <arve@android.com> and Riley Andrews <riandrews@android.com>
-- 
2.7.4

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  parent reply	other threads:[~2017-04-18 18:28 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-18 18:27 [PATCHv4 00/12] Ion cleanup in preparation for moving out of staging Laura Abbott
2017-04-18 18:27 ` Laura Abbott
2017-04-18 18:27 ` Laura Abbott
2017-04-18 18:27 ` [PATCHv4 01/12] cma: Store a name in the cma structure Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27 ` [PATCHv4 02/12] cma: Introduce cma_for_each_area Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27 ` [PATCHv4 03/12] staging: android: ion: Use CMA APIs directly Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27 ` [PATCHv4 04/12] staging: android: ion: Stop butchering the DMA address Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27 ` [PATCHv4 05/12] staging: android: ion: Break the ABI in the name of forward progress Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-19  8:32   ` [Linaro-mm-sig] " Daniel Vetter
2017-04-19  8:32     ` Daniel Vetter
2017-04-19  8:32     ` Daniel Vetter
2017-04-18 18:27 ` [PATCHv4 06/12] staging: android: ion: Get rid of ion_phys_addr_t Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27 ` [PATCHv4 07/12] staging: android: ion: Collapse internal header files Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27 ` [PATCHv4 08/12] staging: android: ion: Rework heap registration/enumeration Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27 ` [PATCHv4 09/12] staging: android: ion: Drop ion_map_kernel interface Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27 ` [PATCHv4 10/12] staging: android: ion: Remove ion_handle and ion_client Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-19  8:34   ` [Linaro-mm-sig] " Daniel Vetter
2017-04-19  8:34     ` Daniel Vetter
2017-04-19  8:34     ` Daniel Vetter
2017-04-18 18:27 ` [PATCHv4 11/12] staging: android: ion: Set query return value Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27 ` Laura Abbott [this message]
2017-04-18 18:27   ` [PATCHv4 12/12] staging/android: Update Ion TODO list Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-18 18:27   ` Laura Abbott
2017-04-19  8:36   ` [Linaro-mm-sig] " Daniel Vetter
2017-04-19  8:36     ` Daniel Vetter
2017-04-19  8:36     ` Daniel Vetter
2017-04-19  8:36     ` Daniel Vetter
2017-04-19 12:42     ` Laurent Pinchart
2017-04-19 12:42       ` Laurent Pinchart
2017-04-19 12:42       ` Laurent Pinchart
2017-04-21 15:31 ` [PATCHv4 00/12] Ion cleanup in preparation for moving out of staging Sumit Semwal
2017-04-21 15:31   ` Sumit Semwal
2017-04-21 15:31   ` Sumit Semwal

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=1492540034-5466-13-git-send-email-labbott@redhat.com \
    --to=labbott@redhat.com \
    --cc=arve@android.com \
    --cc=benjamin.gaignard@linaro.org \
    --cc=brian.starkey@arm.com \
    --cc=broonie@kernel.org \
    --cc=daniel.vetter@intel.com \
    --cc=devel@driverdev.osuosl.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linaro-mm-sig@lists.linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=riandrews@android.com \
    --cc=romlem@google.com \
    --cc=sumit.semwal@linaro.org \
    /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: link
Be 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.