Commit d72b6cec authored by Trond Myklebust's avatar Trond Myklebust

SUNRPC: Remove the 'tk_magic' debugging field

It has not triggered in almost a decade. Time to get rid of it...
Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
parent d60dbb20
...@@ -41,9 +41,6 @@ struct rpc_wait { ...@@ -41,9 +41,6 @@ struct rpc_wait {
* This is the RPC task struct * This is the RPC task struct
*/ */
struct rpc_task { struct rpc_task {
#ifdef RPC_DEBUG
unsigned long tk_magic; /* 0xf00baa */
#endif
atomic_t tk_count; /* Reference count */ atomic_t tk_count; /* Reference count */
struct list_head tk_task; /* global list of tasks */ struct list_head tk_task; /* global list of tasks */
struct rpc_clnt * tk_client; /* RPC client */ struct rpc_clnt * tk_client; /* RPC client */
......
...@@ -25,7 +25,6 @@ ...@@ -25,7 +25,6 @@
#ifdef RPC_DEBUG #ifdef RPC_DEBUG
#define RPCDBG_FACILITY RPCDBG_SCHED #define RPCDBG_FACILITY RPCDBG_SCHED
#define RPC_TASK_MAGIC_ID 0xf00baa
#endif #endif
/* /*
...@@ -237,7 +236,6 @@ static void rpc_task_set_debuginfo(struct rpc_task *task) ...@@ -237,7 +236,6 @@ static void rpc_task_set_debuginfo(struct rpc_task *task)
{ {
static atomic_t rpc_pid; static atomic_t rpc_pid;
task->tk_magic = RPC_TASK_MAGIC_ID;
task->tk_pid = atomic_inc_return(&rpc_pid); task->tk_pid = atomic_inc_return(&rpc_pid);
} }
#else #else
...@@ -360,9 +358,6 @@ static void __rpc_do_wake_up_task(struct rpc_wait_queue *queue, struct rpc_task ...@@ -360,9 +358,6 @@ static void __rpc_do_wake_up_task(struct rpc_wait_queue *queue, struct rpc_task
dprintk("RPC: %5u __rpc_wake_up_task (now %lu)\n", dprintk("RPC: %5u __rpc_wake_up_task (now %lu)\n",
task->tk_pid, jiffies); task->tk_pid, jiffies);
#ifdef RPC_DEBUG
BUG_ON(task->tk_magic != RPC_TASK_MAGIC_ID);
#endif
/* Has the task been executed yet? If not, we cannot wake it up! */ /* Has the task been executed yet? If not, we cannot wake it up! */
if (!RPC_IS_ACTIVATED(task)) { if (!RPC_IS_ACTIVATED(task)) {
printk(KERN_ERR "RPC: Inactive task (%p) being woken up!\n", task); printk(KERN_ERR "RPC: Inactive task (%p) being woken up!\n", task);
...@@ -916,9 +911,6 @@ EXPORT_SYMBOL_GPL(rpc_put_task); ...@@ -916,9 +911,6 @@ EXPORT_SYMBOL_GPL(rpc_put_task);
static void rpc_release_task(struct rpc_task *task) static void rpc_release_task(struct rpc_task *task)
{ {
#ifdef RPC_DEBUG
BUG_ON(task->tk_magic != RPC_TASK_MAGIC_ID);
#endif
dprintk("RPC: %5u release task\n", task->tk_pid); dprintk("RPC: %5u release task\n", task->tk_pid);
if (!list_empty(&task->tk_task)) { if (!list_empty(&task->tk_task)) {
...@@ -930,9 +922,6 @@ static void rpc_release_task(struct rpc_task *task) ...@@ -930,9 +922,6 @@ static void rpc_release_task(struct rpc_task *task)
} }
BUG_ON (RPC_IS_QUEUED(task)); BUG_ON (RPC_IS_QUEUED(task));
#ifdef RPC_DEBUG
task->tk_magic = 0;
#endif
/* Wake up anyone who is waiting for task completion */ /* Wake up anyone who is waiting for task completion */
rpc_mark_complete_task(task); rpc_mark_complete_task(task);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment