diff options
author | Mikulas Patocka <mpatocka@redhat.com> | 2009-12-10 18:52:33 -0500 |
---|---|---|
committer | Alasdair G Kergon <agk@redhat.com> | 2009-12-10 18:52:33 -0500 |
commit | 17aa03326d40614db94bc51fbbc92df628a5c97c (patch) | |
tree | a9661a11274731b6b8e2fcbaace7d4424b9cefb6 /drivers | |
parent | 9fe862548821b0c206c58e8057b782530a173703 (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')
-rw-r--r-- | drivers/md/dm-snap.c | 6 |
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 | ||