diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-05-13 12:51:50 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-05-14 15:09:36 -0400 |
commit | d72b6cec8d42eb7c2a249b613abf2c2b7a6eeb47 (patch) | |
tree | c577536a08739621dfbde8c1d30eb88ba8414464 | |
parent | d60dbb20a74c2cfa142be0a34dac3c6547ea086c (diff) |
SUNRPC: Remove the 'tk_magic' debugging field
It has not triggered in almost a decade. Time to get rid of it...
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
-rw-r--r-- | include/linux/sunrpc/sched.h | 3 | ||||
-rw-r--r-- | net/sunrpc/sched.c | 11 |
2 files changed, 0 insertions, 14 deletions
diff --git a/include/linux/sunrpc/sched.h b/include/linux/sunrpc/sched.h index 46ebef1788c6..41b9f971c9fb 100644 --- a/include/linux/sunrpc/sched.h +++ b/include/linux/sunrpc/sched.h | |||
@@ -41,9 +41,6 @@ struct rpc_wait { | |||
41 | * This is the RPC task struct | 41 | * This is the RPC task struct |
42 | */ | 42 | */ |
43 | struct rpc_task { | 43 | struct rpc_task { |
44 | #ifdef RPC_DEBUG | ||
45 | unsigned long tk_magic; /* 0xf00baa */ | ||
46 | #endif | ||
47 | atomic_t tk_count; /* Reference count */ | 44 | atomic_t tk_count; /* Reference count */ |
48 | struct list_head tk_task; /* global list of tasks */ | 45 | struct list_head tk_task; /* global list of tasks */ |
49 | struct rpc_clnt * tk_client; /* RPC client */ | 46 | struct rpc_clnt * tk_client; /* RPC client */ |
diff --git a/net/sunrpc/sched.c b/net/sunrpc/sched.c index aa7b07ef5d55..4a843b883b89 100644 --- a/net/sunrpc/sched.c +++ b/net/sunrpc/sched.c | |||
@@ -25,7 +25,6 @@ | |||
25 | 25 | ||
26 | #ifdef RPC_DEBUG | 26 | #ifdef RPC_DEBUG |
27 | #define RPCDBG_FACILITY RPCDBG_SCHED | 27 | #define RPCDBG_FACILITY RPCDBG_SCHED |
28 | #define RPC_TASK_MAGIC_ID 0xf00baa | ||
29 | #endif | 28 | #endif |
30 | 29 | ||
31 | /* | 30 | /* |
@@ -237,7 +236,6 @@ static void rpc_task_set_debuginfo(struct rpc_task *task) | |||
237 | { | 236 | { |
238 | static atomic_t rpc_pid; | 237 | static atomic_t rpc_pid; |
239 | 238 | ||
240 | task->tk_magic = RPC_TASK_MAGIC_ID; | ||
241 | task->tk_pid = atomic_inc_return(&rpc_pid); | 239 | task->tk_pid = atomic_inc_return(&rpc_pid); |
242 | } | 240 | } |
243 | #else | 241 | #else |
@@ -360,9 +358,6 @@ static void __rpc_do_wake_up_task(struct rpc_wait_queue *queue, struct rpc_task | |||
360 | dprintk("RPC: %5u __rpc_wake_up_task (now %lu)\n", | 358 | dprintk("RPC: %5u __rpc_wake_up_task (now %lu)\n", |
361 | task->tk_pid, jiffies); | 359 | task->tk_pid, jiffies); |
362 | 360 | ||
363 | #ifdef RPC_DEBUG | ||
364 | BUG_ON(task->tk_magic != RPC_TASK_MAGIC_ID); | ||
365 | #endif | ||
366 | /* Has the task been executed yet? If not, we cannot wake it up! */ | 361 | /* Has the task been executed yet? If not, we cannot wake it up! */ |
367 | if (!RPC_IS_ACTIVATED(task)) { | 362 | if (!RPC_IS_ACTIVATED(task)) { |
368 | printk(KERN_ERR "RPC: Inactive task (%p) being woken up!\n", task); | 363 | printk(KERN_ERR "RPC: Inactive task (%p) being woken up!\n", task); |
@@ -916,9 +911,6 @@ EXPORT_SYMBOL_GPL(rpc_put_task); | |||
916 | 911 | ||
917 | static void rpc_release_task(struct rpc_task *task) | 912 | static void rpc_release_task(struct rpc_task *task) |
918 | { | 913 | { |
919 | #ifdef RPC_DEBUG | ||
920 | BUG_ON(task->tk_magic != RPC_TASK_MAGIC_ID); | ||
921 | #endif | ||
922 | dprintk("RPC: %5u release task\n", task->tk_pid); | 914 | dprintk("RPC: %5u release task\n", task->tk_pid); |
923 | 915 | ||
924 | if (!list_empty(&task->tk_task)) { | 916 | if (!list_empty(&task->tk_task)) { |
@@ -930,9 +922,6 @@ static void rpc_release_task(struct rpc_task *task) | |||
930 | } | 922 | } |
931 | BUG_ON (RPC_IS_QUEUED(task)); | 923 | BUG_ON (RPC_IS_QUEUED(task)); |
932 | 924 | ||
933 | #ifdef RPC_DEBUG | ||
934 | task->tk_magic = 0; | ||
935 | #endif | ||
936 | /* Wake up anyone who is waiting for task completion */ | 925 | /* Wake up anyone who is waiting for task completion */ |
937 | rpc_mark_complete_task(task); | 926 | rpc_mark_complete_task(task); |
938 | 927 | ||