From: Dan Williams <dan.j.williams@intel.com>
To: Vishal Verma <vishal.l.verma@intel.com>
Cc: Linux NVDIMM <nvdimm@lists.linux.dev>,
linux-cxl@vger.kernel.org, Ben Widawsky <ben.widawsky@intel.com>,
Alison Schofield <alison.schofield@intel.com>,
Ira Weiny <ira.weiny@intel.com>
Subject: Re: [ndctl PATCH v3 06/21] test: rename 'ndctl_test' to 'test_ctx'
Date: Thu, 15 Jul 2021 16:51:04 -0700 [thread overview]
Message-ID: <CAPcyv4it2Ubzf62x4ZrFL=ivJJgjJko4JgA=QbigSrD9wfsSnQ@mail.gmail.com> (raw)
In-Reply-To: <20210701201005.3065299-7-vishal.l.verma@intel.com>
On Thu, Jul 1, 2021 at 1:10 PM Vishal Verma <vishal.l.verma@intel.com> wrote:
>
> In preparation for using the common test core for libcxl tests, rename
> the 'ndctl_test' structure to 'test_ctx'
>
Looks good,
Reviewed-by: Dan Williams <dan.j.williams@intel.com>
> Cc: Dan Williams <dan.j.williams@intel.com>
> Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
> ---
> test.h | 36 +++++++++++++++++------------------
> ndctl/bat.c | 2 +-
> ndctl/test.c | 2 +-
> test/ack-shutdown-count-set.c | 8 ++++----
> test/blk_namespaces.c | 6 +++---
> test/core.c | 20 +++++++++----------
> test/dax-dev.c | 4 ++--
> test/dax-pmd.c | 9 +++++----
> test/dax-poison.c | 4 ++--
> test/daxdev-errors.c | 2 +-
> test/device-dax.c | 10 +++++-----
> test/dpa-alloc.c | 6 +++---
> test/dsm-fail.c | 6 +++---
> test/libndctl.c | 34 ++++++++++++++++-----------------
> test/multi-pmem.c | 9 +++++----
> test/parent-uuid.c | 5 +++--
> test/pmem_namespaces.c | 6 +++---
> test/revoke-devmem.c | 6 +++---
> 18 files changed, 89 insertions(+), 86 deletions(-)
>
> diff --git a/test.h b/test.h
> index 7de13fe..c5b9d4a 100644
> --- a/test.h
> +++ b/test.h
> @@ -4,16 +4,16 @@
> #define __TEST_H__
> #include <stdbool.h>
>
> -struct ndctl_test;
> +struct test_ctx;
> struct ndctl_ctx;
> -struct ndctl_test *ndctl_test_new(unsigned int kver);
> -int ndctl_test_result(struct ndctl_test *test, int rc);
> -int ndctl_test_get_skipped(struct ndctl_test *test);
> -int ndctl_test_get_attempted(struct ndctl_test *test);
> -int __ndctl_test_attempt(struct ndctl_test *test, unsigned int kver,
> +struct test_ctx *ndctl_test_new(unsigned int kver);
> +int ndctl_test_result(struct test_ctx *test, int rc);
> +int ndctl_test_get_skipped(struct test_ctx *test);
> +int ndctl_test_get_attempted(struct test_ctx *test);
> +int __ndctl_test_attempt(struct test_ctx *test, unsigned int kver,
> const char *caller, int line);
> #define ndctl_test_attempt(t, v) __ndctl_test_attempt(t, v, __func__, __LINE__)
> -void __ndctl_test_skip(struct ndctl_test *test, const char *caller, int line);
> +void __ndctl_test_skip(struct test_ctx *test, const char *caller, int line);
> #define ndctl_test_skip(t) __ndctl_test_skip(t, __func__, __LINE__)
> struct ndctl_namespace *ndctl_get_test_dev(struct ndctl_ctx *ctx);
> void builtin_xaction_namespace_reset(void);
> @@ -22,27 +22,27 @@ struct kmod_ctx;
> struct kmod_module;
> int ndctl_test_init(struct kmod_ctx **ctx, struct kmod_module **mod,
> struct ndctl_ctx *nd_ctx, int log_level,
> - struct ndctl_test *test);
> + struct test_ctx *test);
>
> struct ndctl_ctx;
> -int test_parent_uuid(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx);
> -int test_multi_pmem(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx);
> +int test_parent_uuid(int loglevel, struct test_ctx *test, struct ndctl_ctx *ctx);
> +int test_multi_pmem(int loglevel, struct test_ctx *test, struct ndctl_ctx *ctx);
> int test_dax_directio(int dax_fd, unsigned long align, void *dax_addr, off_t offset);
> -int test_dax_remap(struct ndctl_test *test, int dax_fd, unsigned long align, void *dax_addr,
> +int test_dax_remap(struct test_ctx *test, int dax_fd, unsigned long align, void *dax_addr,
> off_t offset, bool fsdax);
> #ifdef ENABLE_POISON
> -int test_dax_poison(struct ndctl_test *test, int dax_fd, unsigned long align,
> +int test_dax_poison(struct test_ctx *test, int dax_fd, unsigned long align,
> void *dax_addr, off_t offset, bool fsdax);
> #else
> -static inline int test_dax_poison(struct ndctl_test *test, int dax_fd,
> +static inline int test_dax_poison(struct test_ctx *test, int dax_fd,
> unsigned long align, void *dax_addr, off_t offset, bool fsdax)
> {
> return 0;
> }
> #endif
> -int test_dpa_alloc(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx);
> -int test_dsm_fail(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx);
> -int test_libndctl(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx);
> -int test_blk_namespaces(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx);
> -int test_pmem_namespaces(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx);
> +int test_dpa_alloc(int loglevel, struct test_ctx *test, struct ndctl_ctx *ctx);
> +int test_dsm_fail(int loglevel, struct test_ctx *test, struct ndctl_ctx *ctx);
> +int test_libndctl(int loglevel, struct test_ctx *test, struct ndctl_ctx *ctx);
> +int test_blk_namespaces(int loglevel, struct test_ctx *test, struct ndctl_ctx *ctx);
> +int test_pmem_namespaces(int loglevel, struct test_ctx *test, struct ndctl_ctx *ctx);
> #endif /* __TEST_H__ */
> diff --git a/ndctl/bat.c b/ndctl/bat.c
> index ef00a3b..18773fd 100644
> --- a/ndctl/bat.c
> +++ b/ndctl/bat.c
> @@ -9,7 +9,7 @@
> int cmd_bat(int argc, const char **argv, struct ndctl_ctx *ctx)
> {
> int loglevel = LOG_DEBUG, i, rc;
> - struct ndctl_test *test;
> + struct test_ctx *test;
> bool force = false;
> const char * const u[] = {
> "ndctl bat [<options>]",
> diff --git a/ndctl/test.c b/ndctl/test.c
> index 6a05d8d..7af3681 100644
> --- a/ndctl/test.c
> +++ b/ndctl/test.c
> @@ -18,7 +18,7 @@ static char *result(int rc)
>
> int cmd_test(int argc, const char **argv, struct ndctl_ctx *ctx)
> {
> - struct ndctl_test *test;
> + struct test_ctx *test;
> int loglevel = LOG_DEBUG, i, rc;
> const char * const u[] = {
> "ndctl test [<options>]",
> diff --git a/test/ack-shutdown-count-set.c b/test/ack-shutdown-count-set.c
> index c561ff3..d5c7bcb 100644
> --- a/test/ack-shutdown-count-set.c
> +++ b/test/ack-shutdown-count-set.c
> @@ -54,7 +54,7 @@ static void reset_bus(struct ndctl_bus *bus)
> ndctl_dimm_zero_labels(dimm);
> }
>
> -static int do_test(struct ndctl_ctx *ctx, struct ndctl_test *test)
> +static int do_test(struct ndctl_ctx *ctx, struct test_ctx *test)
> {
> struct ndctl_bus *bus = ndctl_bus_get_by_provider(ctx, "nfit_test.0");
> struct ndctl_dimm *dimm;
> @@ -91,8 +91,8 @@ out:
> return rc;
> }
>
> -static int test_ack_shutdown_count_set(int loglevel, struct ndctl_test *test,
> - struct ndctl_ctx *ctx)
> +static int test_ack_shutdown_count_set(int loglevel, struct test_ctx *test,
> + struct ndctl_ctx *ctx)
> {
> struct kmod_module *mod;
> struct kmod_ctx *kmod_ctx;
> @@ -117,7 +117,7 @@ static int test_ack_shutdown_count_set(int loglevel, struct ndctl_test *test,
>
> int main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> struct ndctl_ctx *ctx;
> int rc;
>
> diff --git a/test/blk_namespaces.c b/test/blk_namespaces.c
> index f076e85..4919890 100644
> --- a/test/blk_namespaces.c
> +++ b/test/blk_namespaces.c
> @@ -198,8 +198,8 @@ static int ns_do_io(const char *bdev)
>
> static const char *comm = "test-blk-namespaces";
>
> -int test_blk_namespaces(int log_level, struct ndctl_test *test,
> - struct ndctl_ctx *ctx)
> +int test_blk_namespaces(int log_level, struct test_ctx *test,
> + struct ndctl_ctx *ctx)
> {
> char bdev[50];
> int rc = -ENXIO;
> @@ -337,7 +337,7 @@ int test_blk_namespaces(int log_level, struct ndctl_test *test,
>
> int __attribute__((weak)) main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> struct ndctl_ctx *ctx;
> int rc;
>
> diff --git a/test/core.c b/test/core.c
> index 2b03aa9..1ef0ac4 100644
> --- a/test/core.c
> +++ b/test/core.c
> @@ -16,7 +16,7 @@
>
> #define KVER_STRLEN 20
>
> -struct ndctl_test {
> +struct test_ctx {
> unsigned int kver;
> int attempt;
> int skip;
> @@ -39,9 +39,9 @@ static unsigned int get_system_kver(void)
> return KERNEL_VERSION(a,b,c);
> }
>
> -struct ndctl_test *ndctl_test_new(unsigned int kver)
> +struct test_ctx *ndctl_test_new(unsigned int kver)
> {
> - struct ndctl_test *test = calloc(1, sizeof(*test));
> + struct test_ctx *test = calloc(1, sizeof(*test));
>
> if (!test)
> return NULL;
> @@ -54,7 +54,7 @@ struct ndctl_test *ndctl_test_new(unsigned int kver)
> return test;
> }
>
> -int ndctl_test_result(struct ndctl_test *test, int rc)
> +int ndctl_test_result(struct test_ctx *test, int rc)
> {
> if (ndctl_test_get_skipped(test))
> fprintf(stderr, "attempted: %d skipped: %d\n",
> @@ -75,8 +75,8 @@ static char *kver_str(char *buf, unsigned int kver)
> return buf;
> }
>
> -int __ndctl_test_attempt(struct ndctl_test *test, unsigned int kver,
> - const char *caller, int line)
> +int __ndctl_test_attempt(struct test_ctx *test, unsigned int kver,
> + const char *caller, int line)
> {
> char requires[KVER_STRLEN], current[KVER_STRLEN];
>
> @@ -90,26 +90,26 @@ int __ndctl_test_attempt(struct ndctl_test *test, unsigned int kver,
> return 0;
> }
>
> -void __ndctl_test_skip(struct ndctl_test *test, const char *caller, int line)
> +void __ndctl_test_skip(struct test_ctx *test, const char *caller, int line)
> {
> test->skip++;
> test->attempt = test->skip;
> fprintf(stderr, "%s: explicit skip %s:%d\n", __func__, caller, line);
> }
>
> -int ndctl_test_get_attempted(struct ndctl_test *test)
> +int ndctl_test_get_attempted(struct test_ctx *test)
> {
> return test->attempt;
> }
>
> -int ndctl_test_get_skipped(struct ndctl_test *test)
> +int ndctl_test_get_skipped(struct test_ctx *test)
> {
> return test->skip;
> }
>
> int ndctl_test_init(struct kmod_ctx **ctx, struct kmod_module **mod,
> struct ndctl_ctx *nd_ctx, int log_level,
> - struct ndctl_test *test)
> + struct test_ctx *test)
> {
> int rc, family = -1;
> unsigned int i;
> diff --git a/test/dax-dev.c b/test/dax-dev.c
> index 6a1b76d..99eda26 100644
> --- a/test/dax-dev.c
> +++ b/test/dax-dev.c
> @@ -87,7 +87,7 @@ struct ndctl_namespace *ndctl_get_test_dev(struct ndctl_ctx *ctx)
> return rc ? NULL : ndns;
> }
>
> -static int emit_e820_device(int loglevel, struct ndctl_test *test)
> +static int emit_e820_device(int loglevel, struct test_ctx *test)
> {
> int err;
> struct ndctl_ctx *ctx;
> @@ -118,7 +118,7 @@ static int emit_e820_device(int loglevel, struct ndctl_test *test)
>
> int __attribute__((weak)) main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> int rc;
>
> if (!test) {
> diff --git a/test/dax-pmd.c b/test/dax-pmd.c
> index 7648e34..e9dedb9 100644
> --- a/test/dax-pmd.c
> +++ b/test/dax-pmd.c
> @@ -37,8 +37,9 @@ static void sigbus(int sig, siginfo_t *siginfo, void *d)
> siglongjmp(sj_env, 1);
> }
>
> -int test_dax_remap(struct ndctl_test *test, int dax_fd, unsigned long align, void *dax_addr,
> - off_t offset, bool fsdax)
> +int test_dax_remap(struct test_ctx *test, int dax_fd, unsigned long align,
> + void *dax_addr,
> + off_t offset, bool fsdax)
> {
> void *anon, *remap, *addr;
> struct sigaction act;
> @@ -272,7 +273,7 @@ int test_dax_directio(int dax_fd, unsigned long align, void *dax_addr, off_t off
> }
>
> /* test_pmd assumes that fd references a pre-allocated + dax-capable file */
> -static int test_pmd(struct ndctl_test *test, int fd)
> +static int test_pmd(struct test_ctx *test, int fd)
> {
> unsigned long long m_align, p_align, pmd_off;
> static const bool fsdax = true;
> @@ -351,7 +352,7 @@ err_mmap:
>
> int __attribute__((weak)) main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> int fd, rc;
>
> if (!test) {
> diff --git a/test/dax-poison.c b/test/dax-poison.c
> index 4e09761..dc62742 100644
> --- a/test/dax-poison.c
> +++ b/test/dax-poison.c
> @@ -44,8 +44,8 @@ static void sigbus_hdl(int sig, siginfo_t *si, void *ptr)
> siglongjmp(sj_env, 1);
> }
>
> -int test_dax_poison(struct ndctl_test *test, int dax_fd, unsigned long align,
> - void *dax_addr, off_t offset, bool fsdax)
> +int test_dax_poison(struct test_ctx *test, int dax_fd, unsigned long align,
> + void *dax_addr, off_t offset, bool fsdax)
> {
> unsigned char *addr = MAP_FAILED;
> struct sigaction act;
> diff --git a/test/daxdev-errors.c b/test/daxdev-errors.c
> index fbbea21..4cb6b4d 100644
> --- a/test/daxdev-errors.c
> +++ b/test/daxdev-errors.c
> @@ -29,7 +29,7 @@
>
> struct check_cmd {
> struct ndctl_cmd *cmd;
> - struct ndctl_test *test;
> + struct test_ctx *test;
> };
>
> static sigjmp_buf sj_env;
> diff --git a/test/device-dax.c b/test/device-dax.c
> index aad8fa5..1837b4d 100644
> --- a/test/device-dax.c
> +++ b/test/device-dax.c
> @@ -90,7 +90,7 @@ static void sigbus(int sig, siginfo_t *siginfo, void *d)
> #define VERIFY_TIME(x) (suseconds_t) ((ALIGN(x, SZ_2M) / SZ_4K) * 60)
>
> static int verify_data(struct daxctl_dev *dev, char *dax_buf,
> - unsigned long align, int salt, struct ndctl_test *test)
> + unsigned long align, int salt, struct test_ctx *test)
> {
> struct timeval tv1, tv2, tv_diff;
> unsigned long i;
> @@ -167,7 +167,7 @@ static int test_dax_soft_offline(struct ndctl_test *test, struct ndctl_namespace
> }
>
> static int __test_device_dax(unsigned long align, int loglevel,
> - struct ndctl_test *test, struct ndctl_ctx *ctx)
> + struct test_ctx *test, struct ndctl_ctx *ctx)
> {
> unsigned long i;
> struct sigaction act;
> @@ -406,8 +406,8 @@ static int __test_device_dax(unsigned long align, int loglevel,
> return rc;
> }
>
> -static int test_device_dax(int loglevel, struct ndctl_test *test,
> - struct ndctl_ctx *ctx)
> +static int test_device_dax(int loglevel, struct test_ctx *test,
> + struct ndctl_ctx *ctx)
> {
> unsigned long i, aligns[] = { SZ_4K, SZ_2M, SZ_1G };
> int rc;
> @@ -423,7 +423,7 @@ static int test_device_dax(int loglevel, struct ndctl_test *test,
>
> int __attribute__((weak)) main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> struct ndctl_ctx *ctx;
> int rc;
>
> diff --git a/test/dpa-alloc.c b/test/dpa-alloc.c
> index 59185cf..d0f5271 100644
> --- a/test/dpa-alloc.c
> +++ b/test/dpa-alloc.c
> @@ -32,7 +32,7 @@ struct test_dpa_namespace {
>
> #define MIN_SIZE SZ_4M
>
> -static int do_test(struct ndctl_ctx *ctx, struct ndctl_test *test)
> +static int do_test(struct ndctl_ctx *ctx, struct test_ctx *test)
> {
> unsigned int default_available_slots, available_slots, i;
> struct ndctl_region *region, *blk_region = NULL;
> @@ -280,7 +280,7 @@ static int do_test(struct ndctl_ctx *ctx, struct ndctl_test *test)
> return 0;
> }
>
> -int test_dpa_alloc(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx)
> +int test_dpa_alloc(int loglevel, struct test_ctx *test, struct ndctl_ctx *ctx)
> {
> struct kmod_module *mod;
> struct kmod_ctx *kmod_ctx;
> @@ -307,7 +307,7 @@ int test_dpa_alloc(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx)
>
> int __attribute__((weak)) main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> struct ndctl_ctx *ctx;
> int rc;
>
> diff --git a/test/dsm-fail.c b/test/dsm-fail.c
> index 0a6383d..74c56de 100644
> --- a/test/dsm-fail.c
> +++ b/test/dsm-fail.c
> @@ -174,7 +174,7 @@ static int test_regions_enable(struct ndctl_bus *bus,
> return 0;
> }
>
> -static int do_test(struct ndctl_ctx *ctx, struct ndctl_test *test)
> +static int do_test(struct ndctl_ctx *ctx, struct test_ctx *test)
> {
> struct ndctl_bus *bus = ndctl_bus_get_by_provider(ctx, "nfit_test.0");
> struct ndctl_region *region, *victim_region = NULL;
> @@ -339,7 +339,7 @@ static int do_test(struct ndctl_ctx *ctx, struct ndctl_test *test)
> return err;
> }
>
> -int test_dsm_fail(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx)
> +int test_dsm_fail(int loglevel, struct test_ctx *test, struct ndctl_ctx *ctx)
> {
> struct kmod_module *mod;
> struct kmod_ctx *kmod_ctx;
> @@ -364,7 +364,7 @@ int test_dsm_fail(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx)
>
> int __attribute__((weak)) main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> struct ndctl_ctx *ctx;
> int rc;
>
> diff --git a/test/libndctl.c b/test/libndctl.c
> index d9b50f4..30d19db 100644
> --- a/test/libndctl.c
> +++ b/test/libndctl.c
> @@ -613,7 +613,7 @@ static int validate_dax(struct ndctl_dax *dax)
> const char *devname = ndctl_namespace_get_devname(ndns);
> struct ndctl_region *region = ndctl_dax_get_region(dax);
> struct ndctl_ctx *ctx = ndctl_dax_get_ctx(dax);
> - struct ndctl_test *test = ndctl_get_private_data(ctx);
> + struct test_ctx *test = ndctl_get_private_data(ctx);
> struct daxctl_region *dax_region = NULL, *found;
> int rc = -ENXIO, fd, count, dax_expect;
> struct daxctl_dev *dax_dev, *seed;
> @@ -725,7 +725,7 @@ static int __check_dax_create(struct ndctl_region *region,
> {
> struct ndctl_dax *dax_seed = ndctl_region_get_dax_seed(region);
> struct ndctl_ctx *ctx = ndctl_region_get_ctx(region);
> - struct ndctl_test *test = ndctl_get_private_data(ctx);
> + struct test_ctx *test = ndctl_get_private_data(ctx);
> enum ndctl_namespace_mode mode;
> struct ndctl_dax *dax;
> const char *devname;
> @@ -835,7 +835,7 @@ static int __check_pfn_create(struct ndctl_region *region,
> {
> struct ndctl_pfn *pfn_seed = ndctl_region_get_pfn_seed(region);
> struct ndctl_ctx *ctx = ndctl_region_get_ctx(region);
> - struct ndctl_test *test = ndctl_get_private_data(ctx);
> + struct test_ctx *test = ndctl_get_private_data(ctx);
> enum ndctl_namespace_mode mode;
> struct ndctl_pfn *pfn;
> const char *devname;
> @@ -978,7 +978,7 @@ static int check_btt_size(struct ndctl_btt *btt)
> unsigned long long actual, expect;
> int size_select, sect_select;
> struct ndctl_ctx *ctx = ndctl_btt_get_ctx(btt);
> - struct ndctl_test *test = ndctl_get_private_data(ctx);
> + struct test_ctx *test = ndctl_get_private_data(ctx);
> struct ndctl_namespace *ndns = ndctl_btt_get_namespace(btt);
> unsigned long long expect_table[][2] = {
> [0] = {
> @@ -1049,7 +1049,7 @@ static int check_btt_create(struct ndctl_region *region, struct ndctl_namespace
> struct namespace *namespace)
> {
> struct ndctl_ctx *ctx = ndctl_region_get_ctx(region);
> - struct ndctl_test *test = ndctl_get_private_data(ctx);
> + struct test_ctx *test = ndctl_get_private_data(ctx);
> struct btt *btt_s = namespace->btt_settings;
> int i, fd, retry = 10;
> struct ndctl_btt *btt;
> @@ -1257,7 +1257,7 @@ static int check_pfn_autodetect(struct ndctl_bus *bus,
> struct ndctl_region *region = ndctl_namespace_get_region(ndns);
> struct ndctl_ctx *ctx = ndctl_region_get_ctx(region);
> const char *devname = ndctl_namespace_get_devname(ndns);
> - struct ndctl_test *test = ndctl_get_private_data(ctx);
> + struct test_ctx *test = ndctl_get_private_data(ctx);
> struct pfn *auto_pfn = namespace->pfn_settings;
> struct ndctl_pfn *pfn, *found = NULL;
> enum ndctl_namespace_mode mode;
> @@ -1354,7 +1354,7 @@ static int check_dax_autodetect(struct ndctl_bus *bus,
> struct ndctl_region *region = ndctl_namespace_get_region(ndns);
> struct ndctl_ctx *ctx = ndctl_region_get_ctx(region);
> const char *devname = ndctl_namespace_get_devname(ndns);
> - struct ndctl_test *test = ndctl_get_private_data(ctx);
> + struct test_ctx *test = ndctl_get_private_data(ctx);
> struct dax *auto_dax = namespace->dax_settings;
> struct ndctl_dax *dax, *found = NULL;
> enum ndctl_namespace_mode mode;
> @@ -1439,7 +1439,7 @@ static int check_btt_autodetect(struct ndctl_bus *bus,
> struct ndctl_region *region = ndctl_namespace_get_region(ndns);
> struct ndctl_ctx *ctx = ndctl_region_get_ctx(region);
> const char *devname = ndctl_namespace_get_devname(ndns);
> - struct ndctl_test *test = ndctl_get_private_data(ctx);
> + struct test_ctx *test = ndctl_get_private_data(ctx);
> struct btt *auto_btt = namespace->btt_settings;
> struct ndctl_btt *btt, *found = NULL;
> enum ndctl_namespace_mode mode;
> @@ -1681,7 +1681,7 @@ static int check_namespaces(struct ndctl_region *region,
> struct namespace **namespaces, enum ns_mode mode)
> {
> struct ndctl_ctx *ctx = ndctl_region_get_ctx(region);
> - struct ndctl_test *test = ndctl_get_private_data(ctx);
> + struct test_ctx *test = ndctl_get_private_data(ctx);
> struct ndctl_bus *bus = ndctl_region_get_bus(region);
> struct ndctl_namespace **ndns_save;
> struct namespace *namespace;
> @@ -2044,7 +2044,7 @@ static int check_btts(struct ndctl_region *region, struct btt **btts)
> struct check_cmd {
> int (*check_fn)(struct ndctl_bus *bus, struct ndctl_dimm *dimm, struct check_cmd *check);
> struct ndctl_cmd *cmd;
> - struct ndctl_test *test;
> + struct test_ctx *test;
> };
>
> static struct check_cmd *check_cmds;
> @@ -2412,7 +2412,7 @@ static int check_smart_threshold(struct ndctl_bus *bus, struct ndctl_dimm *dimm,
> #define BITS_PER_LONG 32
> static int check_commands(struct ndctl_bus *bus, struct ndctl_dimm *dimm,
> unsigned long bus_commands, unsigned long dimm_commands,
> - struct ndctl_test *test)
> + struct test_ctx *test)
> {
> /*
> * For now, by coincidence, these are indexed in test execution
> @@ -2483,7 +2483,7 @@ static int check_commands(struct ndctl_bus *bus, struct ndctl_dimm *dimm,
>
> static int check_dimms(struct ndctl_bus *bus, struct dimm *dimms, int n,
> unsigned long bus_commands, unsigned long dimm_commands,
> - struct ndctl_test *test)
> + struct test_ctx *test)
> {
> long long dsc;
> int i, j, rc;
> @@ -2604,7 +2604,7 @@ static void reset_bus(struct ndctl_bus *bus)
> ndctl_region_enable(region);
> }
>
> -static int do_test0(struct ndctl_ctx *ctx, struct ndctl_test *test)
> +static int do_test0(struct ndctl_ctx *ctx, struct test_ctx *test)
> {
> struct ndctl_bus *bus = ndctl_bus_get_by_provider(ctx, NFIT_PROVIDER0);
> struct ndctl_region *region;
> @@ -2662,7 +2662,7 @@ static int do_test0(struct ndctl_ctx *ctx, struct ndctl_test *test)
> return check_regions(bus, regions0, ARRAY_SIZE(regions0), BTT);
> }
>
> -static int do_test1(struct ndctl_ctx *ctx, struct ndctl_test *test)
> +static int do_test1(struct ndctl_ctx *ctx, struct test_ctx *test)
> {
> struct ndctl_bus *bus = ndctl_bus_get_by_provider(ctx, NFIT_PROVIDER1);
> int rc;
> @@ -2686,13 +2686,13 @@ static int do_test1(struct ndctl_ctx *ctx, struct ndctl_test *test)
> return check_regions(bus, regions1, ARRAY_SIZE(regions1), BTT);
> }
>
> -typedef int (*do_test_fn)(struct ndctl_ctx *ctx, struct ndctl_test *test);
> +typedef int (*do_test_fn)(struct ndctl_ctx *ctx, struct test_ctx *test);
> static do_test_fn do_test[] = {
> do_test0,
> do_test1,
> };
>
> -int test_libndctl(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx)
> +int test_libndctl(int loglevel, struct test_ctx *test, struct ndctl_ctx *ctx)
> {
> unsigned int i;
> struct kmod_module *mod;
> @@ -2732,7 +2732,7 @@ int test_libndctl(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx)
>
> int __attribute__((weak)) main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> struct ndctl_ctx *ctx;
> int rc;
>
> diff --git a/test/multi-pmem.c b/test/multi-pmem.c
> index 3ea08cc..e2f3bc5 100644
> --- a/test/multi-pmem.c
> +++ b/test/multi-pmem.c
> @@ -53,7 +53,7 @@ static void destroy_namespace(struct ndctl_namespace *ndns)
>
> /* Check that the namespace device is gone (if it wasn't the seed) */
> static int check_deleted(struct ndctl_region *region, const char *devname,
> - struct ndctl_test *test)
> + struct test_ctx *test)
> {
> struct ndctl_namespace *ndns;
>
> @@ -73,7 +73,7 @@ static int check_deleted(struct ndctl_region *region, const char *devname,
> return 0;
> }
>
> -static int do_multi_pmem(struct ndctl_ctx *ctx, struct ndctl_test *test)
> +static int do_multi_pmem(struct ndctl_ctx *ctx, struct test_ctx *test)
> {
> int i;
> char devname[100];
> @@ -238,7 +238,8 @@ static int do_multi_pmem(struct ndctl_ctx *ctx, struct ndctl_test *test)
> return 0;
> }
>
> -int test_multi_pmem(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx)
> +int test_multi_pmem(int loglevel, struct test_ctx *test,
> + struct ndctl_ctx *ctx)
> {
> struct kmod_module *mod;
> struct kmod_ctx *kmod_ctx;
> @@ -267,7 +268,7 @@ int test_multi_pmem(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx
>
> int __attribute__((weak)) main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> struct ndctl_ctx *ctx;
> int rc;
>
> diff --git a/test/parent-uuid.c b/test/parent-uuid.c
> index bded33a..dd007c7 100644
> --- a/test/parent-uuid.c
> +++ b/test/parent-uuid.c
> @@ -208,7 +208,8 @@ static int do_test(struct ndctl_ctx *ctx)
> return 0;
> }
>
> -int test_parent_uuid(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ctx)
> +int test_parent_uuid(int loglevel, struct test_ctx *test,
> + struct ndctl_ctx *ctx)
> {
> struct kmod_module *mod;
> struct kmod_ctx *kmod_ctx;
> @@ -235,7 +236,7 @@ int test_parent_uuid(int loglevel, struct ndctl_test *test, struct ndctl_ctx *ct
>
> int __attribute__((weak)) main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> struct ndctl_ctx *ctx;
> int rc;
>
> diff --git a/test/pmem_namespaces.c b/test/pmem_namespaces.c
> index a4db1ae..eddf32a 100644
> --- a/test/pmem_namespaces.c
> +++ b/test/pmem_namespaces.c
> @@ -161,8 +161,8 @@ static int ns_do_io(const char *bdev)
>
> static const char *comm = "test-pmem-namespaces";
>
> -int test_pmem_namespaces(int log_level, struct ndctl_test *test,
> - struct ndctl_ctx *ctx)
> +int test_pmem_namespaces(int log_level, struct test_ctx *test,
> + struct ndctl_ctx *ctx)
> {
> struct ndctl_region *region, *pmem_region = NULL;
> struct kmod_ctx *kmod_ctx = NULL;
> @@ -262,7 +262,7 @@ int test_pmem_namespaces(int log_level, struct ndctl_test *test,
>
> int __attribute__((weak)) main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> struct ndctl_ctx *ctx;
> int rc;
>
> diff --git a/test/revoke-devmem.c b/test/revoke-devmem.c
> index bb8979e..0d67d93 100644
> --- a/test/revoke-devmem.c
> +++ b/test/revoke-devmem.c
> @@ -32,8 +32,8 @@ static void sigbus(int sig, siginfo_t *siginfo, void *d)
> #define err(fmt, ...) \
> fprintf(stderr, "%s: " fmt, __func__, ##__VA_ARGS__)
>
> -static int test_devmem(int loglevel, struct ndctl_test *test,
> - struct ndctl_ctx *ctx)
> +static int test_devmem(int loglevel, struct test_ctx *test,
> + struct ndctl_ctx *ctx)
> {
> void *buf;
> int fd, rc;
> @@ -124,7 +124,7 @@ out_devmem:
>
> int main(int argc, char *argv[])
> {
> - struct ndctl_test *test = ndctl_test_new(0);
> + struct test_ctx *test = ndctl_test_new(0);
> struct ndctl_ctx *ctx;
> int rc;
>
> --
> 2.31.1
>
next prev parent reply other threads:[~2021-07-15 23:51 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-01 20:09 [ndctl PATCH v3 00/21] Initial CXL support Vishal Verma
2021-07-01 20:09 ` [ndctl PATCH v3 01/21] ndctl: add .clang-format Vishal Verma
2021-07-10 1:12 ` Dan Williams
2021-07-01 20:09 ` [ndctl PATCH v3 02/21] cxl: add a cxl utility and libcxl library Vishal Verma
2021-07-10 1:12 ` Dan Williams
2021-07-13 21:14 ` Verma, Vishal L
2021-07-01 20:09 ` [ndctl PATCH v3 03/21] cxl: add a local copy of the cxl_mem UAPI header Vishal Verma
2021-07-10 1:13 ` Dan Williams
2021-07-01 20:09 ` [ndctl PATCH v3 04/21] libcxl: add support for command query and submission Vishal Verma
2021-07-13 5:12 ` Dan Williams
2021-07-13 21:17 ` Verma, Vishal L
2021-07-13 21:39 ` Dan Williams
2021-07-01 20:09 ` [ndctl PATCH v3 05/21] libcxl: add support for the 'Identify Device' command Vishal Verma
2021-07-15 23:50 ` Dan Williams
2021-07-01 20:09 ` [ndctl PATCH v3 06/21] test: rename 'ndctl_test' to 'test_ctx' Vishal Verma
2021-07-15 23:51 ` Dan Williams [this message]
2021-07-01 20:09 ` [ndctl PATCH v3 07/21] test: rename 'ndctl_test_*' helpers to 'test_*' Vishal Verma
2021-07-15 23:51 ` Dan Williams
2021-07-01 20:09 ` [ndctl PATCH v3 08/21] test: introduce a libcxl unit test Vishal Verma
2021-07-15 23:53 ` Dan Williams
2021-07-01 20:09 ` [ndctl PATCH v3 09/21] libcxl: add GET_HEALTH_INFO mailbox command and accessors Vishal Verma
2021-07-16 1:11 ` Dan Williams
2021-07-01 20:09 ` [ndctl PATCH v3 10/21] libcxl: add support for the 'GET_LSA' command Vishal Verma
2021-07-16 2:24 ` Dan Williams
2021-07-01 20:09 ` [ndctl PATCH v3 11/21] util/hexdump: Add a util helper to print a buffer in hex Vishal Verma
2021-07-01 20:09 ` [ndctl PATCH v3 12/21] test/libcxl: add a test for {set, get}_lsa commands Vishal Verma
2021-07-01 20:09 ` [ndctl PATCH v3 13/21] test/libcxl: introduce a command size fuzzing test Vishal Verma
2021-07-01 20:09 ` [ndctl PATCH v3 14/21] libcxl: add lsa_size to cxl_memdev, and an API to retrieve it Vishal Verma
2021-07-01 20:09 ` [ndctl PATCH v3 15/21] libcxl: PLACEHOLDER: add an interface to determine whether a memdev is active Vishal Verma
2021-07-01 20:10 ` [ndctl PATCH v3 16/21] libcxl: add interfaces for label operations Vishal Verma
2021-07-01 20:10 ` [ndctl PATCH v3 17/21] test/libcxl: add a test for cxl_memdev_{get,set}_lsa Vishal Verma
2021-07-01 20:10 ` [ndctl PATCH v3 18/21] cxl: add commands to read, write, and zero labels Vishal Verma
2021-07-01 20:10 ` [ndctl PATCH v3 19/21] Documentation/cxl: add library API documentation Vishal Verma
2021-07-01 20:10 ` [ndctl PATCH v3 20/21] ndctl: Add CXL packages to the RPM spec Vishal Verma
2021-07-01 20:10 ` [ndctl PATCH v3 21/21] cxl-cli: add bash completion Vishal Verma
2021-07-01 20:13 ` [ndctl PATCH v3 00/21] Initial CXL support Verma, Vishal L
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='CAPcyv4it2Ubzf62x4ZrFL=ivJJgjJko4JgA=QbigSrD9wfsSnQ@mail.gmail.com' \
--to=dan.j.williams@intel.com \
--cc=alison.schofield@intel.com \
--cc=ben.widawsky@intel.com \
--cc=ira.weiny@intel.com \
--cc=linux-cxl@vger.kernel.org \
--cc=nvdimm@lists.linux.dev \
--cc=vishal.l.verma@intel.com \
/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 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).