* [ndctl PATCH] Skip region filtering if numa_node attribute is not present
@ 2020-04-14 11:37 Santosh Sivaraj
2020-04-14 17:48 ` Dan Williams
0 siblings, 1 reply; 2+ messages in thread
From: Santosh Sivaraj @ 2020-04-14 11:37 UTC (permalink / raw)
To: linux-nvdimm, Dan Williams; +Cc: harish
For kernel versions older than 5.4, the numa_node attribute is not
present for regions; due to which `ndctl list -U 1` fails to list
namespaces.
Signed-off-by: Santosh Sivaraj <santosh@fossix.org>
---
ndctl/lib/libndctl.c | 11 +++++++++++
ndctl/lib/libndctl.sym | 1 +
ndctl/libndctl.h | 1 +
util/filter.c | 12 +++++++++++-
4 files changed, 24 insertions(+), 1 deletion(-)
diff --git a/ndctl/lib/libndctl.c b/ndctl/lib/libndctl.c
index ee737cb..fc82084 100644
--- a/ndctl/lib/libndctl.c
+++ b/ndctl/lib/libndctl.c
@@ -2471,6 +2471,17 @@ NDCTL_EXPORT struct ndctl_dimm *ndctl_region_get_next_dimm(struct ndctl_region *
return NULL;
}
+NDCTL_EXPORT int ndctl_region_has_numa_attr(struct ndctl_region *region)
+{
+ char *path = region->region_buf;
+
+ sprintf(path, "%s/numa_node", region->region_path);
+ if (access(path, F_OK) != -1)
+ return 1;
+
+ return 0;
+}
+
NDCTL_EXPORT int ndctl_region_get_numa_node(struct ndctl_region *region)
{
return region->numa_node;
diff --git a/ndctl/lib/libndctl.sym b/ndctl/lib/libndctl.sym
index ac575a2..b7c72a2 100644
--- a/ndctl/lib/libndctl.sym
+++ b/ndctl/lib/libndctl.sym
@@ -430,4 +430,5 @@ LIBNDCTL_23 {
ndctl_region_get_target_node;
ndctl_region_get_align;
ndctl_region_set_align;
+ ndctl_region_has_numa_attr;
} LIBNDCTL_22;
diff --git a/ndctl/libndctl.h b/ndctl/libndctl.h
index 2580f43..4e233d8 100644
--- a/ndctl/libndctl.h
+++ b/ndctl/libndctl.h
@@ -385,6 +385,7 @@ struct ndctl_dimm *ndctl_region_get_first_dimm(struct ndctl_region *region);
struct ndctl_dimm *ndctl_region_get_next_dimm(struct ndctl_region *region,
struct ndctl_dimm *dimm);
int ndctl_region_get_numa_node(struct ndctl_region *region);
+int ndctl_region_has_numa_attr(struct ndctl_region *region);
int ndctl_region_get_target_node(struct ndctl_region *region);
struct ndctl_region *ndctl_bus_get_region_by_physical_address(struct ndctl_bus *bus,
unsigned long long address);
diff --git a/util/filter.c b/util/filter.c
index af72793..8e60cfa 100644
--- a/util/filter.c
+++ b/util/filter.c
@@ -467,7 +467,13 @@ int util_filter_walk(struct ndctl_ctx *ctx, struct util_filter_ctx *fctx,
param->namespace))
continue;
- if (numa_node != NUMA_NO_NODE &&
+ /*
+ * if numa_node attribute is not available for regions
+ * (which is true for pre 5.4 kernels), don't skip the
+ * region, let namespace filter handle the filtering.
+ */
+ if (ndctl_region_has_numa_attr(region) &&
+ numa_node != NUMA_NO_NODE &&
ndctl_region_get_numa_node(region) != numa_node)
continue;
@@ -489,6 +495,10 @@ int util_filter_walk(struct ndctl_ctx *ctx, struct util_filter_ctx *fctx,
if (param->mode && util_nsmode(param->mode) != mode)
continue;
+ if (numa_node != NUMA_NO_NODE &&
+ ndctl_namespace_get_numa_node(ndns) != numa_node)
+ continue;
+
fctx->filter_namespace(ndns, fctx);
}
}
--
2.25.2
_______________________________________________
Linux-nvdimm mailing list -- linux-nvdimm@lists.01.org
To unsubscribe send an email to linux-nvdimm-leave@lists.01.org
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [ndctl PATCH] Skip region filtering if numa_node attribute is not present
2020-04-14 11:37 [ndctl PATCH] Skip region filtering if numa_node attribute is not present Santosh Sivaraj
@ 2020-04-14 17:48 ` Dan Williams
0 siblings, 0 replies; 2+ messages in thread
From: Dan Williams @ 2020-04-14 17:48 UTC (permalink / raw)
To: Santosh Sivaraj; +Cc: linux-nvdimm, harish
On Tue, Apr 14, 2020 at 4:39 AM Santosh Sivaraj <santosh@fossix.org> wrote:
>
> For kernel versions older than 5.4, the numa_node attribute is not
> present for regions; due to which `ndctl list -U 1` fails to list
> namespaces.
>
> Signed-off-by: Santosh Sivaraj <santosh@fossix.org>
> ---
> ndctl/lib/libndctl.c | 11 +++++++++++
> ndctl/lib/libndctl.sym | 1 +
> ndctl/libndctl.h | 1 +
> util/filter.c | 12 +++++++++++-
> 4 files changed, 24 insertions(+), 1 deletion(-)
>
> diff --git a/ndctl/lib/libndctl.c b/ndctl/lib/libndctl.c
> index ee737cb..fc82084 100644
> --- a/ndctl/lib/libndctl.c
> +++ b/ndctl/lib/libndctl.c
> @@ -2471,6 +2471,17 @@ NDCTL_EXPORT struct ndctl_dimm *ndctl_region_get_next_dimm(struct ndctl_region *
> return NULL;
> }
>
> +NDCTL_EXPORT int ndctl_region_has_numa_attr(struct ndctl_region *region)
I'd shorten this to just ndctl_region_has_numa().
> +{
> + char *path = region->region_buf;
> +
> + sprintf(path, "%s/numa_node", region->region_path);
> + if (access(path, F_OK) != -1)
> + return 1;
You should be able to cache this result at the beginning of time at
add_region() time to check for the ENOENT error return here:
sprintf(path, "%s/numa_node", region_base);
if (sysfs_read_attr(ctx, path, buf) == 0)
region->numa_node = strtol(buf, NULL, 0);
else
region->numa_node = -1;
> +
> + return 0;
> +}
> +
> NDCTL_EXPORT int ndctl_region_get_numa_node(struct ndctl_region *region)
> {
> return region->numa_node;
> diff --git a/ndctl/lib/libndctl.sym b/ndctl/lib/libndctl.sym
> index ac575a2..b7c72a2 100644
> --- a/ndctl/lib/libndctl.sym
> +++ b/ndctl/lib/libndctl.sym
> @@ -430,4 +430,5 @@ LIBNDCTL_23 {
> ndctl_region_get_target_node;
> ndctl_region_get_align;
> ndctl_region_set_align;
> + ndctl_region_has_numa_attr;
> } LIBNDCTL_22;
> diff --git a/ndctl/libndctl.h b/ndctl/libndctl.h
> index 2580f43..4e233d8 100644
> --- a/ndctl/libndctl.h
> +++ b/ndctl/libndctl.h
> @@ -385,6 +385,7 @@ struct ndctl_dimm *ndctl_region_get_first_dimm(struct ndctl_region *region);
> struct ndctl_dimm *ndctl_region_get_next_dimm(struct ndctl_region *region,
> struct ndctl_dimm *dimm);
> int ndctl_region_get_numa_node(struct ndctl_region *region);
> +int ndctl_region_has_numa_attr(struct ndctl_region *region);
> int ndctl_region_get_target_node(struct ndctl_region *region);
> struct ndctl_region *ndctl_bus_get_region_by_physical_address(struct ndctl_bus *bus,
> unsigned long long address);
> diff --git a/util/filter.c b/util/filter.c
> index af72793..8e60cfa 100644
> --- a/util/filter.c
> +++ b/util/filter.c
> @@ -467,7 +467,13 @@ int util_filter_walk(struct ndctl_ctx *ctx, struct util_filter_ctx *fctx,
> param->namespace))
> continue;
>
> - if (numa_node != NUMA_NO_NODE &&
> + /*
> + * if numa_node attribute is not available for regions
> + * (which is true for pre 5.4 kernels), don't skip the
> + * region, let namespace filter handle the filtering.
> + */
> + if (ndctl_region_has_numa_attr(region) &&
> + numa_node != NUMA_NO_NODE &&
This should also check that namespaces are being requested otherwise
failing here is the right thing to do. An error message stating that
the kernel lacks region-numa support when no namespaces are present
would also be useful.
_______________________________________________
Linux-nvdimm mailing list -- linux-nvdimm@lists.01.org
To unsubscribe send an email to linux-nvdimm-leave@lists.01.org
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-04-14 17:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-14 11:37 [ndctl PATCH] Skip region filtering if numa_node attribute is not present Santosh Sivaraj
2020-04-14 17:48 ` Dan Williams
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).