diff options
Diffstat (limited to 'drivers/md/dm-raid1.c')
-rw-r--r-- | drivers/md/dm-raid1.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/drivers/md/dm-raid1.c b/drivers/md/dm-raid1.c index b08df8b9b2ca..863282513753 100644 --- a/drivers/md/dm-raid1.c +++ b/drivers/md/dm-raid1.c | |||
@@ -375,16 +375,18 @@ static void rh_inc(struct region_hash *rh, region_t region) | |||
375 | 375 | ||
376 | read_lock(&rh->hash_lock); | 376 | read_lock(&rh->hash_lock); |
377 | reg = __rh_find(rh, region); | 377 | reg = __rh_find(rh, region); |
378 | |||
379 | atomic_inc(®->pending); | ||
380 | |||
381 | spin_lock_irq(&rh->region_lock); | ||
378 | if (reg->state == RH_CLEAN) { | 382 | if (reg->state == RH_CLEAN) { |
379 | rh->log->type->mark_region(rh->log, reg->key); | 383 | rh->log->type->mark_region(rh->log, reg->key); |
380 | 384 | ||
381 | spin_lock_irq(&rh->region_lock); | ||
382 | reg->state = RH_DIRTY; | 385 | reg->state = RH_DIRTY; |
383 | list_del_init(®->list); /* take off the clean list */ | 386 | list_del_init(®->list); /* take off the clean list */ |
384 | spin_unlock_irq(&rh->region_lock); | ||
385 | } | 387 | } |
388 | spin_unlock_irq(&rh->region_lock); | ||
386 | 389 | ||
387 | atomic_inc(®->pending); | ||
388 | read_unlock(&rh->hash_lock); | 390 | read_unlock(&rh->hash_lock); |
389 | } | 391 | } |
390 | 392 | ||
@@ -408,6 +410,10 @@ static void rh_dec(struct region_hash *rh, region_t region) | |||
408 | 410 | ||
409 | if (atomic_dec_and_test(®->pending)) { | 411 | if (atomic_dec_and_test(®->pending)) { |
410 | spin_lock_irqsave(&rh->region_lock, flags); | 412 | spin_lock_irqsave(&rh->region_lock, flags); |
413 | if (atomic_read(®->pending)) { /* check race */ | ||
414 | spin_unlock_irqrestore(&rh->region_lock, flags); | ||
415 | return; | ||
416 | } | ||
411 | if (reg->state == RH_RECOVERING) { | 417 | if (reg->state == RH_RECOVERING) { |
412 | list_add_tail(®->list, &rh->quiesced_regions); | 418 | list_add_tail(®->list, &rh->quiesced_regions); |
413 | } else { | 419 | } else { |