From patchwork Tue Oct 20 22:46:57 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mike Snitzer X-Patchwork-Id: 55028 Received: from hormel.redhat.com (hormel1.redhat.com [209.132.177.33]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n9KMlJvb021646 for ; Tue, 20 Oct 2009 22:47:19 GMT Received: from listman.util.phx.redhat.com (listman.util.phx.redhat.com [10.8.4.110]) by hormel.redhat.com (Postfix) with ESMTP id C2033619BA8; Tue, 20 Oct 2009 18:47:18 -0400 (EDT) Received: from int-mx04.intmail.prod.int.phx2.redhat.com (nat-pool.util.phx.redhat.com [10.8.5.200]) by listman.util.phx.redhat.com (8.13.1/8.13.1) with ESMTP id n9KMlCx0017701 for ; Tue, 20 Oct 2009 18:47:12 -0400 Received: from localhost (dhcp-100-19-150.bos.redhat.com [10.16.19.150]) by int-mx04.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id n9KMlBWj019056; Tue, 20 Oct 2009 18:47:12 -0400 From: Mike Snitzer To: dm-devel@redhat.com Date: Tue, 20 Oct 2009 18:46:57 -0400 Message-Id: <1256078825-11331-10-git-send-email-snitzer@redhat.com> In-Reply-To: <1256078825-11331-1-git-send-email-snitzer@redhat.com> References: <1256078825-11331-1-git-send-email-snitzer@redhat.com> X-Scanned-By: MIMEDefang 2.67 on 10.5.11.17 X-loop: dm-devel@redhat.com Cc: Mikulas Patocka Subject: [dm-devel] [PATCH v2 09/17] dm snapshot: merge target should not allocate new exceptions X-BeenThere: dm-devel@redhat.com X-Mailman-Version: 2.1.5 Precedence: junk Reply-To: device-mapper development List-Id: device-mapper development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com diff --git a/drivers/md/dm-snap.c b/drivers/md/dm-snap.c index d83a26e..ea3ff81 100644 --- a/drivers/md/dm-snap.c +++ b/drivers/md/dm-snap.c @@ -147,6 +147,11 @@ static int bdev_equal(struct block_device *lhs, struct block_device *rhs) return lhs == rhs; } +static int snapshot_merge_map(struct dm_target *ti, struct bio *bio, + union map_info *map_context); + +#define is_merge(ti) ((ti)->type->map == snapshot_merge_map) + struct dm_snap_pending_exception { struct dm_exception e; @@ -1239,6 +1244,40 @@ static int snapshot_map(struct dm_target *ti, struct bio *bio, return r; } +static int snapshot_merge_map(struct dm_target *ti, struct bio *bio, + union map_info *map_context) +{ + struct dm_exception *e; + struct dm_snapshot *s = ti->private; + int r = DM_MAPIO_REMAPPED; + chunk_t chunk; + + chunk = sector_to_chunk(s->store, bio->bi_sector); + + down_read(&s->lock); + + /* Full snapshots are not usable */ + if (!s->valid) { + r = -EIO; + goto out_unlock; + } + + /* If the block is already remapped - use that */ + e = dm_lookup_exception(&s->complete, chunk); + if (e) { + remap_exception(s, e, bio, chunk); + goto out_unlock; + } + + bio->bi_bdev = s->origin->bdev; + + out_unlock: + up_read(&s->lock); + + return r; +} + + static int snapshot_end_io(struct dm_target *ti, struct bio *bio, int error, union map_info *map_context) { @@ -1358,6 +1397,11 @@ static int __origin_write(struct list_head *snapshots, /* Do all the snapshots on this origin */ list_for_each_entry (snap, snapshots, list) { + /* If the snapshot is merging, don't make new exceptions in it + - but in this case no one should be writing to the origin */ + if (is_merge(snap->ti)) + continue; + down_write(&snap->lock); /* Only deal with valid and active snapshots */ @@ -1585,7 +1629,7 @@ static struct target_type merge_target = { .module = THIS_MODULE, .ctr = snapshot_ctr, .dtr = snapshot_dtr, - .map = snapshot_map, + .map = snapshot_merge_map, .end_io = snapshot_end_io, .resume = snapshot_resume, .status = snapshot_status,