On Tue 29-09-15 12:47:51, Tejun Heo wrote: > wakeup_dirtytime_writeback() walks and wakes up all wb's of all bdi's; > unfortunately, it was always waking up bdi->wb instead of the wb being > walked. Fix it. Looks good. You can add: Reviewed-by: Jan Kara <jack@xxxxxxxx> Honza > > Signed-off-by: Tejun Heo <tj@xxxxxxxxxx> > Fixes: 001fe6f617b1 ("writeback: make wakeup_dirtytime_writeback() handle multiple bdi_writeback's") > --- > fs/fs-writeback.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c > index 091a364..d0da306 100644 > --- a/fs/fs-writeback.c > +++ b/fs/fs-writeback.c > @@ -1897,8 +1897,8 @@ static void wakeup_dirtytime_writeback(struct work_struct *w) > struct wb_iter iter; > > bdi_for_each_wb(wb, bdi, &iter, 0) > - if (!list_empty(&bdi->wb.b_dirty_time)) > - wb_wakeup(&bdi->wb); > + if (!list_empty(&wb->b_dirty_time)) > + wb_wakeup(wb); > } > rcu_read_unlock(); > schedule_delayed_work(&dirtytime_work, dirtytime_expire_interval * HZ); > -- > 2.4.3 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html