diff options
author | Joel Becker <joel.becker@oracle.com> | 2008-01-29 19:59:55 -0500 |
---|---|---|
committer | Mark Fasheh <mfasheh@suse.com> | 2008-04-18 11:56:04 -0400 |
commit | cf0acdcd640e9466059e69951c557e90b4bee45a (patch) | |
tree | c54bccbed204770005265176500ae0549d4ca31a /fs | |
parent | 1693a5c0117f8ccd010a666f97aaf0f14fb0a0e4 (diff) |
ocfs2: Abstract out a debugging function for underlying dlms.
dlmglue.c was still referencing a raw o2dlm lksb in one instance. Let's
create a generic ocfs2_dlm_dump_lksb() function. This allows underlying
DLMs to print whatever they want about their lock.
We then move the o2dlm dump into stackglue.c where it belongs.
Signed-off-by: Joel Becker <joel.becker@oracle.com>
Signed-off-by: Mark Fasheh <mfasheh@suse.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/ocfs2/dlmglue.c | 3 | ||||
-rw-r--r-- | fs/ocfs2/stackglue.c | 5 | ||||
-rw-r--r-- | fs/ocfs2/stackglue.h | 1 |
3 files changed, 7 insertions, 2 deletions
diff --git a/fs/ocfs2/dlmglue.c b/fs/ocfs2/dlmglue.c index b640423b936a..f41ff1c10ae8 100644 --- a/fs/ocfs2/dlmglue.c +++ b/fs/ocfs2/dlmglue.c | |||
@@ -2803,8 +2803,7 @@ static int ocfs2_drop_lock(struct ocfs2_super *osb, | |||
2803 | if (ret) { | 2803 | if (ret) { |
2804 | ocfs2_log_dlm_error("ocfs2_dlm_unlock", ret, lockres); | 2804 | ocfs2_log_dlm_error("ocfs2_dlm_unlock", ret, lockres); |
2805 | mlog(ML_ERROR, "lockres flags: %lu\n", lockres->l_flags); | 2805 | mlog(ML_ERROR, "lockres flags: %lu\n", lockres->l_flags); |
2806 | /* XXX Need to abstract this */ | 2806 | ocfs2_dlm_dump_lksb(&lockres->l_lksb); |
2807 | dlm_print_one_lock(lockres->l_lksb.lksb_o2dlm.lockid); | ||
2808 | BUG(); | 2807 | BUG(); |
2809 | } | 2808 | } |
2810 | mlog(0, "lock %s, successfull return from ocfs2_dlm_unlock\n", | 2809 | mlog(0, "lock %s, successfull return from ocfs2_dlm_unlock\n", |
diff --git a/fs/ocfs2/stackglue.c b/fs/ocfs2/stackglue.c index abdb9f6f4cc9..bd805411a856 100644 --- a/fs/ocfs2/stackglue.c +++ b/fs/ocfs2/stackglue.c | |||
@@ -252,6 +252,11 @@ void *ocfs2_dlm_lvb(union ocfs2_dlm_lksb *lksb) | |||
252 | return (void *)(lksb->lksb_o2dlm.lvb); | 252 | return (void *)(lksb->lksb_o2dlm.lvb); |
253 | } | 253 | } |
254 | 254 | ||
255 | void ocfs2_dlm_dump_lksb(union ocfs2_dlm_lksb *lksb) | ||
256 | { | ||
257 | dlm_print_one_lock(lksb->lksb_o2dlm.lockid); | ||
258 | } | ||
259 | |||
255 | /* | 260 | /* |
256 | * Called from the dlm when it's about to evict a node. This is how the | 261 | * Called from the dlm when it's about to evict a node. This is how the |
257 | * classic stack signals node death. | 262 | * classic stack signals node death. |
diff --git a/fs/ocfs2/stackglue.h b/fs/ocfs2/stackglue.h index 22af77b9a690..01e3c9b9192a 100644 --- a/fs/ocfs2/stackglue.h +++ b/fs/ocfs2/stackglue.h | |||
@@ -91,6 +91,7 @@ int ocfs2_dlm_unlock(struct ocfs2_cluster_connection *conn, | |||
91 | 91 | ||
92 | int ocfs2_dlm_lock_status(union ocfs2_dlm_lksb *lksb); | 92 | int ocfs2_dlm_lock_status(union ocfs2_dlm_lksb *lksb); |
93 | void *ocfs2_dlm_lvb(union ocfs2_dlm_lksb *lksb); | 93 | void *ocfs2_dlm_lvb(union ocfs2_dlm_lksb *lksb); |
94 | void ocfs2_dlm_dump_lksb(union ocfs2_dlm_lksb *lksb); | ||
94 | 95 | ||
95 | void o2cb_get_stack(struct ocfs2_locking_protocol *proto); | 96 | void o2cb_get_stack(struct ocfs2_locking_protocol *proto); |
96 | void o2cb_put_stack(void); | 97 | void o2cb_put_stack(void); |