NeilBrown <neilb@xxxxxxx> wrote: > > + wait_event_lock_irq(conf->wait_for_stripe, > + !list_empty(&conf->inactive_list), > + conf->device_lock, > + unplug_slaves(conf->mddev); > + ); Boy, that's an ugly-looking thing, isn't it? __wait_event_lock_irq() already puts a semicolon after `cmd' so I think the one here isn't needed, which would make it a bit less of a surprise to look at. - To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
