diff options
| author | Petr Mladek <pmladek@suse.com> | 2015-10-05 15:39:52 +0300 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.cz> | 2015-11-06 17:06:05 +0300 |
| commit | 8d090f47315507c3064ca4eefa9a1da52390b52e (patch) | |
| tree | 74d7c641433c927b6de427bbc0fa7eb7518a4d10 | |
| parent | 0c6cac7ab44435d8837931a561cc4d58530cc032 (diff) | |
| download | linux-8d090f47315507c3064ca4eefa9a1da52390b52e.tar.xz | |
bcache: Really show state of work pending bit
WORK_STRUCT_PENDING is a mask for testing the pending bit.
test_bit() expects the number of the bit and we need to
use WORK_STRUCT_PENDING_BIT there.
Also work_data_bits() is defined in workqueues.h now.
I have noticed this just by chance when looking how
WORK_STRUCT_PENDING_BIT is used. The change is compile
tested.
Signed-off-by: Petr Mladek <pmladek@suse.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
| -rw-r--r-- | drivers/md/bcache/closure.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/md/bcache/closure.c b/drivers/md/bcache/closure.c index 7a228de95fd7..9eaf1d6e8302 100644 --- a/drivers/md/bcache/closure.c +++ b/drivers/md/bcache/closure.c @@ -167,8 +167,6 @@ EXPORT_SYMBOL(closure_debug_destroy); static struct dentry *debug; -#define work_data_bits(work) ((unsigned long *)(&(work)->data)) - static int debug_seq_show(struct seq_file *f, void *data) { struct closure *cl; @@ -182,7 +180,7 @@ static int debug_seq_show(struct seq_file *f, void *data) r & CLOSURE_REMAINING_MASK); seq_printf(f, "%s%s%s%s\n", - test_bit(WORK_STRUCT_PENDING, + test_bit(WORK_STRUCT_PENDING_BIT, work_data_bits(&cl->work)) ? "Q" : "", r & CLOSURE_RUNNING ? "R" : "", r & CLOSURE_STACK ? "S" : "", |