nvdimm.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
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
>

  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).