diff options
| author |   Petr Mladek <pmladek@suse.com> | 2015-10-05 14:39:52 +0200 |
|---|---|---|
| committer |   Jiri Kosina <jkosina@suse.cz> | 2015-11-06 15:06:05 +0100 |
| commit | 8d090f47315507c3064ca4eefa9a1da52390b52e (patch) | |
| tree | 74d7c641433c927b6de427bbc0fa7eb7518a4d10 /drivers/md | |
| parent | hwmon: applesmc: fix comment typos (diff) | |
| download | linux-dev-8d090f47315507c3064ca4eefa9a1da52390b52e.tar.xz linux-dev-8d090f47315507c3064ca4eefa9a1da52390b52e.zip | |
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>
Diffstat (limited to 'drivers/md')
| -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" : "", |