* [Cluster-devel] [PATCH dlm/next 2/3] fs: dlm: const void resource name parameter
2022-08-02 17:21 [Cluster-devel] [PATCH dlm/next 1/3] fs: dlm: allow lockspaces have zero lvblen Alexander Aring
@ 2022-08-02 17:21 ` Alexander Aring
2022-08-02 17:21 ` [Cluster-devel] [PATCH dlm/next 3/3] fs: dlm: initial commit of locktorture Alexander Aring
1 sibling, 0 replies; 3+ messages in thread
From: Alexander Aring @ 2022-08-02 17:21 UTC (permalink / raw)
To: cluster-devel.redhat.com
The resource name parameter should never be changed by DLM so we declare
it as const. At some point it is handled as a char pointer, a resource
name can be a non printable ascii string as well. This patch change it
to handle it as void pointer as it is offered by DLM API.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
---
fs/dlm/lock.c | 23 +++++++++++++----------
fs/dlm/lock.h | 2 +-
include/linux/dlm.h | 2 +-
include/trace/events/dlm.h | 4 ++--
4 files changed, 17 insertions(+), 14 deletions(-)
diff --git a/fs/dlm/lock.c b/fs/dlm/lock.c
index 078af3805a88..fb64f5a53603 100644
--- a/fs/dlm/lock.c
+++ b/fs/dlm/lock.c
@@ -401,7 +401,7 @@ static int pre_rsb_struct(struct dlm_ls *ls)
unlock any spinlocks, go back and call pre_rsb_struct again.
Otherwise, take an rsb off the list and return it. */
-static int get_rsb_struct(struct dlm_ls *ls, char *name, int len,
+static int get_rsb_struct(struct dlm_ls *ls, const void *name, int len,
struct dlm_rsb **r_ret)
{
struct dlm_rsb *r;
@@ -412,7 +412,8 @@ static int get_rsb_struct(struct dlm_ls *ls, char *name, int len,
count = ls->ls_new_rsb_count;
spin_unlock(&ls->ls_new_rsb_spin);
log_debug(ls, "find_rsb retry %d %d %s",
- count, dlm_config.ci_new_rsb_count, name);
+ count, dlm_config.ci_new_rsb_count,
+ (const char *)name);
return -EAGAIN;
}
@@ -448,7 +449,7 @@ static int rsb_cmp(struct dlm_rsb *r, const char *name, int nlen)
return memcmp(r->res_name, maxname, DLM_RESNAME_MAXLEN);
}
-int dlm_search_rsb_tree(struct rb_root *tree, char *name, int len,
+int dlm_search_rsb_tree(struct rb_root *tree, const void *name, int len,
struct dlm_rsb **r_ret)
{
struct rb_node *node = tree->rb_node;
@@ -546,7 +547,7 @@ static int rsb_insert(struct dlm_rsb *rsb, struct rb_root *tree)
* while that rsb has a potentially stale master.)
*/
-static int find_rsb_dir(struct dlm_ls *ls, char *name, int len,
+static int find_rsb_dir(struct dlm_ls *ls, const void *name, int len,
uint32_t hash, uint32_t b,
int dir_nodeid, int from_nodeid,
unsigned int flags, struct dlm_rsb **r_ret)
@@ -724,7 +725,7 @@ static int find_rsb_dir(struct dlm_ls *ls, char *name, int len,
dlm_recover_locks) before we've made ourself master (in
dlm_recover_masters). */
-static int find_rsb_nodir(struct dlm_ls *ls, char *name, int len,
+static int find_rsb_nodir(struct dlm_ls *ls, const void *name, int len,
uint32_t hash, uint32_t b,
int dir_nodeid, int from_nodeid,
unsigned int flags, struct dlm_rsb **r_ret)
@@ -818,8 +819,9 @@ static int find_rsb_nodir(struct dlm_ls *ls, char *name, int len,
return error;
}
-static int find_rsb(struct dlm_ls *ls, char *name, int len, int from_nodeid,
- unsigned int flags, struct dlm_rsb **r_ret)
+static int find_rsb(struct dlm_ls *ls, const void *name, int len,
+ int from_nodeid, unsigned int flags,
+ struct dlm_rsb **r_ret)
{
uint32_t hash, b;
int dir_nodeid;
@@ -3316,8 +3318,9 @@ static int _cancel_lock(struct dlm_rsb *r, struct dlm_lkb *lkb)
* request_lock(), convert_lock(), unlock_lock(), cancel_lock()
*/
-static int request_lock(struct dlm_ls *ls, struct dlm_lkb *lkb, char *name,
- int len, struct dlm_args *args)
+static int request_lock(struct dlm_ls *ls, struct dlm_lkb *lkb,
+ const void *name, int len,
+ struct dlm_args *args)
{
struct dlm_rsb *r;
int error;
@@ -3416,7 +3419,7 @@ int dlm_lock(dlm_lockspace_t *lockspace,
int mode,
struct dlm_lksb *lksb,
uint32_t flags,
- void *name,
+ const void *name,
unsigned int namelen,
uint32_t parent_lkid,
void (*ast) (void *astarg),
diff --git a/fs/dlm/lock.h b/fs/dlm/lock.h
index a7b6474f009d..40c76b5544da 100644
--- a/fs/dlm/lock.h
+++ b/fs/dlm/lock.h
@@ -36,7 +36,7 @@ static inline void dlm_adjust_timeouts(struct dlm_ls *ls) { }
int dlm_master_lookup(struct dlm_ls *ls, int nodeid, char *name, int len,
unsigned int flags, int *r_nodeid, int *result);
-int dlm_search_rsb_tree(struct rb_root *tree, char *name, int len,
+int dlm_search_rsb_tree(struct rb_root *tree, const void *name, int len,
struct dlm_rsb **r_ret);
void dlm_recover_purge(struct dlm_ls *ls);
diff --git a/include/linux/dlm.h b/include/linux/dlm.h
index f5f55c2138ae..c6bc2b5ee7e6 100644
--- a/include/linux/dlm.h
+++ b/include/linux/dlm.h
@@ -131,7 +131,7 @@ int dlm_lock(dlm_lockspace_t *lockspace,
int mode,
struct dlm_lksb *lksb,
uint32_t flags,
- void *name,
+ const void *name,
unsigned int namelen,
uint32_t parent_lkid,
void (*lockast) (void *astarg),
diff --git a/include/trace/events/dlm.h b/include/trace/events/dlm.h
index 92d263aab848..546579dee6a9 100644
--- a/include/trace/events/dlm.h
+++ b/include/trace/events/dlm.h
@@ -49,7 +49,7 @@
/* note: we begin tracing dlm_lock_start() only if ls and lkb are found */
TRACE_EVENT(dlm_lock_start,
- TP_PROTO(struct dlm_ls *ls, struct dlm_lkb *lkb, void *name,
+ TP_PROTO(struct dlm_ls *ls, struct dlm_lkb *lkb, const void *name,
unsigned int namelen, int mode, __u32 flags),
TP_ARGS(ls, lkb, name, namelen, mode, flags),
@@ -91,7 +91,7 @@ TRACE_EVENT(dlm_lock_start,
TRACE_EVENT(dlm_lock_end,
- TP_PROTO(struct dlm_ls *ls, struct dlm_lkb *lkb, void *name,
+ TP_PROTO(struct dlm_ls *ls, struct dlm_lkb *lkb, const void *name,
unsigned int namelen, int mode, __u32 flags, int error,
bool kernel_lock),
--
2.31.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* [Cluster-devel] [PATCH dlm/next 3/3] fs: dlm: initial commit of locktorture
2022-08-02 17:21 [Cluster-devel] [PATCH dlm/next 1/3] fs: dlm: allow lockspaces have zero lvblen Alexander Aring
2022-08-02 17:21 ` [Cluster-devel] [PATCH dlm/next 2/3] fs: dlm: const void resource name parameter Alexander Aring
@ 2022-08-02 17:21 ` Alexander Aring
1 sibling, 0 replies; 3+ messages in thread
From: Alexander Aring @ 2022-08-02 17:21 UTC (permalink / raw)
To: cluster-devel.redhat.com
This patch will introduce a locktorture test for DLM subsystem. The idea
is to have a torture test to proof some performance indication for DLM.
This torture test will allocate for each lock task a DLM lock and try to
acquire the lock as much as it can. In a homogeneous cluster (all nodes
have equal hardware) other nodes will try to acquire those locks as well.
You can run it by "modprobe dlm_locktorture cluster=$CLUSTER_NAME", note
that cluster is required to provide a cluster name for the cluster
manager.
Currently there is only one simple lock operation which is to create a
lock in NL state and switch to EX and NL with a little bit of delay in
the middle to simulate lock contention. This locktorture module uses the
locktorture API from the linux kernel to provide such functionality.
However the lock ops are very different and currently handled as
start/stop and a iteration of a "testing step" e.g. switch to EX and NL
over some looping behaviour. In future we can add more test regarding
different lock modes in dlm or pressure tests for functionality such as
lock request cancellation functionality.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
---
fs/dlm/Kconfig | 11 +
fs/dlm/Makefile | 1 +
fs/dlm/dlm_locktorture.c | 514 +++++++++++++++++++++++++++++++++++++++
3 files changed, 526 insertions(+)
create mode 100644 fs/dlm/dlm_locktorture.c
diff --git a/fs/dlm/Kconfig b/fs/dlm/Kconfig
index 1105ce3c80cb..50313d598489 100644
--- a/fs/dlm/Kconfig
+++ b/fs/dlm/Kconfig
@@ -25,3 +25,14 @@ config DLM_DEBUG
Under the debugfs mount point, the name of each lockspace will
appear as a file in the "dlm" directory. The output is the
list of resource and locks the local node knows about.
+
+config DLM_LOCKTORTURE
+ tristate "DLM locktorture"
+ depends on DLM && m
+ select TORTURE_TEST
+ help
+ This options provides a kernel module that runs torture tests on
+ the DLM subsystem. If loaded on a homogeneous cluster setup (e.g.
+ all cluster nodes have the same architecture) it will run
+ concurrent lock and unlock procedures. The printed stats will show
+ how many lock acquire were possible.
diff --git a/fs/dlm/Makefile b/fs/dlm/Makefile
index 71dab733cf9a..4d333b4502ba 100644
--- a/fs/dlm/Makefile
+++ b/fs/dlm/Makefile
@@ -19,4 +19,5 @@ dlm-y := ast.o \
util.o
dlm-$(CONFIG_DLM_DEPRECATED_API) += netlink.o
dlm-$(CONFIG_DLM_DEBUG) += debug_fs.o
+obj-$(CONFIG_DLM_LOCKTORTURE) += dlm_locktorture.o
diff --git a/fs/dlm/dlm_locktorture.c b/fs/dlm/dlm_locktorture.c
new file mode 100644
index 000000000000..00ac6ac5fe42
--- /dev/null
+++ b/fs/dlm/dlm_locktorture.c
@@ -0,0 +1,514 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Module-based torture test facility for dlm locking
+ *
+ * Copyright (C) 2022 Red Hat, Inc. All rights reserved.
+ * Copyright (C) IBM Corporation, 2014
+ *
+ * Authors: Alexander Aring <aahringo@redhat.com>
+ *
+ * Original Authors: Paul E. McKenney <paulmck@linux.ibm.com>
+ * Davidlohr Bueso <dave@stgolabs.net>
+ *
+ * Based on kernel/locking/locktorture.c.
+ */
+
+#define pr_fmt(fmt) fmt
+
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/kthread.h>
+#include <linux/sched/rt.h>
+#include <linux/smp.h>
+#include <linux/interrupt.h>
+#include <linux/sched.h>
+#include <uapi/linux/sched/types.h>
+#include <linux/moduleparam.h>
+#include <linux/delay.h>
+#include <linux/slab.h>
+#include <linux/torture.h>
+#include <linux/reboot.h>
+#include <linux/dlm.h>
+
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Alexander Aring <aahringo@redhat.com>");
+
+torture_param(int, nlock_stress, -1,
+ "Number of locking stress-test threads");
+torture_param(int, shuffle_interval, 3,
+ "Number of jiffies between shuffles, 0=disable");
+torture_param(int, shutdown_secs, 0, "Shutdown time (j), <= zero to disable.");
+torture_param(int, stat_interval, 60,
+ "Number of seconds between stats printk()s");
+torture_param(int, verbose, 1,
+ "Enable verbose debugging printk()s");
+/* because torture_param() wants to use charp */
+typedef char *charp;
+torture_param(charp, cluster, NULL,
+ "Cluster name that lockspace will join");
+
+#define DLM_LOCKTORTURE_RES_NAME_LEN (DLM_RESNAME_MAXLEN + 1)
+
+static struct task_struct **lock_tasks;
+static struct task_struct *stats_task;
+static char *torture_type = "exnl";
+static dlm_lockspace_t *ls;
+static long long prev_sum;
+
+struct lock_stress_stats {
+ long n_iter;
+};
+
+struct lock_data {
+ struct lock_stress_stats s;
+
+ char res_name[DLM_LOCKTORTURE_RES_NAME_LEN];
+ size_t res_name_len;
+};
+
+/*
+ * Operations vector for selecting different types of tests.
+ */
+struct lock_torture_ops {
+ int (*start)(struct dlm_lksb *lksb, const char *res_name,
+ size_t res_name_len);
+ int (*iter)(struct dlm_lksb *lksb, const char *res_name,
+ size_t res_name_len, struct torture_random_state *trsp);
+ int (*stop)(struct dlm_lksb *lksb);
+
+ const char *name;
+};
+
+struct lock_torture_cxt {
+ int nreallock_stress;
+ struct lock_torture_ops *cur_ops;
+ struct lock_data *lwd;
+};
+static struct lock_torture_cxt cxt = { 0, NULL, NULL};
+
+static void ast(void *astarg)
+{
+ complete(astarg);
+ pr_debug("dlm_locktorture: %s\n", __func__);
+}
+
+static void bast(void *astarg, int mode)
+{
+ pr_debug("dlm_locktorture: %s mode: %d\n", __func__, mode);
+}
+
+static void torture_delay(struct torture_random_state *trsp)
+{
+ const unsigned long longdelay_ms = 100;
+
+ /* We want a long delay occasionally to force massive contention. */
+ if (!(torture_random(trsp) %
+ (cxt.nreallock_stress * 2000 * longdelay_ms)))
+ mdelay(longdelay_ms * 5);
+ else
+ mdelay(longdelay_ms / 5);
+ if (!(torture_random(trsp) % (cxt.nreallock_stress * 20000)))
+ torture_preempt_schedule(); /* Allow test to be preempted. */
+}
+
+static int torture_dlm_lock_sync(int mode, uint32_t flags,
+ struct dlm_lksb *lksb, const char *res_name,
+ size_t res_name_len)
+{
+ struct completion completion;
+ int ret;
+
+ init_completion(&completion);
+retry:
+ ret = dlm_lock(ls, mode, lksb, flags, res_name, res_name_len, 0, ast,
+ &completion, bast);
+ switch (ret) {
+ case 0:
+ wait_for_completion(&completion);
+ return 0;
+ case -EBUSY:
+ goto retry;
+ default:
+ break;
+ }
+
+ return ret;
+}
+
+static int torture_start(struct dlm_lksb *lksb, const char *res_name,
+ size_t res_name_len)
+{
+ return torture_dlm_lock_sync(DLM_LOCK_NL, 0, lksb, res_name,
+ res_name_len);
+}
+
+static int torture_stop(struct dlm_lksb *lksb)
+{
+ struct completion completion;
+ int ret;
+
+ init_completion(&completion);
+retry:
+ ret = dlm_unlock(ls, lksb->sb_lkid, 0, lksb, &completion);
+ switch (ret) {
+ case 0:
+ wait_for_completion(&completion);
+ return 0;
+ case -EBUSY:
+ goto retry;
+ default:
+ break;
+ }
+
+ return ret;
+}
+
+/* exclusive lock case, switch between EX and NL */
+
+static int torture_ex_iter(struct dlm_lksb *lksb, const char *res_name,
+ size_t res_name_len, struct torture_random_state *trsp)
+{
+ int ret;
+
+ ret = torture_dlm_lock_sync(DLM_LOCK_EX, DLM_LKF_CONVERT,
+ lksb, res_name, res_name_len);
+ if (ret)
+ return ret;
+
+ /* fake lock contention */
+ torture_delay(trsp);
+
+ ret = torture_dlm_lock_sync(DLM_LOCK_NL, DLM_LKF_CONVERT,
+ lksb, res_name, res_name_len);
+ if (ret)
+ return ret;
+
+ return 0;
+}
+
+static struct lock_torture_ops exnl_lock_ops = {
+ .start = torture_start,
+ .iter = torture_ex_iter,
+ .stop = torture_stop,
+ .name = "exnl"
+};
+
+/*
+ * Lock torture lock kthread. Repeatedly acquires and releases
+ * the lock, checking for duplicate acquisitions.
+ */
+static int lock_torture(void *arg)
+{
+ struct lock_data *lwd = arg;
+ DEFINE_TORTURE_RANDOM(rand);
+ struct dlm_lksb lksb;
+ int ret;
+
+ VERBOSE_TOROUT_STRING("lock_torture task started");
+ set_user_nice(current, MAX_NICE);
+
+ ret = cxt.cur_ops->start(&lksb, lwd->res_name, lwd->res_name_len);
+ if (WARN_ON_ONCE(ret))
+ return ret;
+
+ do {
+ if ((torture_random(&rand) & 0xfffff) == 0)
+ schedule_timeout_uninterruptible(1);
+
+ ret = cxt.cur_ops->iter(&lksb, lwd->res_name,
+ lwd->res_name_len, &rand);
+ if (WARN_ON_ONCE(ret))
+ break;
+
+ lwd->s.n_iter++;
+ } while (!torture_must_stop());
+
+ ret = cxt.cur_ops->stop(&lksb);
+ if (WARN_ON_ONCE(ret))
+ return ret;
+
+ torture_kthread_stopping("lock_torture");
+ return 0;
+}
+
+/*
+ * Create an lock-torture-statistics message in the specified buffer.
+ */
+static void __torture_print_stats(char *page,
+ struct lock_data *ld)
+{
+ long long sum = 0, sum_diff;
+ int i, n_stress;
+
+ n_stress = cxt.nreallock_stress;
+ for (i = 0; i < n_stress; i++)
+ sum += ld[i].s.n_iter;
+
+ sum_diff = sum - prev_sum;
+ prev_sum = sum;
+
+ page += sprintf(page, "Iterations: %lld\n", sum_diff);
+}
+
+/*
+ * Print torture statistics. Caller must ensure that there is only one
+ * call to this function@a given time!!! This is normally accomplished
+ * by relying on the module system to only have one copy of the module
+ * loaded, and then by giving the lock_torture_stats kthread full control
+ * (or the init/cleanup functions when lock_torture_stats thread is not
+ * running).
+ */
+static void lock_torture_stats_print(void)
+{
+ int size = cxt.nreallock_stress * 200 + 8192;
+ char *buf;
+
+ buf = kmalloc(size, GFP_KERNEL);
+ if (!buf) {
+ pr_err("%s: Out of memory, need: %d",
+ __func__, size);
+ return;
+ }
+
+ __torture_print_stats(buf, cxt.lwd);
+ pr_alert("%s", buf);
+ kfree(buf);
+}
+
+/*
+ * Periodically prints torture statistics, if periodic statistics printing
+ * was specified via the stat_interval module parameter.
+ *
+ * No need to worry about fullstop here, since this one doesn't reference
+ * volatile state or register callbacks.
+ */
+static int lock_torture_stats(void *arg)
+{
+ VERBOSE_TOROUT_STRING("lock_torture_stats task started");
+ do {
+ schedule_timeout_interruptible(stat_interval * HZ);
+ lock_torture_stats_print();
+ torture_shutdown_absorb("lock_torture_stats");
+ } while (!torture_must_stop());
+ torture_kthread_stopping("lock_torture_stats");
+ return 0;
+}
+
+static inline void
+lock_torture_print_module_parms(struct lock_torture_ops *cur_ops,
+ const char *tag)
+{
+ pr_alert("%s" TORTURE_FLAG
+ "--- %s: cluser=%s nlock_stress=%d stat_interval=%d verbose=%d shuffle_interval=%d shutdown_secs=%d\n",
+ torture_type, tag, cluster, cxt.nreallock_stress,
+ stat_interval, verbose, shuffle_interval, shutdown_secs);
+}
+
+static void lock_torture_cleanup(void)
+{
+ int i, ret;
+
+ if (torture_cleanup_begin())
+ return;
+
+ /*
+ * Indicates early cleanup, meaning that the test has not run,
+ * such as when passing bogus args when loading the module.
+ * However cxt->cur_ops.init() may have been invoked, so beside
+ * perform the underlying torture-specific cleanups, cur_ops.exit()
+ * will be invoked if needed.
+ */
+ if (!cxt.lwd)
+ goto end;
+
+ if (lock_tasks) {
+ for (i = 0; i < cxt.nreallock_stress; i++)
+ torture_stop_kthread(lock_torture, lock_tasks[i]);
+
+ kfree(lock_tasks);
+ lock_tasks = NULL;
+
+ ret = dlm_release_lockspace(ls, 2);
+ WARN_ON(ret);
+ ls = NULL;
+ }
+
+ torture_stop_kthread(lock_torture_stats, stats_task);
+
+ lock_torture_stats_print(); /* -After- the stats thread is stopped! */
+
+ if (torture_onoff_failures())
+ lock_torture_print_module_parms(cxt.cur_ops,
+ "End of test: LOCK_HOTPLUG");
+ else
+ lock_torture_print_module_parms(cxt.cur_ops,
+ "End of test: SUCCESS");
+
+ kfree(cxt.lwd);
+ cxt.lwd = NULL;
+
+end:
+ torture_cleanup_end();
+}
+
+static void recover_prep(void *arg)
+{
+ pr_info("dlm_locktorture: %s\n", __func__);
+}
+
+static void recover_slot(void *arg, struct dlm_slot *slot)
+{
+ pr_info("dlm_locktorture: %s nodeid: %d slot: %d\n", __func__,
+ slot->nodeid, slot->slot);
+}
+
+static void recover_done(void *arg, struct dlm_slot *slots,
+ int num_slots, int our_slot,
+ uint32_t generation)
+{
+ int i;
+
+ pr_info("dlm_locktorture: %s num_slots: %d our_slot: %d generation: %u\n",
+ __func__, num_slots, our_slot, generation);
+
+ for (i = 0; i < num_slots; i++) {
+ pr_info("dlm_locktorture: %s slot->nodeid: %d slot->slot: %d\n",
+ __func__, slots[i].nodeid, slots[i].slot);
+ }
+}
+
+static const struct dlm_lockspace_ops torture_ls_ops = {
+ .recover_prep = recover_prep,
+ .recover_slot = recover_slot,
+ .recover_done = recover_done,
+};
+
+static int __init lock_torture_init(void)
+{
+ static struct lock_torture_ops *torture_ops[] = {
+ &exnl_lock_ops,
+ };
+ char str[DLM_LOCKTORTURE_RES_NAME_LEN];
+ int i, ret;
+
+ if (!cluster) {
+ pr_err("dlm_locktorture: cluster parameter required\n");
+ return -EINVAL;
+ }
+
+ if (!torture_init_begin(torture_type, verbose))
+ return -EBUSY;
+
+ /* Process args and tell the world that the torturer is on the job. */
+ for (i = 0; i < ARRAY_SIZE(torture_ops); i++) {
+ cxt.cur_ops = torture_ops[i];
+ if (strcmp(torture_type, cxt.cur_ops->name) == 0)
+ break;
+ }
+ if (i == ARRAY_SIZE(torture_ops)) {
+ pr_alert("lock-torture: invalid torture type: \"%s\"\n",
+ torture_type);
+ pr_alert("lock-torture types:");
+ for (i = 0; i < ARRAY_SIZE(torture_ops); i++)
+ pr_alert(" %s", torture_ops[i]->name);
+ pr_alert("\n");
+ ret = -EINVAL;
+ goto err;
+ }
+
+ if (nlock_stress >= 0)
+ cxt.nreallock_stress = nlock_stress;
+ else
+ cxt.nreallock_stress = 2 * num_online_cpus();
+
+ /* Initialize the statistics so that each run gets its own numbers. */
+ if (nlock_stress) {
+ cxt.lwd = kmalloc_array(cxt.nreallock_stress,
+ sizeof(*cxt.lwd),
+ GFP_KERNEL);
+ if (cxt.lwd == NULL) {
+ VERBOSE_TOROUT_STRING("cxt.lwd: Out of memory");
+ ret = -ENOMEM;
+ goto err;
+ }
+
+ for (i = 0; i < cxt.nreallock_stress; i++) {
+ cxt.lwd[i].s.n_iter = 0;
+
+ snprintf(str, DLM_LOCKTORTURE_RES_NAME_LEN, "%s_%d",
+ cxt.cur_ops->name, i);
+ snprintf(cxt.lwd[i].res_name, DLM_LOCKTORTURE_RES_NAME_LEN,
+ "%-64s", str);
+ cxt.lwd[i].res_name_len = strlen(cxt.lwd[i].res_name);
+ }
+ }
+
+ lock_torture_print_module_parms(cxt.cur_ops, "Start of test");
+
+ /* Prepare torture context. */
+ if (shuffle_interval > 0) {
+ ret = torture_shuffle_init(shuffle_interval);
+ if (ret)
+ goto err;
+ }
+
+ if (shutdown_secs > 0) {
+ ret = torture_shutdown_init(shutdown_secs,
+ lock_torture_cleanup);
+ if (ret)
+ goto err;
+ }
+
+ if (nlock_stress) {
+ lock_tasks = kcalloc(cxt.nreallock_stress,
+ sizeof(lock_tasks[0]), GFP_KERNEL);
+ if (lock_tasks == NULL) {
+ TOROUT_ERRSTRING("lock_tasks: Out of memory");
+ ret = -ENOMEM;
+ goto err;
+ }
+
+ ret = dlm_new_lockspace("locktorture", cluster, 0, 64, &torture_ls_ops,
+ NULL, &ret, &ls);
+ if (ret)
+ goto err;
+ }
+
+ /*
+ * Create the kthreads and start torturing (oh, those poor little dlm locks).
+ */
+ for (i = 0; i < cxt.nreallock_stress; i++) {
+ /* Create lockers. */
+ ret = torture_create_kthread(lock_torture, &cxt.lwd[i],
+ lock_tasks[i]);
+ if (ret)
+ goto err;
+ }
+
+ if (stat_interval > 0) {
+ ret = torture_create_kthread(lock_torture_stats, NULL,
+ stats_task);
+ if (ret)
+ goto err;
+ }
+
+ torture_init_end();
+
+ return 0;
+
+err:
+ torture_init_end();
+
+ if (ls) {
+ ret = dlm_release_lockspace(ls, 2);
+ WARN_ON(ret);
+ }
+
+ if (shutdown_secs)
+ kernel_power_off();
+
+ return ret;
+}
+
+module_init(lock_torture_init);
+module_exit(lock_torture_cleanup);
--
2.31.1
^ permalink raw reply related [flat|nested] 3+ messages in thread