Commit fe630de0 authored by Vitaly Mayatskikh's avatar Vitaly Mayatskikh Committed by Song Liu

md/raid10: avoid deadlock on recovery.

When disk failure happens and the array has a spare drive, resync thread
kicks in and starts to refill the spare. However it may get blocked by
a retry thread that resubmits failed IO to a mirror and itself can get
blocked on a barrier raised by the resync thread.
Acked-by: default avatarNigel Croxon <ncroxon@redhat.com>
Signed-off-by: default avatarVitaly Mayatskikh <vmayatskikh@digitalocean.com>
Signed-off-by: default avatarSong Liu <songliubraving@fb.com>
parent c333f949
...@@ -980,6 +980,7 @@ static void wait_barrier(struct r10conf *conf) ...@@ -980,6 +980,7 @@ static void wait_barrier(struct r10conf *conf)
{ {
spin_lock_irq(&conf->resync_lock); spin_lock_irq(&conf->resync_lock);
if (conf->barrier) { if (conf->barrier) {
struct bio_list *bio_list = current->bio_list;
conf->nr_waiting++; conf->nr_waiting++;
/* Wait for the barrier to drop. /* Wait for the barrier to drop.
* However if there are already pending * However if there are already pending
...@@ -994,9 +995,16 @@ static void wait_barrier(struct r10conf *conf) ...@@ -994,9 +995,16 @@ static void wait_barrier(struct r10conf *conf)
wait_event_lock_irq(conf->wait_barrier, wait_event_lock_irq(conf->wait_barrier,
!conf->barrier || !conf->barrier ||
(atomic_read(&conf->nr_pending) && (atomic_read(&conf->nr_pending) &&
current->bio_list && bio_list &&
(!bio_list_empty(&current->bio_list[0]) || (!bio_list_empty(&bio_list[0]) ||
!bio_list_empty(&current->bio_list[1]))), !bio_list_empty(&bio_list[1]))) ||
/* move on if recovery thread is
* blocked by us
*/
(conf->mddev->thread->tsk == current &&
test_bit(MD_RECOVERY_RUNNING,
&conf->mddev->recovery) &&
conf->nr_queued > 0),
conf->resync_lock); conf->resync_lock);
conf->nr_waiting--; conf->nr_waiting--;
if (!conf->nr_waiting) if (!conf->nr_waiting)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment