aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/md/dm-snap.c
diff options
context:
space:
mode:
authorMikulas Patocka <mpatocka@redhat.com>2009-12-10 18:52:33 -0500
committerAlasdair G Kergon <agk@redhat.com>2009-12-10 18:52:33 -0500
commit17aa03326d40614db94bc51fbbc92df628a5c97c (patch)
treea9661a11274731b6b8e2fcbaace7d4424b9cefb6 /drivers/md/dm-snap.c
parent9fe862548821b0c206c58e8057b782530a173703 (diff)
dm snapshot: delay merging a chunk until writes to it complete
Track writes to chunks that are currently being merged and delay merging a chunk until all writes to that chunk finish. Signed-off-by: Mikulas Patocka <mpatocka@redhat.com> Reviewed-by: Mike Snitzer <snitzer@redhat.com> Signed-off-by: Alasdair G Kergon <agk@redhat.com>
Diffstat (limited to 'drivers/md/dm-snap.c')
-rw-r--r--drivers/md/dm-snap.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/md/dm-snap.c b/drivers/md/dm-snap.c
index 91a47c522b09..bc52776c69cc 100644
--- a/drivers/md/dm-snap.c
+++ b/drivers/md/dm-snap.c
@@ -892,7 +892,7 @@ static void snapshot_merge_next_chunks(struct dm_snapshot *s)
892 s->num_merging_chunks = 1; 892 s->num_merging_chunks = 1;
893 up_write(&s->lock); 893 up_write(&s->lock);
894 894
895 /* !!! FIXME: wait until writes to this chunk drain */ 895 __check_for_conflicting_io(s, old_chunk);
896 896
897 dm_kcopyd_copy(s->kcopyd_client, &src, 1, &dest, 0, merge_callback, s); 897 dm_kcopyd_copy(s->kcopyd_client, &src, 1, &dest, 0, merge_callback, s);
898 return; 898 return;
@@ -1635,7 +1635,11 @@ static int snapshot_merge_map(struct dm_target *ti, struct bio *bio,
1635 r = DM_MAPIO_SUBMITTED; 1635 r = DM_MAPIO_SUBMITTED;
1636 goto out_unlock; 1636 goto out_unlock;
1637 } 1637 }
1638
1638 remap_exception(s, e, bio, chunk); 1639 remap_exception(s, e, bio, chunk);
1640
1641 if (bio_rw(bio) == WRITE)
1642 map_context->ptr = track_chunk(s, chunk);
1639 goto out_unlock; 1643 goto out_unlock;
1640 } 1644 }
1641 1645