On Mon, 25 Feb 2013 09:11:02 -0700 Tregaron Bayly <tbayly@xxxxxxxxxxxx> wrote: > > Actually don't bother. I think I've found the problem. It is related to > > pending_count and is easy to fix. > > Could you try this patch please? > > > > Thanks. > > NeilBrown > > > > diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c > > index 6e5d5a5..fd86b37 100644 > > --- a/drivers/md/raid1.c > > +++ b/drivers/md/raid1.c > > @@ -967,6 +967,7 @@ static void raid1_unplug(struct blk_plug_cb *cb, bool from_schedule) > > bio_list_merge(&conf->pending_bio_list, &plug->pending); > > conf->pending_count += plug->pending_cnt; > > spin_unlock_irq(&conf->device_lock); > > + wake_up(&conf->wait_barrier); > > md_wakeup_thread(mddev->thread); > > kfree(plug); > > return; > > Running 15 hours now and no sign of the problem, which is 12 hours > longer than it took to trigger the bug in the past. I'll continue > testing to be sure but I think this patch is a fix. Thanks. I've added a "Tested-by" you to the patch header. Should go to Linus, and thence to -stable in a day or 2. NeilBrown > > Thanks for the fast response! > > Tregaron Bayly
Attachment:
signature.asc
Description: PGP signature
