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 { |
