[RFC PATCH 08/12] cma: Store a name in the cma structure

On 03/10/2017 12:53 AM, Sumit Semwal wrote:

Hi Laura,

Thanks for the patch.

On 3 March 2017 at 03:14, Laura Abbott [off-list ref] wrote:

quoted

Frameworks that may want to enumerate CMA heaps (e.g. Ion) will find it
useful to have an explicit name attached to each region. Store the name
in each CMA structure.

Signed-off-by: Laura Abbott <redacted>
---
 drivers/base/dma-contiguous.c |  5 +++--
 include/linux/cma.h           |  4 +++-
 mm/cma.c                      | 11 +++++++++--
 mm/cma.h                      |  1 +
 mm/cma_debug.c                |  2 +-
 5 files changed, 17 insertions(+), 6 deletions(-)

<snip>

quoted

+const char *cma_get_name(const struct cma *cma)
+{
+       return cma->name ? cma->name : "(undefined)";
+}
+

Would it make sense to perhaps have the idx stored as the name,
instead of 'undefined'? That would make sure that the various cma
names are still unique.

Good suggestion. I'll see about cleaning that up.

quoted

 static unsigned long cma_bitmap_aligned_mask(const struct cma *cma,
                                             int align_order)
 {

@@ -168,6 +173,7 @@ core_initcall(cma_init_reserved_areas);
  */
 int __init cma_init_reserved_mem(phys_addr_t base, phys_addr_t size,
                                 unsigned int order_per_bit,
+                                const char *name,
                                 struct cma **res_cma)
 {

Best regards,
Sumit.