All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Haripriya S" <sharipriya@novell.com>
To: dm-devel@redhat.com
Subject: Re: dm-snapshot scalability - chained delta snapshots approach
Date: Wed, 25 Oct 2006 06:09:22 -0600	[thread overview]
Message-ID: <453FA29D.A648.00B6.0@novell.com> (raw)
In-Reply-To: <20061024095200.GE4729@hasse.suse.de>

  
>>> Jan Blunck <jblunck@suse.de> 10/24/06 3:22 PM >>> 
On Mon, Oct 23, Molle Bestefich wrote:

> > >Although that includes a complete redesign of the exception store
code.
> > 
> > Especially when you say stuff like that :- ).
> > 

> The chained- snapshots approach needs that too.

 In the chained snapshots, the only addition is a way to tell if an
exception 
 is to be preserved or can be written over. So for every disk-exception
an 
additional field is required (Alasdair also recently suggested that we
could use 
a bitmap to save space). So I would say this is not a major
re-architecture of 
the disk exception structures but a simple (but incompatible) format
change.

> > >The throughput issues should be addressed by only
> > >writing to one exception store.
> > 
> > Wouldn't this make debugging more complex, and further add to
> > the difficulty of snapshot resizing?

> Resizing? Nope, you only need to resize the exception store thats it.
Resizing
> the chained- snapshots approach is complex however: in the worst case
you have
> to move the exception stores to get enough free space.

I agree that there is work to be done while resizing. It seems simple
to code 
though and can be done similar to a snapshot delete. If a snapshot is
being 
resized, and will lose exception data, then we need to move the
exception 
and data to the first snapshot after this snapshot in the write chain
which has 
space to hold that data. Yes, data move is involved here. btw I
couldn't figure out 
how resize will work with the common exception store approach. Can you

please explain that in detail ?

Thanks and Regards,
Haripriya

  reply	other threads:[~2006-10-25 12:09 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <44DA246B020000B60000BD22@lucius.provo.novell.com>
     [not found] ` <44DB5BDD020000B60000BD96@lucius.provo.novell.com>
2006-08-10 10:46   ` snapshot scalability Haripriya S
2006-09-27  5:24     ` dm-snapshot scalability - chained delta snapshots approach Haripriya S
2006-09-27  9:17       ` Jan Blunck
2006-09-27 10:40         ` rgammans
2006-09-27 14:47           ` Bill Rugolsky Jr.
2006-10-23 17:16         ` Molle Bestefich
2006-10-24  9:52           ` Jan Blunck
2006-10-25 12:09             ` Haripriya S [this message]
2006-10-26  8:29           ` Haripriya S

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=453FA29D.A648.00B6.0@novell.com \
    --to=sharipriya@novell.com \
    --cc=dm-devel@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.