[PATCH ndctl v1 00/10] daxctl: Support for sub-dividing soft-reserved regions

[PATCH v2 00/12] ACPI/NVDIMM:...

[GIT PULL] libnvdimm fix for...

Joao Martins

Friday, 3 April 2020 Fri, 3 Apr '20

1:58 p.m.

Hey, This series introduces the daxctl support for sub-dividing soft-reserved regions created by EFI/HMAT/efi_fake_mem. It's the userspace counterpart of this recent patch series [0]. These new 'dynamic' regions can be partitioned into multiple different devices which its subdivisions can consist of one or more ranges. This is in contrast to static dax regions -- created with ndctl-create-namespace -m devdax -- which can't be subdivided neither discontiguous. See also cover-letter of [0]. The daxctl changes in these patches are depicted as: * {create,destroy,disable,enable}-device: These orchestrate/manage the sub-division devices. It mimmics the same as namespaces equivalent commands. * Allow reconfigure-device to change the size of an existing *dynamic* dax device. * Add test coverage (so far tried to cover all range allocation code paths, but I am still fishing for bugs). Additionally, there are bugs so applying [0] may not make it pass the added functional test yet. I am sending the series earlier (i.e. before the kernel patches get merged) mainly to share a common unit tests, and also letting others try it out. The only TODOs left is documentation, and perhaps listing of the mappingX sysfs entries. Thoughts, comments appreciated. :) Thanks! Joao [0] "device-dax: Support sub-dividing soft-reserved ranges", https://lore.kernel.org/linux-nvdimm/158500767138.2088294.171316462598039... Dan Williams (1): daxctl: Cleanup whitespace Joao Martins (9): libdaxctl: add daxctl_dev_set_size() daxctl: add resize support in reconfigure-device daxctl: add command to disable devdax device daxctl: add command to enable devdax device libdaxctl: add daxctl_region_create_dev() daxctl: add command to create device libdaxctl: add daxctl_region_destroy_dev() daxctl: add command to destroy device daxctl/test: Add tests for dynamic dax regions daxctl/builtin.h | 4 + daxctl/daxctl.c | 4 + daxctl/device.c | 310 ++++++++++++++++++++++++++++++++++++++- daxctl/lib/libdaxctl.c | 67 +++++++++ daxctl/lib/libdaxctl.sym | 7 + daxctl/libdaxctl.h | 3 + test/Makefile.am | 1 + test/daxctl-create.sh | 293 ++++++++++++++++++++++++++++++++++++ util/filter.c | 2 +- 9 files changed, 686 insertions(+), 5 deletions(-) create mode 100755 test/daxctl-create.sh -- 2.17.1

Show replies by date

Joao Martins

Friday, 3 April Fri, 3 Apr

1:58 p.m.

New subject: [PATCH ndctl v1 01/10] daxctl: Cleanup whitespace

From: Dan Williams <dan.j.williams(a)intel.com> Re-indent util_daxctl_region_filter() to match expectations. Signed-off-by: Dan Williams <dan.j.williams(a)intel.com> --- util/filter.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/filter.c b/util/filter.c index af72793929e2..7d0159f8cef6 100644 --- a/util/filter.c +++ b/util/filter.c @@ -344,7 +344,7 @@ struct daxctl_region *util_daxctl_region_filter(struct daxctl_region *region, return region; if ((sscanf(ident, "%d", &region_id) == 1 - || sscanf(ident, "region%d", &region_id) == 1) + || sscanf(ident, "region%d", &region_id) == 1) && daxctl_region_get_id(region) == region_id) return region; -- 2.17.1

Joao Martins

1:58 p.m.

New subject: [PATCH ndctl v1 02/10] libdaxctl: add daxctl_dev_set_size()

Add an API for setting the size of a dax device. This sysfs attribute is only writeable for a dynamic dax device such as the one exported by hmem. Signed-off-by: Joao Martins <joao.m.martins(a)oracle.com> Signed-off-by: Dan Williams <dan.j.williams(a)intel.com> --- daxctl/lib/libdaxctl.c | 24 ++++++++++++++++++++++++ daxctl/lib/libdaxctl.sym | 5 +++++ daxctl/libdaxctl.h | 1 + 3 files changed, 30 insertions(+) diff --git a/daxctl/lib/libdaxctl.c b/daxctl/lib/libdaxctl.c index ee4a069eb463..00d5f7fe61ad 100644 --- a/daxctl/lib/libdaxctl.c +++ b/daxctl/lib/libdaxctl.c @@ -1019,6 +1019,30 @@ DAXCTL_EXPORT unsigned long long daxctl_dev_get_size(struct daxctl_dev *dev) return dev->size; } +DAXCTL_EXPORT int daxctl_dev_set_size(struct daxctl_dev *dev, unsigned long long size) +{ + struct daxctl_ctx *ctx = daxctl_dev_get_ctx(dev); + char buf[SYSFS_ATTR_SIZE]; + char *path = dev->dev_buf; + int len = dev->buf_len; + + if (snprintf(path, len, "%s/size", dev->dev_path) >= len) { + err(ctx, "%s: buffer too small!\n", + daxctl_dev_get_devname(dev)); + return -ENXIO; + } + + sprintf(buf, "%#llx\n", size); + if (sysfs_write_attr(ctx, path, buf) < 0) { + err(ctx, "%s: failed to set size\n", + daxctl_dev_get_devname(dev)); + return -ENXIO; + } + + dev->size = size; + return 0; +} + DAXCTL_EXPORT int daxctl_dev_get_target_node(struct daxctl_dev *dev) { return dev->target_node; diff --git a/daxctl/lib/libdaxctl.sym b/daxctl/lib/libdaxctl.sym index 87d02366a917..d8b4137c76b7 100644 --- a/daxctl/lib/libdaxctl.sym +++ b/daxctl/lib/libdaxctl.sym @@ -75,3 +75,8 @@ global: daxctl_memory_is_movable; daxctl_memory_online_no_movable; } LIBDAXCTL_6; + +LIBDAXCTL_8 { +global: + daxctl_dev_set_size; +} LIBDAXCTL_7; diff --git a/daxctl/libdaxctl.h b/daxctl/libdaxctl.h index 6c545e1f3055..e3d482743cc6 100644 --- a/daxctl/libdaxctl.h +++ b/daxctl/libdaxctl.h @@ -68,6 +68,7 @@ int daxctl_dev_get_major(struct daxctl_dev *dev); int daxctl_dev_get_minor(struct daxctl_dev *dev); unsigned long long daxctl_dev_get_resource(struct daxctl_dev *dev); unsigned long long daxctl_dev_get_size(struct daxctl_dev *dev); +int daxctl_dev_set_size(struct daxctl_dev *dev, unsigned long long size); struct daxctl_ctx *daxctl_dev_get_ctx(struct daxctl_dev *dev); int daxctl_dev_is_enabled(struct daxctl_dev *dev); int daxctl_dev_disable(struct daxctl_dev *dev); -- 2.17.1

