Blame SOURCES/0092-cxl-list-Support-multiple-memdev-device-name-filter-.patch

e0018b
From 0ce0152d8e29f85325a3a59f94051228540abf6a Mon Sep 17 00:00:00 2001
e0018b
From: Dan Williams <dan.j.williams@intel.com>
e0018b
Date: Sun, 23 Jan 2022 16:52:20 -0800
e0018b
Subject: [PATCH 092/217] cxl/list: Support multiple memdev device name filter
e0018b
 arguments
e0018b
e0018b
Similar to 'ndctl list', allow for a syntax like:
e0018b
e0018b
    cxl list -m "$(seq -s ' ' 2 5)"
e0018b
e0018b
...to filter the output to just those 4 memdevs.
e0018b
e0018b
Link: https://lore.kernel.org/r/164298554075.3021641.17678360870961637912.stgit@dwillia2-desk3.amr.corp.intel.com
e0018b
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
e0018b
Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
e0018b
---
e0018b
 Documentation/cxl/cxl-list.txt | 21 ++++++++++++++++++-
e0018b
 cxl/filter.c                   | 38 ++++++++++++++++++++++++----------
e0018b
 2 files changed, 47 insertions(+), 12 deletions(-)
e0018b
e0018b
diff --git a/Documentation/cxl/cxl-list.txt b/Documentation/cxl/cxl-list.txt
e0018b
index c8d10fb..686e0ea 100644
e0018b
--- a/Documentation/cxl/cxl-list.txt
e0018b
+++ b/Documentation/cxl/cxl-list.txt
e0018b
@@ -30,7 +30,7 @@ OPTIONS
e0018b
 -------
e0018b
 -m::
e0018b
 --memdev=::
e0018b
-	Specify a cxl memory device name to filter the listing. For example:
e0018b
+	Specify CXL memory device name(s), or device id(s), to filter the listing. For example:
e0018b
 ----
e0018b
 # cxl list --memdev=mem0
e0018b
 {
e0018b
@@ -38,6 +38,25 @@ OPTIONS
e0018b
   "pmem_size":268435456,
e0018b
   "ram_size":0,
e0018b
 }
e0018b
+
e0018b
+# cxl list -m "0 mem1 2"
e0018b
+[
e0018b
+  {
e0018b
+    "memdev":"mem0",
e0018b
+    "pmem_size":268435456,
e0018b
+    "ram_size":0
e0018b
+  },
e0018b
+  {
e0018b
+    "memdev":"mem2",
e0018b
+    "pmem_size":268435456,
e0018b
+    "ram_size":268435456
e0018b
+  },
e0018b
+  {
e0018b
+    "memdev":"mem1",
e0018b
+    "pmem_size":268435456,
e0018b
+    "ram_size":268435456
e0018b
+  }
e0018b
+]
e0018b
 ----
e0018b
 
e0018b
 -M::
e0018b
diff --git a/cxl/filter.c b/cxl/filter.c
e0018b
index 21322ed..efafaf5 100644
e0018b
--- a/cxl/filter.c
e0018b
+++ b/cxl/filter.c
e0018b
@@ -2,24 +2,40 @@
e0018b
 // Copyright (C) 2015-2020 Intel Corporation. All rights reserved.
e0018b
 #include <stdio.h>
e0018b
 #include <string.h>
e0018b
+#include <stdlib.h>
e0018b
 #include <cxl/libcxl.h>
e0018b
 #include "filter.h"
e0018b
 
e0018b
 struct cxl_memdev *util_cxl_memdev_filter(struct cxl_memdev *memdev,
e0018b
-                                         const char *ident)
e0018b
+					  const char *__ident)
e0018b
 {
e0018b
-       int memdev_id;
e0018b
+	char *ident, *save;
e0018b
+	const char *name;
e0018b
+	int memdev_id;
e0018b
 
e0018b
-       if (!ident || strcmp(ident, "all") == 0)
e0018b
-               return memdev;
e0018b
+	if (!__ident)
e0018b
+		return memdev;
e0018b
 
e0018b
-       if (strcmp(ident, cxl_memdev_get_devname(memdev)) == 0)
e0018b
-               return memdev;
e0018b
+	ident = strdup(__ident);
e0018b
+	if (!ident)
e0018b
+		return NULL;
e0018b
 
e0018b
-       if ((sscanf(ident, "%d", &memdev_id) == 1
e0018b
-                       || sscanf(ident, "mem%d", &memdev_id) == 1)
e0018b
-                       && cxl_memdev_get_id(memdev) == memdev_id)
e0018b
-               return memdev;
e0018b
+	for (name = strtok_r(ident, " ", &save); name;
e0018b
+	     name = strtok_r(NULL, " ", &save)) {
e0018b
+		if (strcmp(name, "all") == 0)
e0018b
+			break;
e0018b
 
e0018b
-       return NULL;
e0018b
+		if ((sscanf(name, "%d", &memdev_id) == 1 ||
e0018b
+		     sscanf(name, "mem%d", &memdev_id) == 1) &&
e0018b
+		    cxl_memdev_get_id(memdev) == memdev_id)
e0018b
+			break;
e0018b
+
e0018b
+		if (strcmp(name, cxl_memdev_get_devname(memdev)) == 0)
e0018b
+			break;
e0018b
+	}
e0018b
+
e0018b
+	free(ident);
e0018b
+	if (name)
e0018b
+		return memdev;
e0018b
+	return NULL;
e0018b
 }
e0018b
-- 
e0018b
2.27.0
e0018b