lkml.org 
[lkml]   [2009]   [Nov]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v4 06/13] dm snapshot: merge target should not allocate new exceptions
On Fri, Nov 20 2009 at  3:27pm -0500,
Mike Snitzer <snitzer@redhat.com> wrote:

> If a write request to a merging snapshot device is to be dispatched
> directly to the origin (because the chunk is not remapped or was already
> merged), snapshot_merge_map() must make exceptions in other snapshots.
...
> +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;
> +
> + if (bio_rw(bio) == WRITE) {
> + up_write(&s->lock);
> + return do_origin(s->origin, bio);
> + }

OK, I caught this in review but clearly didn't refresh the patch; the
above up_write() should be up_read().

Mike


\
 
 \ /
  Last update: 2009-11-20 21:57    [W:0.092 / U:0.356 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site