[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240827180236.316946-5-aahringo@redhat.com>
Date: Tue, 27 Aug 2024 14:02:33 -0400
From: Alexander Aring <aahringo@...hat.com>
To: teigland@...hat.com
Cc: gfs2@...ts.linux.dev,
song@...nel.org,
yukuai3@...wei.com,
agruenba@...hat.com,
mark@...heh.com,
jlbec@...lplan.org,
joseph.qi@...ux.alibaba.com,
gregkh@...uxfoundation.org,
rafael@...nel.org,
akpm@...ux-foundation.org,
linux-kernel@...r.kernel.org,
linux-raid@...r.kernel.org,
ocfs2-devel@...ts.linux.dev,
netdev@...r.kernel.org,
vvidic@...entin-vidic.from.hr,
heming.zhao@...e.com,
lucien.xin@...il.com,
paulmck@...nel.org,
rcu@...r.kernel.org,
juri.lelli@...hat.com,
williams@...hat.com,
aahringo@...hat.com
Subject: [RFC 4/7] dlm: add our_nodeid to tracepoints
This patch adds our_nodeid to some DLM tracepoints that are necessary
for the DLM kernel verifier to know from which nodeid the traceevent
comes from. This is useful when using DLM in net-namespaces to get a
whole cluster-view of DLM in traces.
Signed-off-by: Alexander Aring <aahringo@...hat.com>
---
fs/dlm/ast.c | 23 +++++++++++++----------
fs/dlm/dlm_internal.h | 1 +
fs/dlm/user.c | 9 +++++----
include/trace/events/dlm.h | 36 +++++++++++++++++++++++-------------
4 files changed, 42 insertions(+), 27 deletions(-)
diff --git a/fs/dlm/ast.c b/fs/dlm/ast.c
index 0fe8d80ce5e8..01de0d4b9450 100644
--- a/fs/dlm/ast.c
+++ b/fs/dlm/ast.c
@@ -18,20 +18,21 @@
#include "user.h"
#include "ast.h"
-static void dlm_run_callback(uint32_t ls_id, uint32_t lkb_id, int8_t mode,
- uint32_t flags, uint8_t sb_flags, int sb_status,
- struct dlm_lksb *lksb,
+static void dlm_run_callback(int our_nodeid, uint32_t ls_id, uint32_t lkb_id,
+ int8_t mode, uint32_t flags, uint8_t sb_flags,
+ int sb_status, struct dlm_lksb *lksb,
void (*astfn)(void *astparam),
void (*bastfn)(void *astparam, int mode),
void *astparam, const char *res_name,
size_t res_length)
{
if (flags & DLM_CB_BAST) {
- trace_dlm_bast(ls_id, lkb_id, mode, res_name, res_length);
+ trace_dlm_bast(our_nodeid, ls_id, lkb_id, mode, res_name,
+ res_length);
bastfn(astparam, mode);
} else if (flags & DLM_CB_CAST) {
- trace_dlm_ast(ls_id, lkb_id, sb_flags, sb_status, res_name,
- res_length);
+ trace_dlm_ast(our_nodeid, ls_id, lkb_id, sb_flags, sb_status,
+ res_name, res_length);
lksb->sb_status = sb_status;
lksb->sb_flags = sb_flags;
astfn(astparam);
@@ -40,8 +41,8 @@ static void dlm_run_callback(uint32_t ls_id, uint32_t lkb_id, int8_t mode,
static void dlm_do_callback(struct dlm_callback *cb)
{
- dlm_run_callback(cb->ls_id, cb->lkb_id, cb->mode, cb->flags,
- cb->sb_flags, cb->sb_status, cb->lkb_lksb,
+ dlm_run_callback(cb->our_nodeid, cb->ls_id, cb->lkb_id, cb->mode,
+ cb->flags, cb->sb_flags, cb->sb_status, cb->lkb_lksb,
cb->astfn, cb->bastfn, cb->astparam,
cb->res_name, cb->res_length);
dlm_free_cb(cb);
@@ -130,6 +131,7 @@ int dlm_get_cb(struct dlm_lkb *lkb, uint32_t flags, int mode,
return -ENOMEM;
/* for tracing */
+ (*cb)->our_nodeid = ls->ls_dn->our_node->id;
(*cb)->lkb_id = lkb->lkb_id;
(*cb)->ls_id = ls->ls_global_id;
memcpy((*cb)->res_name, rsb->res_name, rsb->res_length);
@@ -185,8 +187,9 @@ void dlm_add_cb(struct dlm_lkb *lkb, uint32_t flags, int mode, int status,
list_add(&cb->list, &ls->ls_cb_delay);
} else {
if (test_bit(LSFL_SOFTIRQ, &ls->ls_flags)) {
- dlm_run_callback(ls->ls_global_id, lkb->lkb_id, mode, flags,
- sbflags, status, lkb->lkb_lksb,
+ dlm_run_callback(ls->ls_dn->our_node->id,
+ ls->ls_global_id, lkb->lkb_id, mode,
+ flags, sbflags, status, lkb->lkb_lksb,
lkb->lkb_astfn, lkb->lkb_bastfn,
lkb->lkb_astparam, rsb->res_name,
rsb->res_length);
diff --git a/fs/dlm/dlm_internal.h b/fs/dlm/dlm_internal.h
index 2de5ef2653cd..bc3ff1b64e0c 100644
--- a/fs/dlm/dlm_internal.h
+++ b/fs/dlm/dlm_internal.h
@@ -234,6 +234,7 @@ struct dlm_callback {
bool copy_lvb;
struct dlm_lksb *lkb_lksb;
unsigned char lvbptr[DLM_USER_LVB_LEN];
+ int our_nodeid;
union {
void *astparam; /* caller's ast arg */
diff --git a/fs/dlm/user.c b/fs/dlm/user.c
index 1b682f8f95b6..c4d6e67ff63e 100644
--- a/fs/dlm/user.c
+++ b/fs/dlm/user.c
@@ -868,13 +868,14 @@ static ssize_t device_read(struct file *file, char __user *buf, size_t count,
spin_unlock_bh(&proc->asts_spin);
if (cb->flags & DLM_CB_BAST) {
- trace_dlm_bast(cb->ls_id, cb->lkb_id, cb->mode, cb->res_name,
- cb->res_length);
+ trace_dlm_bast(cb->our_nodeid, cb->ls_id, cb->lkb_id,
+ cb->mode, cb->res_name, cb->res_length);
} else if (cb->flags & DLM_CB_CAST) {
cb->lkb_lksb->sb_status = cb->sb_status;
cb->lkb_lksb->sb_flags = cb->sb_flags;
- trace_dlm_ast(cb->ls_id, cb->lkb_id, cb->sb_status,
- cb->sb_flags, cb->res_name, cb->res_length);
+ trace_dlm_ast(cb->our_nodeid, cb->ls_id, cb->lkb_id,
+ cb->sb_status, cb->sb_flags, cb->res_name,
+ cb->res_length);
}
ret = copy_result_to_user(&cb->ua,
diff --git a/include/trace/events/dlm.h b/include/trace/events/dlm.h
index af160082c9e3..2621bb7ac3a8 100644
--- a/include/trace/events/dlm.h
+++ b/include/trace/events/dlm.h
@@ -98,6 +98,7 @@ TRACE_EVENT(dlm_lock_start,
TP_ARGS(ls, lkb, name, namelen, mode, flags),
TP_STRUCT__entry(
+ __field(unsigned int, our_nodeid)
__field(__u32, ls_id)
__field(__u32, lkb_id)
__field(int, mode)
@@ -109,6 +110,7 @@ TRACE_EVENT(dlm_lock_start,
TP_fast_assign(
struct dlm_rsb *r;
+ __entry->our_nodeid = ls->ls_dn->our_node->id;
__entry->ls_id = ls->ls_global_id;
__entry->lkb_id = lkb->lkb_id;
__entry->mode = mode;
@@ -123,8 +125,8 @@ TRACE_EVENT(dlm_lock_start,
__get_dynamic_array_len(res_name));
),
- TP_printk("ls_id=%u lkb_id=%x mode=%s flags=%s res_name=%s",
- __entry->ls_id, __entry->lkb_id,
+ TP_printk("our_nodeid=%u ls_id=%u lkb_id=%x mode=%s flags=%s res_name=%s",
+ __entry->our_nodeid, __entry->ls_id, __entry->lkb_id,
show_lock_mode(__entry->mode),
show_lock_flags(__entry->flags),
__print_hex_str(__get_dynamic_array(res_name),
@@ -141,6 +143,7 @@ TRACE_EVENT(dlm_lock_end,
TP_ARGS(ls, lkb, name, namelen, mode, flags, error, kernel_lock),
TP_STRUCT__entry(
+ __field(unsigned int, our_nodeid)
__field(__u32, ls_id)
__field(__u32, lkb_id)
__field(int, mode)
@@ -153,6 +156,7 @@ TRACE_EVENT(dlm_lock_end,
TP_fast_assign(
struct dlm_rsb *r;
+ __entry->our_nodeid = ls->ls_dn->our_node->id;
__entry->ls_id = ls->ls_global_id;
__entry->lkb_id = lkb->lkb_id;
__entry->mode = mode;
@@ -178,8 +182,8 @@ TRACE_EVENT(dlm_lock_end,
),
- TP_printk("ls_id=%u lkb_id=%x mode=%s flags=%s error=%d res_name=%s",
- __entry->ls_id, __entry->lkb_id,
+ TP_printk("our_nodeid=%u ls_id=%u lkb_id=%x mode=%s flags=%s error=%d res_name=%s",
+ __entry->our_nodeid, __entry->ls_id, __entry->lkb_id,
show_lock_mode(__entry->mode),
show_lock_flags(__entry->flags), __entry->error,
__print_hex_str(__get_dynamic_array(res_name),
@@ -189,12 +193,13 @@ TRACE_EVENT(dlm_lock_end,
TRACE_EVENT(dlm_bast,
- TP_PROTO(__u32 ls_id, __u32 lkb_id, int mode,
+ TP_PROTO(unsigned int our_nodeid, __u32 ls_id, __u32 lkb_id, int mode,
const char *res_name, size_t res_length),
- TP_ARGS(ls_id, lkb_id, mode, res_name, res_length),
+ TP_ARGS(our_nodeid, ls_id, lkb_id, mode, res_name, res_length),
TP_STRUCT__entry(
+ __field(unsigned int, our_nodeid)
__field(__u32, ls_id)
__field(__u32, lkb_id)
__field(int, mode)
@@ -202,6 +207,7 @@ TRACE_EVENT(dlm_bast,
),
TP_fast_assign(
+ __entry->our_nodeid = our_nodeid;
__entry->ls_id = ls_id;
__entry->lkb_id = lkb_id;
__entry->mode = mode;
@@ -210,8 +216,8 @@ TRACE_EVENT(dlm_bast,
__get_dynamic_array_len(res_name));
),
- TP_printk("ls_id=%u lkb_id=%x mode=%s res_name=%s",
- __entry->ls_id, __entry->lkb_id,
+ TP_printk("our_nodeid=%u ls_id=%u lkb_id=%x mode=%s res_name=%s",
+ __entry->our_nodeid, __entry->ls_id, __entry->lkb_id,
show_lock_mode(__entry->mode),
__print_hex_str(__get_dynamic_array(res_name),
__get_dynamic_array_len(res_name)))
@@ -220,12 +226,15 @@ TRACE_EVENT(dlm_bast,
TRACE_EVENT(dlm_ast,
- TP_PROTO(__u32 ls_id, __u32 lkb_id, __u8 sb_flags, int sb_status,
- const char *res_name, size_t res_length),
+ TP_PROTO(unsigned int our_nodeid, __u32 ls_id, __u32 lkb_id,
+ __u8 sb_flags, int sb_status, const char *res_name,
+ size_t res_length),
- TP_ARGS(ls_id, lkb_id, sb_flags, sb_status, res_name, res_length),
+ TP_ARGS(our_nodeid, ls_id, lkb_id, sb_flags, sb_status, res_name,
+ res_length),
TP_STRUCT__entry(
+ __field(unsigned int, our_nodeid)
__field(__u32, ls_id)
__field(__u32, lkb_id)
__field(__u8, sb_flags)
@@ -234,6 +243,7 @@ TRACE_EVENT(dlm_ast,
),
TP_fast_assign(
+ __entry->our_nodeid = our_nodeid;
__entry->ls_id = ls_id;
__entry->lkb_id = lkb_id;
__entry->sb_flags = sb_flags;
@@ -243,8 +253,8 @@ TRACE_EVENT(dlm_ast,
__get_dynamic_array_len(res_name));
),
- TP_printk("ls_id=%u lkb_id=%x sb_flags=%s sb_status=%d res_name=%s",
- __entry->ls_id, __entry->lkb_id,
+ TP_printk("our_nodeid=%u ls_id=%u lkb_id=%x sb_flags=%s sb_status=%d res_name=%s",
+ __entry->our_nodeid, __entry->ls_id, __entry->lkb_id,
show_dlm_sb_flags(__entry->sb_flags), __entry->sb_status,
__print_hex_str(__get_dynamic_array(res_name),
__get_dynamic_array_len(res_name)))
--
2.43.0
Powered by blists - more mailing lists