Joao Martins

1:58 p.m.

New subject: [PATCH ndctl v1 03/10] daxctl: add resize support in reconfigure-device

Add the ability to change the size of an inactive dax device. Uses of this includes adjusting existing devices or enterily freeing the region device to make the space available. $ daxctl disable-device dax0.0 disabled 1 device $ daxctl reconfigure-device -s 0 dax0.0 reconfigured 1 device $ daxctl reconfigure-device -s 4G dax0.0 reconfigured 1 device @size (-s) and @mode (-m) are mutually exclusive as the latter relates to assigning memory to System-RAM through kmem as opposed to reconfiguring dynamic dax devices. Signed-off-by: Joao Martins <joao.m.martins(a)oracle.com> Signed-off-by: Dan Williams <dan.j.williams(a)intel.com> --- daxctl/device.c | 26 +++++++++++++++++++++++--- 1 file changed, 23 insertions(+), 3 deletions(-) diff --git a/daxctl/device.c b/daxctl/device.c index 705f1f8ff7f6..b483d2777ecb 100644 --- a/daxctl/device.c +++ b/daxctl/device.c @@ -34,6 +34,7 @@ enum dev_mode { }; static enum dev_mode reconfig_mode = DAXCTL_DEV_MODE_UNKNOWN; +static long long size = -1; static unsigned long flags; enum memory_zone { @@ -66,6 +67,7 @@ OPT_BOOLEAN('\0', "no-movable", &param.no_movable, \ static const struct option reconfig_options[] = { BASE_OPTIONS(), + CREATE_OPTIONS(), RECONFIG_OPTIONS(), ZONE_OPTIONS(), OPT_END(), @@ -135,12 +137,14 @@ static const char *parse_device_options(int argc, const char **argv, /* Handle action-specific options */ switch (action) { case ACTION_RECONFIG: - if (!param.mode) { - fprintf(stderr, "error: a 'mode' option is required\n"); + if (!param.size && !param.mode) { + fprintf(stderr, "error: a 'mode' or 'size' option is required\n"); usage_with_options(u, reconfig_options); rc = -EINVAL; } - if (strcmp(param.mode, "system-ram") == 0) { + if (param.size) { + size = __parse_size64(param.size, &units); + } else if (strcmp(param.mode, "system-ram") == 0) { reconfig_mode = DAXCTL_DEV_MODE_RAM; if (param.no_movable) mem_zone = MEM_ZONE_NORMAL; @@ -309,6 +313,17 @@ static int dev_offline_memory(struct daxctl_dev *dev) return 0; } +static int dev_resize(struct daxctl_dev *dev, unsigned long long val) +{ + int rc; + + rc = daxctl_dev_set_size(dev, val); + if (rc < 0) + return rc; + + return 0; +} + static int disable_devdax_device(struct daxctl_dev *dev) { struct daxctl_memory *mem = daxctl_dev_get_memory(dev); @@ -418,6 +433,11 @@ static int do_reconfig(struct daxctl_dev *dev, enum dev_mode mode, struct json_object *jdev; int rc = 0; + if (size >= 0) { + rc = dev_resize(dev, size); + return rc; + } + switch (mode) { case DAXCTL_DEV_MODE_RAM: rc = reconfig_mode_system_ram(dev); -- 2.17.1

Joao Martins

1:58 p.m.

New subject: [PATCH ndctl v1 04/10] daxctl: add command to disable devdax device

Add a 'disable-device' command, required prior to reconfiguration or destruction of the dax device. Mimmics the same functionality as seen in ndctl-disable-namespace. Signed-off-by: Joao Martins <joao.m.martins(a)oracle.com> Signed-off-by: Dan Williams <dan.j.williams(a)intel.com> --- daxctl/builtin.h | 1 + daxctl/daxctl.c | 1 + daxctl/device.c | 61 ++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 63 insertions(+) diff --git a/daxctl/builtin.h b/daxctl/builtin.h index f5a0147f0e11..c9848953bbd8 100644 --- a/daxctl/builtin.h +++ b/daxctl/builtin.h @@ -7,6 +7,7 @@ struct daxctl_ctx; int cmd_list(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_migrate(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_reconfig_device(int argc, const char **argv, struct daxctl_ctx *ctx); +int cmd_disable_device(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_online_memory(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_offline_memory(int argc, const char **argv, struct daxctl_ctx *ctx); #endif /* _DAXCTL_BUILTIN_H_ */ diff --git a/daxctl/daxctl.c b/daxctl/daxctl.c index 1ab073200313..1707a9ff0791 100644 --- a/daxctl/daxctl.c +++ b/daxctl/daxctl.c @@ -74,6 +74,7 @@ static struct cmd_struct commands[] = { { "reconfigure-device", .d_fn = cmd_reconfig_device }, { "online-memory", .d_fn = cmd_online_memory }, { "offline-memory", .d_fn = cmd_offline_memory }, + { "disable-device", .d_fn = cmd_disable_device }, }; int main(int argc, const char **argv) diff --git a/daxctl/device.c b/daxctl/device.c index b483d2777ecb..7a49852f78f4 100644 --- a/daxctl/device.c +++ b/daxctl/device.c @@ -47,6 +47,7 @@ enum device_action { ACTION_RECONFIG, ACTION_ONLINE, ACTION_OFFLINE, + ACTION_DISABLE, }; #define BASE_OPTIONS() \ @@ -84,6 +85,11 @@ static const struct option offline_options[] = { OPT_END(), }; +static const struct option disable_options[] = { + BASE_OPTIONS(), + OPT_END(), +}; + static const char *parse_device_options(int argc, const char **argv, enum device_action action, const struct option *options, const char *usage, struct daxctl_ctx *ctx) @@ -110,6 +116,9 @@ static const char *parse_device_options(int argc, const char **argv, case ACTION_OFFLINE: action_string = "offline memory for"; break; + case ACTION_DISABLE: + action_string = "disable"; + break; default: action_string = "<>"; break; @@ -162,6 +171,7 @@ static const char *parse_device_options(int argc, const char **argv, mem_zone = MEM_ZONE_NORMAL; /* fall through */ case ACTION_OFFLINE: + case ACTION_DISABLE: /* nothing special */ break; } @@ -491,6 +501,35 @@ static int do_xline(struct daxctl_dev *dev, enum device_action action) return rc; } +static int do_xble(struct daxctl_dev *dev, enum device_action action) +{ + struct daxctl_memory *mem = daxctl_dev_get_memory(dev); + const char *devname = daxctl_dev_get_devname(dev); + int rc; + + if (mem) { + fprintf(stderr, + "%s: status operations are only applicable in devdax mode\n", + devname); + return -ENXIO; + } + + switch (action) { + case ACTION_DISABLE: + rc = daxctl_dev_disable(dev); + if (rc) { + fprintf(stderr, "%s: disable failed: %s\n", + daxctl_dev_get_devname(dev), strerror(-rc)); + return rc; + } + break; + default: + fprintf(stderr, "%s: invalid action: %d\n", devname, action); + rc = -EINVAL; + } + return rc; +} + static int do_xaction_device(const char *device, enum device_action action, struct daxctl_ctx *ctx, int *processed) { @@ -525,6 +564,11 @@ static int do_xaction_device(const char *device, enum device_action action, if (rc == 0) (*processed)++; break; + case ACTION_DISABLE: + rc = do_xble(dev, action); + if (rc == 0) + (*processed)++; + break; default: rc = -EINVAL; break; @@ -560,6 +604,23 @@ int cmd_reconfig_device(int argc, const char **argv, struct daxctl_ctx *ctx) return rc; } +int cmd_disable_device(int argc, const char **argv, struct daxctl_ctx *ctx) +{ + char *usage = "daxctl disable-device <device>"; + const char *device = parse_device_options(argc, argv, ACTION_DISABLE, + disable_options, usage, ctx); + int processed, rc; + + rc = do_xaction_device(device, ACTION_DISABLE, ctx, &processed); + if (rc < 0) + fprintf(stderr, "error disabling device: %s\n", + strerror(-rc)); + + fprintf(stderr, "disabled %d device%s\n", processed, + processed == 1 ? "" : "s"); + return rc; +} + int cmd_online_memory(int argc, const char **argv, struct daxctl_ctx *ctx) { char *usage = "daxctl online-memory <device> [<options>]"; -- 2.17.1

Joao Martins

1:58 p.m.

New subject: [PATCH ndctl v1 05/10] daxctl: add command to enable devdax device

Add a 'enable-device' command, required prior to reconfiguration of the dax device. Mimics the same functionality as seen in ndctl-enable-namespace. Signed-off-by: Joao Martins <joao.m.martins(a)oracle.com> Signed-off-by: Dan Williams <dan.j.williams(a)intel.com> --- daxctl/builtin.h | 1 + daxctl/daxctl.c | 1 + daxctl/device.c | 40 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 42 insertions(+) diff --git a/daxctl/builtin.h b/daxctl/builtin.h index c9848953bbd8..8f344f86ad20 100644 --- a/daxctl/builtin.h +++ b/daxctl/builtin.h @@ -8,6 +8,7 @@ int cmd_list(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_migrate(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_reconfig_device(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_disable_device(int argc, const char **argv, struct daxctl_ctx *ctx); +int cmd_enable_device(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_online_memory(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_offline_memory(int argc, const char **argv, struct daxctl_ctx *ctx); #endif /* _DAXCTL_BUILTIN_H_ */ diff --git a/daxctl/daxctl.c b/daxctl/daxctl.c index 1707a9ff0791..a4699b3780bd 100644 --- a/daxctl/daxctl.c +++ b/daxctl/daxctl.c @@ -75,6 +75,7 @@ static struct cmd_struct commands[] = { { "online-memory", .d_fn = cmd_online_memory }, { "offline-memory", .d_fn = cmd_offline_memory }, { "disable-device", .d_fn = cmd_disable_device }, + { "enable-device", .d_fn = cmd_enable_device }, }; int main(int argc, const char **argv) diff --git a/daxctl/device.c b/daxctl/device.c index 7a49852f78f4..596b6efca34d 100644 --- a/daxctl/device.c +++ b/daxctl/device.c @@ -48,6 +48,7 @@ enum device_action { ACTION_ONLINE, ACTION_OFFLINE, ACTION_DISABLE, + ACTION_ENABLE, }; #define BASE_OPTIONS() \ @@ -90,6 +91,11 @@ static const struct option disable_options[] = { OPT_END(), }; +static const struct option enable_options[] = { + BASE_OPTIONS(), + OPT_END(), +}; + static const char *parse_device_options(int argc, const char **argv, enum device_action action, const struct option *options, const char *usage, struct daxctl_ctx *ctx) @@ -119,6 +125,9 @@ static const char *parse_device_options(int argc, const char **argv, case ACTION_DISABLE: action_string = "disable"; break; + case ACTION_ENABLE: + action_string = "enable"; + break; default: action_string = "<>"; break; @@ -172,6 +181,7 @@ static const char *parse_device_options(int argc, const char **argv, /* fall through */ case ACTION_OFFLINE: case ACTION_DISABLE: + case ACTION_ENABLE: /* nothing special */ break; } @@ -515,6 +525,14 @@ static int do_xble(struct daxctl_dev *dev, enum device_action action) } switch (action) { + case ACTION_ENABLE: + rc = daxctl_dev_enable_devdax(dev); + if (rc) { + fprintf(stderr, "%s: enable failed: %s\n", + daxctl_dev_get_devname(dev), strerror(-rc)); + return rc; + } + break; case ACTION_DISABLE: rc = daxctl_dev_disable(dev); if (rc) { @@ -564,6 +582,11 @@ static int do_xaction_device(const char *device, enum device_action action, if (rc == 0) (*processed)++; break; + case ACTION_ENABLE: + rc = do_xble(dev, action); + if (rc == 0) + (*processed)++; + break; case ACTION_DISABLE: rc = do_xble(dev, action); if (rc == 0) @@ -621,6 +644,23 @@ int cmd_disable_device(int argc, const char **argv, struct daxctl_ctx *ctx) return rc; } +int cmd_enable_device(int argc, const char **argv, struct daxctl_ctx *ctx) +{ + char *usage = "daxctl enable-device <device>"; + const char *device = parse_device_options(argc, argv, ACTION_DISABLE, + enable_options, usage, ctx); + int processed, rc; + + rc = do_xaction_device(device, ACTION_ENABLE, ctx, &processed); + if (rc < 0) + fprintf(stderr, "error enabling device: %s\n", + strerror(-rc)); + + fprintf(stderr, "enabled %d device%s\n", processed, + processed == 1 ? "" : "s"); + return rc; +} + int cmd_online_memory(int argc, const char **argv, struct daxctl_ctx *ctx) { char *usage = "daxctl online-memory <device> [<options>]"; -- 2.17.1

Joao Martins

1:58 p.m.

New subject: [PATCH ndctl v1 06/10] libdaxctl: add daxctl_region_create_dev()

Add an API to create an empty seed device. This sysfs attribute is only writeable for a dynamic dax device such as the one exported by hmem. Signed-off-by: Joao Martins <joao.m.martins(a)oracle.com> Signed-off-by: Dan Williams <dan.j.williams(a)intel.com> --- daxctl/lib/libdaxctl.c | 26 ++++++++++++++++++++++++++ daxctl/lib/libdaxctl.sym | 1 + daxctl/libdaxctl.h | 1 + 3 files changed, 28 insertions(+) diff --git a/daxctl/lib/libdaxctl.c b/daxctl/lib/libdaxctl.c index 00d5f7fe61ad..d17ff7a02bad 100644 --- a/daxctl/lib/libdaxctl.c +++ b/daxctl/lib/libdaxctl.c @@ -583,6 +583,32 @@ DAXCTL_EXPORT unsigned long long daxctl_region_get_available_size( return 0; } +DAXCTL_EXPORT int daxctl_region_create_dev(struct daxctl_region *region) +{ + struct daxctl_ctx *ctx = daxctl_region_get_ctx(region); + char *path = region->region_buf; + int rc, len = region->buf_len; + char *num_devices; + + if (snprintf(path, len, "%s/%s/create", region->region_path, attrs) >= len) { + err(ctx, "%s: buffer too small!\n", + daxctl_region_get_devname(region)); + return -EFAULT; + } + + if (asprintf(&num_devices, "%d", 1) < 0) { + err(ctx, "%s: buffer too small!\n", + daxctl_region_get_devname(region)); + return -EFAULT; + } + + rc = sysfs_write_attr(ctx, path, num_devices); + if (rc) + return rc; + + return 0; +} + DAXCTL_EXPORT struct daxctl_dev *daxctl_region_get_dev_seed( struct daxctl_region *region) { diff --git a/daxctl/lib/libdaxctl.sym b/daxctl/lib/libdaxctl.sym index d8b4137c76b7..26987ba021ab 100644 --- a/daxctl/lib/libdaxctl.sym +++ b/daxctl/lib/libdaxctl.sym @@ -79,4 +79,5 @@ global: LIBDAXCTL_8 { global: daxctl_dev_set_size; + daxctl_region_create_dev; } LIBDAXCTL_7; diff --git a/daxctl/libdaxctl.h b/daxctl/libdaxctl.h index e3d482743cc6..a579ddd1d43c 100644 --- a/daxctl/libdaxctl.h +++ b/daxctl/libdaxctl.h @@ -56,6 +56,7 @@ unsigned long daxctl_region_get_align(struct daxctl_region *region); const char *daxctl_region_get_devname(struct daxctl_region *region); const char *daxctl_region_get_path(struct daxctl_region *region); +int daxctl_region_create_dev(struct daxctl_region *region); struct daxctl_dev *daxctl_region_get_dev_seed(struct daxctl_region *region); struct daxctl_dev; -- 2.17.1

Joao Martins

1:58 p.m.

New subject: [PATCH ndctl v1 07/10] daxctl: add command to create device

Add a 'create-device' command which creates a devdax region. So far the only new option is size, which when omitted uses the entire available space. Example usage: $ daxctl reconfigure-device -s 0 dax0.0 reconfigured 1 device $ daxctl create-device -s 4G -r 0 [ { "chardev":"dax0.1", "size":4294967296, "target_node":0, "mode":"devdax" } ] created 1 device Or using the whole available space like default behaviour or ndctl-create-namespace: $ daxctl create-device [ { "chardev":"dax0.1", "size":120259084288, "target_node":0, "mode":"devdax" } ] created 1 device Signed-off-by: Joao Martins <joao.m.martins(a)oracle.com> Signed-off-by: Dan Williams <dan.j.williams(a)intel.com> --- daxctl/builtin.h | 1 + daxctl/daxctl.c | 1 + daxctl/device.c | 117 ++++++++++++++++++++++++++++++++++++++++++++++- 3 files changed, 118 insertions(+), 1 deletion(-) diff --git a/daxctl/builtin.h b/daxctl/builtin.h index 8f344f86ad20..19b33933b91b 100644 --- a/daxctl/builtin.h +++ b/daxctl/builtin.h @@ -6,6 +6,7 @@ struct daxctl_ctx; int cmd_list(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_migrate(int argc, const char **argv, struct daxctl_ctx *ctx); +int cmd_create_device(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_reconfig_device(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_disable_device(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_enable_device(int argc, const char **argv, struct daxctl_ctx *ctx); diff --git a/daxctl/daxctl.c b/daxctl/daxctl.c index a4699b3780bd..1f315168c513 100644 --- a/daxctl/daxctl.c +++ b/daxctl/daxctl.c @@ -71,6 +71,7 @@ static struct cmd_struct commands[] = { { "list", .d_fn = cmd_list }, { "help", .d_fn = cmd_help }, { "migrate-device-model", .d_fn = cmd_migrate }, + { "create-device", .d_fn = cmd_create_device }, { "reconfigure-device", .d_fn = cmd_reconfig_device }, { "online-memory", .d_fn = cmd_online_memory }, { "offline-memory", .d_fn = cmd_offline_memory }, diff --git a/daxctl/device.c b/daxctl/device.c index 596b6efca34d..c038abba8063 100644 --- a/daxctl/device.c +++ b/daxctl/device.c @@ -9,6 +9,7 @@ #include <sys/stat.h> #include <sys/types.h> #include <sys/sysmacros.h> +#include <util/size.h> #include <util/json.h> #include <util/filter.h> #include <json-c/json.h> @@ -20,6 +21,7 @@ static struct { const char *dev; const char *mode; const char *region; + const char *size; bool no_online; bool no_movable; bool force; @@ -47,6 +49,7 @@ enum device_action { ACTION_RECONFIG, ACTION_ONLINE, ACTION_OFFLINE, + ACTION_CREATE, ACTION_DISABLE, ACTION_ENABLE, }; @@ -63,10 +66,21 @@ OPT_BOOLEAN('N', "no-online", &param.no_online, \ OPT_BOOLEAN('f', "force", &param.force, \ "attempt to offline memory sections before reconfiguration") +#define CREATE_OPTIONS() \ +OPT_STRING('s', "size", &param.size, "size", "size to switch the device to") + #define ZONE_OPTIONS() \ OPT_BOOLEAN('\0', "no-movable", &param.no_movable, \ "online memory in ZONE_NORMAL") +static const struct option create_options[] = { + BASE_OPTIONS(), + CREATE_OPTIONS(), + RECONFIG_OPTIONS(), + ZONE_OPTIONS(), + OPT_END(), +}; + static const struct option reconfig_options[] = { BASE_OPTIONS(), CREATE_OPTIONS(), @@ -104,12 +118,14 @@ static const char *parse_device_options(int argc, const char **argv, usage, NULL }; + unsigned long long units = 1; int i, rc = 0; argc = parse_options(argc, argv, options, u, 0); /* Handle action-agnostic non-option arguments */ - if (argc == 0) { + if (argc == 0 && + action != ACTION_CREATE) { char *action_string; switch (action) { @@ -175,6 +191,10 @@ static const char *parse_device_options(int argc, const char **argv, } } break; + case ACTION_CREATE: + if (param.size) + size = __parse_size64(param.size, &units); + /* fall through */ case ACTION_ONLINE: if (param.no_movable) mem_zone = MEM_ZONE_NORMAL; @@ -446,6 +466,47 @@ static int reconfig_mode_devdax(struct daxctl_dev *dev) return 0; } +static int do_create(struct daxctl_region *region, long long val, + struct json_object **jdevs) +{ + struct json_object *jdev; + struct daxctl_dev *dev; + int rc = 0; + + if (daxctl_region_create_dev(region)) + return -ENOSPC; + + dev = daxctl_region_get_dev_seed(region); + if (!dev) + return -ENOSPC; + + if (val == -1) + val = daxctl_region_get_available_size(region); + + if (val <= 0) + return -ENOSPC; + + rc = daxctl_dev_set_size(dev, val); + if (rc < 0) + return rc; + + rc = daxctl_dev_enable_devdax(dev); + if (rc) { + fprintf(stderr, "%s: enable failed: %s\n", + daxctl_dev_get_devname(dev), strerror(-rc)); + return rc; + } + + *jdevs = json_object_new_array(); + if (*jdevs) { + jdev = util_daxctl_dev_to_json(dev, flags); + if (jdev) + json_object_array_add(*jdevs, jdev); + } + + return 0; +} + static int do_reconfig(struct daxctl_dev *dev, enum dev_mode mode, struct json_object **jdevs) { @@ -548,6 +609,42 @@ static int do_xble(struct daxctl_dev *dev, enum device_action action) return rc; } +static int do_xaction_region(enum device_action action, + struct daxctl_ctx *ctx, int *processed) +{ + struct json_object *jdevs = NULL; + struct daxctl_region *region; + int rc = -ENXIO; + + *processed = 0; + + daxctl_region_foreach(ctx, region) { + if (!util_daxctl_region_filter(region, param.region)) + continue; + + switch (action) { + case ACTION_CREATE: + rc = do_create(region, size, &jdevs); + if (rc == 0) + (*processed)++; + break; + default: + rc = -EINVAL; + break; + } + } + + /* + * jdevs is the containing json array for all devices we are reporting + * on. It therefore needs to be outside the region/device iterators, + * and passed in to the do_<action> functions to add their objects to + */ + if (jdevs) + util_display_json_array(stdout, jdevs, flags); + + return rc; +} + static int do_xaction_device(const char *device, enum device_action action, struct daxctl_ctx *ctx, int *processed) { @@ -610,6 +707,24 @@ static int do_xaction_device(const char *device, enum device_action action, return rc; } +int cmd_create_device(int argc, const char **argv, struct daxctl_ctx *ctx) +{ + char *usage = "daxctl create-device [<options>]"; + int processed, rc; + + parse_device_options(argc, argv, ACTION_CREATE, + create_options, usage, ctx); + + rc = do_xaction_region(ACTION_CREATE, ctx, &processed); + if (rc < 0) + fprintf(stderr, "error creating devices: %s\n", + strerror(-rc)); + + fprintf(stderr, "created %d device%s\n", processed, + processed == 1 ? "" : "s"); + return rc; +} + int cmd_reconfig_device(int argc, const char **argv, struct daxctl_ctx *ctx) { char *usage = "daxctl reconfigure-device <device> [<options>]"; -- 2.17.1

Joao Martins

1:58 p.m.

New subject: [PATCH ndctl v1 08/10] libdaxctl: add daxctl_region_destroy_dev()

Add an API to destroy an empty device. This 'delete' sysfs attribute is only writeable for a dynamic dax device such as the one exported by hmem. Signed-off-by: Joao Martins <joao.m.martins(a)oracle.com> Signed-off-by: Dan Williams <dan.j.williams(a)intel.com> --- daxctl/lib/libdaxctl.c | 17 +++++++++++++++++ daxctl/lib/libdaxctl.sym | 1 + daxctl/libdaxctl.h | 1 + 3 files changed, 19 insertions(+) diff --git a/daxctl/lib/libdaxctl.c b/daxctl/lib/libdaxctl.c index d17ff7a02bad..9b43b68facfe 100644 --- a/daxctl/lib/libdaxctl.c +++ b/daxctl/lib/libdaxctl.c @@ -609,6 +609,23 @@ DAXCTL_EXPORT int daxctl_region_create_dev(struct daxctl_region *region) return 0; } +DAXCTL_EXPORT int daxctl_region_destroy_dev(struct daxctl_region *region, + struct daxctl_dev *dev) +{ + struct daxctl_ctx *ctx = daxctl_region_get_ctx(region); + char *path = region->region_buf; + int rc, len = region->buf_len; + + if (snprintf(path, len, "%s/%s/delete", region->region_path, attrs) >= len) { + err(ctx, "%s: buffer too small!\n", + daxctl_region_get_devname(region)); + return -EFAULT; + } + + rc = sysfs_write_attr(ctx, path, daxctl_dev_get_devname(dev)); + return rc; +} + DAXCTL_EXPORT struct daxctl_dev *daxctl_region_get_dev_seed( struct daxctl_region *region) { diff --git a/daxctl/lib/libdaxctl.sym b/daxctl/lib/libdaxctl.sym index 26987ba021ab..33c926411037 100644 --- a/daxctl/lib/libdaxctl.sym +++ b/daxctl/lib/libdaxctl.sym @@ -80,4 +80,5 @@ LIBDAXCTL_8 { global: daxctl_dev_set_size; daxctl_region_create_dev; + daxctl_region_destroy_dev; } LIBDAXCTL_7; diff --git a/daxctl/libdaxctl.h b/daxctl/libdaxctl.h index a579ddd1d43c..2b14faad1895 100644 --- a/daxctl/libdaxctl.h +++ b/daxctl/libdaxctl.h @@ -61,6 +61,7 @@ struct daxctl_dev *daxctl_region_get_dev_seed(struct daxctl_region *region); struct daxctl_dev; struct daxctl_dev *daxctl_dev_get_first(struct daxctl_region *region); +int daxctl_region_destroy_dev(struct daxctl_region *region, struct daxctl_dev *dev); struct daxctl_dev *daxctl_dev_get_next(struct daxctl_dev *dev); struct daxctl_region *daxctl_dev_get_region(struct daxctl_dev *dev); int daxctl_dev_get_id(struct daxctl_dev *dev); -- 2.17.1

Joao Martins

1:58 p.m.

New subject: [PATCH ndctl v1 09/10] daxctl: add command to destroy device

Add a 'destroy-device' command which destroys a dax device. Like namespaces, the device needs to be disabled in order to be destroyed. Example usage: $ daxctl disable-device dax0.1 disabled 1 device $ daxctl destroy-device dax0.1 destroyed 1 device Signed-off-by: Joao Martins <joao.m.martins(a)oracle.com> Signed-off-by: Dan Williams <dan.j.williams(a)intel.com> --- daxctl/builtin.h | 1 + daxctl/daxctl.c | 1 + daxctl/device.c | 66 ++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 68 insertions(+) diff --git a/daxctl/builtin.h b/daxctl/builtin.h index 19b33933b91b..29ba63ca17aa 100644 --- a/daxctl/builtin.h +++ b/daxctl/builtin.h @@ -7,6 +7,7 @@ struct daxctl_ctx; int cmd_list(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_migrate(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_create_device(int argc, const char **argv, struct daxctl_ctx *ctx); +int cmd_destroy_device(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_reconfig_device(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_disable_device(int argc, const char **argv, struct daxctl_ctx *ctx); int cmd_enable_device(int argc, const char **argv, struct daxctl_ctx *ctx); diff --git a/daxctl/daxctl.c b/daxctl/daxctl.c index 1f315168c513..bd5539900391 100644 --- a/daxctl/daxctl.c +++ b/daxctl/daxctl.c @@ -72,6 +72,7 @@ static struct cmd_struct commands[] = { { "help", .d_fn = cmd_help }, { "migrate-device-model", .d_fn = cmd_migrate }, { "create-device", .d_fn = cmd_create_device }, + { "destroy-device", .d_fn = cmd_destroy_device }, { "reconfigure-device", .d_fn = cmd_reconfig_device }, { "online-memory", .d_fn = cmd_online_memory }, { "offline-memory", .d_fn = cmd_offline_memory }, diff --git a/daxctl/device.c b/daxctl/device.c index c038abba8063..05293d6c38ee 100644 --- a/daxctl/device.c +++ b/daxctl/device.c @@ -52,6 +52,7 @@ enum device_action { ACTION_CREATE, ACTION_DISABLE, ACTION_ENABLE, + ACTION_DESTROY, }; #define BASE_OPTIONS() \ @@ -69,6 +70,10 @@ OPT_BOOLEAN('f', "force", &param.force, \ #define CREATE_OPTIONS() \ OPT_STRING('s', "size", &param.size, "size", "size to switch the device to") +#define DESTROY_OPTIONS() \ +OPT_BOOLEAN('f', "force", &param.force, \ + "attempt to disable before destroying device") + #define ZONE_OPTIONS() \ OPT_BOOLEAN('\0', "no-movable", &param.no_movable, \ "online memory in ZONE_NORMAL") @@ -110,6 +115,12 @@ static const struct option enable_options[] = { OPT_END(), }; +static const struct option destroy_options[] = { + BASE_OPTIONS(), + DESTROY_OPTIONS(), + OPT_END(), +}; + static const char *parse_device_options(int argc, const char **argv, enum device_action action, const struct option *options, const char *usage, struct daxctl_ctx *ctx) @@ -144,6 +155,9 @@ static const char *parse_device_options(int argc, const char **argv, case ACTION_ENABLE: action_string = "enable"; break; + case ACTION_DESTROY: + action_string = "destroy"; + break; default: action_string = "<>"; break; @@ -199,6 +213,7 @@ static const char *parse_device_options(int argc, const char **argv, if (param.no_movable) mem_zone = MEM_ZONE_NORMAL; /* fall through */ + case ACTION_DESTROY: case ACTION_OFFLINE: case ACTION_DISABLE: case ACTION_ENABLE: @@ -364,6 +379,35 @@ static int dev_resize(struct daxctl_dev *dev, unsigned long long val) return 0; } +static int dev_destroy(struct daxctl_dev *dev) +{ + const char *devname = daxctl_dev_get_devname(dev); + int rc; + + if (daxctl_dev_is_enabled(dev) && !param.force) { + fprintf(stderr, "%s is active, specify --force for deletion\n", + devname); + return -ENXIO; + } else { + rc = daxctl_dev_disable(dev); + if (rc) { + fprintf(stderr, "%s: disable failed: %s\n", + daxctl_dev_get_devname(dev), strerror(-rc)); + return rc; + } + } + + rc = daxctl_dev_set_size(dev, 0); + if (rc < 0) + return rc; + + rc = daxctl_region_destroy_dev(daxctl_dev_get_region(dev), dev); + if (rc < 0) + return rc; + + return 0; +} + static int disable_devdax_device(struct daxctl_dev *dev) { struct daxctl_memory *mem = daxctl_dev_get_memory(dev); @@ -689,6 +733,11 @@ static int do_xaction_device(const char *device, enum device_action action, if (rc == 0) (*processed)++; break; + case ACTION_DESTROY: + rc = dev_destroy(dev); + if (rc == 0) + (*processed)++; + break; default: rc = -EINVAL; break; @@ -725,6 +774,23 @@ int cmd_create_device(int argc, const char **argv, struct daxctl_ctx *ctx) return rc; } +int cmd_destroy_device(int argc, const char **argv, struct daxctl_ctx *ctx) +{ + char *usage = "daxctl destroy-device <device> [<options>]"; + const char *device = parse_device_options(argc, argv, ACTION_DESTROY, + destroy_options, usage, ctx); + int processed, rc; + + rc = do_xaction_device(device, ACTION_DESTROY, ctx, &processed); + if (rc < 0) + fprintf(stderr, "error destroying devices: %s\n", + strerror(-rc)); + + fprintf(stderr, "destroyed %d device%s\n", processed, + processed == 1 ? "" : "s"); + return rc; +} + int cmd_reconfig_device(int argc, const char **argv, struct daxctl_ctx *ctx) { char *usage = "daxctl reconfigure-device <device> [<options>]"; -- 2.17.1

Joao Martins

1:59 p.m.

New subject: [PATCH ndctl v1 10/10] daxctl/test: Add tests for dynamic dax regions

Add a couple tests which exercise the new sysfs based interface for Soft-Reserved regions (by EFI/HMAT, or efi_fake_mem). The tests exercise the daxctl orchestration surrounding for creating/disabling/destroying/reconfiguring devices. Furthermore it exercises dax region space allocation code paths particularly: 1) zeroing out and reconfiguring a dax device from its current size to be max available and back to initial size 2) creates devices from holes in the beginning, middle of the region. 3) reconfigures devices in a interleaving fashion 4) test adjust of the region towards beginning and end The tests assume you pass a valid efi_fake_mem parameter marked as EFI_MEMORY_SP e.g. efi_fake_mem=112G@16G:0x40000 Naturally it bails out from the test if hmem device driver isn't loaded/builtin or no region is found. Signed-off-by: Joao Martins <joao.m.martins(a)oracle.com> --- test/Makefile.am | 1 + test/daxctl-create.sh | 293 ++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 294 insertions(+) create mode 100755 test/daxctl-create.sh diff --git a/test/Makefile.am b/test/Makefile.am index 1d24a65ded8b..6b7c82f9a4e2 100644 --- a/test/Makefile.am +++ b/test/Makefile.am @@ -58,6 +58,7 @@ TESTS +=\ device-dax \ device-dax-fio.sh \ daxctl-devices.sh \ + daxctl-create.sh \ dm.sh \ mmap.sh diff --git a/test/daxctl-create.sh b/test/daxctl-create.sh new file mode 100755 index 000000000000..f46b9273ef20 --- /dev/null +++ b/test/daxctl-create.sh @@ -0,0 +1,293 @@ +#!/bin/bash -Ex +# SPDX-License-Identifier: GPL-2.0 +# Copyright(c) 2020, Oracle and/or its affiliates. + +rc=77 +. $(dirname $0)/common + +trap 'cleanup $LINENO' ERR + +cleanup() +{ + printf "Error at line %d\n" "$1" + [[ $testdev ]] && reset + exit $rc +} + +find_testdev() +{ + local rc=77 + + # The hmem driver is needed to change the device mode, only + # kernels >= v5.6 might have it available. Skip if not. + if ! modinfo hmem; then + # check if hmem is builtin + if [ ! -d "/sys/module/device_hmem" ]; then + printf "Unable to find hmem module\n" + exit $rc + fi + fi + + # find a victim region provided by dax_hmem + testpath=$("$DAXCTL" list -r 0 | jq -er '.[0].path | .//""') + if [[ ! "$testpath" == *"hmem"* ]]; then + printf "Unable to find a victim region\n" + exit "$rc" + fi + + # find a victim device + testdev=$("$DAXCTL" list -D -r 0 | jq -er '.[0].chardev | .//""') + if [[ ! $testdev ]]; then + printf "Unable to find a victim device\n" + exit "$rc" + fi + printf "Found victim dev: %s on region id 0\n" "$testdev" +} + +setup_dev() +{ + test -n "$testdev" + + "$DAXCTL" disable-device "$testdev" + "$DAXCTL" reconfigure-device -s 0 "$testdev" + available=$("$DAXCTL" list -r 0 | jq -er '.[0].available_size | .//""') +} + +reset_dev() +{ + test -n "$testdev" + + "$DAXCTL" disable-device "$testdev" + "$DAXCTL" reconfigure-device -s $available "$testdev" +} + +reset() +{ + test -n "$testdev" + + "$DAXCTL" disable-device all + "$DAXCTL" destroy-device all + "$DAXCTL" reconfigure-device -s $available "$testdev" +} + +clear_dev() +{ + "$DAXCTL" disable-device "$testdev" + "$DAXCTL" reconfigure-device -s 0 "$testdev" +} + +test_pass() +{ + local rc=1 + + # Available size + _available_size=$("$DAXCTL" list -r 0 | jq -er '.[0].available_size | .//""') + if [[ ! $_available_size == $available ]]; then + printf "Unexpected available size $_available_size != $available\n" + exit "$rc" + fi +} + +fail_if_available() +{ + local rc=1 + + _size=$("$DAXCTL" list -r 0 | jq -er '.[0].available_size | .//""') + if [[ $_size ]]; then + printf "Unexpected available size $_size\n" + exit "$rc" + fi +} + +daxctl_get_dev() +{ + "$DAXCTL" list -d "$1" | jq -er '.[].chardev' +} + +daxctl_get_mode() +{ + "$DAXCTL" list -d "$1" | jq -er '.[].mode' +} + +daxctl_test_multi() +{ + local daxdev + + size=$(expr $available / 4) + + if [[ $2 ]]; then + "$DAXCTL" disable-device "$testdev" + "$DAXCTL" reconfigure-device -s $size "$testdev" + fi + + daxdev_1=$("$DAXCTL" create-device -s $size | jq -er '.[].chardev') + test -n $daxdev_1 + + daxdev_2=$("$DAXCTL" create-device -s $size | jq -er '.[].chardev') + test -n $daxdev_2 + + if [[ ! $2 ]]; then + daxdev_3=$("$DAXCTL" create-device -s $size | jq -er '.[].chardev') + test -n $daxdev_3 + fi + + # Hole + "$DAXCTL" disable-device $1 && "$DAXCTL" destroy-device $1 + + # Pick space in the created hole and at the end + new_size=$(expr $size \* 2) + daxdev_4=$("$DAXCTL" create-device -s $new_size | jq -er '.[].chardev') + test -n $daxdev_4 + + fail_if_available + + "$DAXCTL" disable-device all + "$DAXCTL" destroy-device all +} + +daxctl_test_multi_reconfig() +{ + local ncfgs=$1 + local daxdev + + size=$(expr $available / $ncfgs) + + test -n "$testdev" + + "$DAXCTL" disable-device "$testdev" + "$DAXCTL" reconfigure-device -s $size "$testdev" + "$DAXCTL" disable-device "$testdev" + + daxdev_1=$("$DAXCTL" create-device -s $size | jq -er '.[].chardev') + "$DAXCTL" disable-device "$daxdev_1" + + start=$(expr $size + $size) + max=$(expr $ncfgs / 2 \* $size) + for i in $(seq $start $size $max) + do + "$DAXCTL" disable-device "$testdev" + "$DAXCTL" reconfigure-device -s $i "$testdev" + + "$DAXCTL" disable-device "$daxdev_1" + "$DAXCTL" reconfigure-device -s $i "$daxdev_1" + done + + fail_if_available + + "$DAXCTL" disable-device "$daxdev_1" && "$DAXCTL" destroy-device "$daxdev_1" +} + +daxctl_test_adjust() +{ + local rc=1 + local ncfgs=4 + local daxdev + + size=$(expr $available / $ncfgs) + + test -n "$testdev" + + start=$(expr $size + $size) + for i in $(seq 1 1 $ncfgs) + do + daxdev=$("$DAXCTL" create-device -s $size) + done + + daxdev=$(daxctl_get_dev "dax0.1") + "$DAXCTL" disable-device "$daxdev" && "$DAXCTL" destroy-device "$daxdev" + daxdev=$(daxctl_get_dev "dax0.4") + "$DAXCTL" disable-device "$daxdev" && "$DAXCTL" destroy-device "$daxdev" + + daxdev=$(daxctl_get_dev "dax0.2") + "$DAXCTL" disable-device "$daxdev" + "$DAXCTL" reconfigure-device -s $(expr $size \* 2) "$daxdev" + + daxdev=$(daxctl_get_dev "dax0.3") + "$DAXCTL" disable-device "$daxdev" + "$DAXCTL" reconfigure-device -s $(expr $size \* 2) "$daxdev" + + fail_if_available + + daxdev=$(daxctl_get_dev "dax0.3") + "$DAXCTL" disable-device "$daxdev" && "$DAXCTL" destroy-device "$daxdev" + daxdev=$(daxctl_get_dev "dax0.2") + "$DAXCTL" disable-device "$daxdev" && "$DAXCTL" destroy-device "$daxdev" +} + +# Test 0: +# Sucessfully zero out the region device and allocate the whole space again. +daxctl_test0() +{ + clear_dev + test_pass +} + +# Test 1: +# Sucessfully creates and destroys a device with the whole available space +daxctl_test1() +{ + local daxdev + + daxdev=$("$DAXCTL" create-device | jq -er '.[].chardev') + + test -n "$daxdev" + fail_if_available + + "$DAXCTL" disable-device "$daxdev" && "$DAXCTL" destroy-device "$daxdev" + + clear_dev + test_pass +} + +# Test 2: space at the middle and at the end +# Successfully pick space in the middle and space at the end, by +# having the region device reconfigured with some of the memory. +daxctl_test2() +{ + daxctl_test_multi "dax0.1" 1 + clear_dev + test_pass +} + +# Test 3: space at the beginning and at the end +# Successfully pick space in the beginning and space at the end, by +# having the region device emptied (so region beginning starts with dax0.1). +daxctl_test3() +{ + daxctl_test_multi "dax0.1" + clear_dev + test_pass +} + +# Test 4: space at the end +# Successfully reconfigure two devices in increasingly bigger allocations. +# The difference is that it reuses an existing resource, and only needs to +# pick at the end of the region +daxctl_test4() +{ + daxctl_test_multi_reconfig 8 + clear_dev + test_pass +} + +# Test 5: space adjust +# Successfully adjusts two resources to fill the whole region +# First adjusts towards the beginning of region, the second towards the end. +daxctl_test5() +{ + daxctl_test_adjust + clear_dev + test_pass +} + +find_testdev +rc=1 +setup_dev +daxctl_test0 +daxctl_test1 +daxctl_test2 +daxctl_test3 +daxctl_test4 +daxctl_test5 +reset_dev +exit 0 -- 2.17.1

Dan Williams

Sunday, 12 July Sun, 12 Jul

8:04 a.m.

New subject: [PATCH ndctl v1 10/10] daxctl/test: Add tests for dynamic dax regions

On Fri, Apr 3, 2020 at 1:59 PM Joao Martins <joao.m.martins(a)oracle.com> wrote:

...

So, I finally have the kernel passing this test. Thank you! I did notice that I need to make sure that there is only one hmem range defined otherwise the test gets confused and fails. I expect it spills over to other regions when the test expects that the region should be full. Could you take a look at adjusting the test to constrain its allocations to a single region?

Joao Martins

Monday, 13 July Mon, 13 Jul

2:46 a.m.

New subject: [PATCH ndctl v1 10/10] daxctl/test: Add tests for dynamic dax regions

On 7/12/20 4:04 PM, Dan Williams wrote:

...

On Fri, Apr 3, 2020 at 1:59 PM Joao Martins <joao.m.martins(a)oracle.com> wrote: > > Add a couple tests which exercise the new sysfs based > interface for Soft-Reserved regions (by EFI/HMAT, or > efi_fake_mem). > > The tests exercise the daxctl orchestration surrounding > for creating/disabling/destroying/reconfiguring devices. > Furthermore it exercises dax region space allocation > code paths particularly: > > 1) zeroing out and reconfiguring a dax device from > its current size to be max available and back to initial > size > > 2) creates devices from holes in the beginning, > middle of the region. > > 3) reconfigures devices in a interleaving fashion > > 4) test adjust of the region towards beginning and end > > The tests assume you pass a valid efi_fake_mem parameter > marked as EFI_MEMORY_SP e.g. > > efi_fake_mem=112G@16G:0x40000 > > Naturally it bails out from the test if hmem device driver > isn't loaded/builtin or no region is found. So, I finally have the kernel passing this test. Thank you! I did notice that I need to make sure that there is only one hmem range defined otherwise the test gets confused and fails. I expect it spills over to other regions when the test expects that the region should be full. Could you take a look at adjusting the test to constrain its allocations to a single region?

Yeap, will fix and send v2.

430

days inactive

531

days old

linux-nvdimm@lists.01.org

Manage subscription

12 comments

2 participants

tags (0)

participants (2)

Dan Williams
Joao Martins

2021

2020

2019

2018

2017

2016

2015

2014

[PATCH ndctl v1 00/10] daxctl: Support for sub-dividing soft-reserved regions