Re: [commit cbf76b702] bcache: Really show state of work pending bit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 27 Feb 2016, Greg KH wrote:
> On Fri, Feb 26, 2016 at 05:34:19PM -0800, Eric Wheeler wrote:
> > To whom it may concern:
> > 
> > Please flag commit cbf76b702 for inclusion into stable branches.  It 
> > applies cleanly in 4.1 (and likely 3.18).  We have been testing it for a 
> > while.
> > 
> > git diff --stat cbf76b702~1..cbf76b702
> >  drivers/md/bcache/closure.c | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> > 
> > Thank you!
> 
> $ git show cbf76b702
> fatal: ambiguous argument 'cbf76b702': unknown revision or path not in the working tree.
> 
> You are going to have to be a bit more specific...

This one:


commit cbf76b702e85802683c166ad9067874bc3e1e843
Author: Petr Mladek <pmladek@xxxxxxxx>
Date:   Mon Oct 5 14:39:52 2015 +0200

    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@xxxxxxxx>
    Signed-off-by: Jiri Kosina <jkosina@xxxxxxx>

diff --git a/drivers/md/bcache/closure.c b/drivers/md/bcache/closure.c
index 7a228de..9eaf1d6 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" : "",

--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]