[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/5] blockdev: add discard suboption to -drive
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 2/5] blockdev: add discard suboption to -drive |
Date: |
Thu, 21 Feb 2013 10:01:30 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Feb 21, 2013 at 09:48:56AM +0100, Stefan Hajnoczi wrote:
> On Wed, Feb 20, 2013 at 03:09:51PM +0100, Paolo Bonzini wrote:
> > Il 13/02/2013 11:39, Stefan Hajnoczi ha scritto:
> > >> > .type = QEMU_OPT_STRING,
> > >> > .help = "disk image",
> > >> > },{
> > >> > + .name = "discard",
> > >> > + .type = QEMU_OPT_STRING,
> > >> > + .help = "discard operation (ignore/off, unmap/on)",
> > > off/on aliases will be confusing when we plan to add "anchor" in the
> > > future, making this non-boolean. Please support ignore/unmap only.
> >
> > My problem here is that "unmap" will not be the default but at the same
> > time it will be what people want to use. Hence, it is important to have
> > an easy to remember and discover option; "discard=on" is immediately
> > understandable, while "discard=unmap" is not.
>
> I understand. The risk is that we end up confusing users rather than
> helping, because the actual ignore/anchor/unmap modes need to be
> understood to use this feature properly.
>
> If you and Kevin like the off/on alias, go ahead. It's not critical.
I actually think both are good points and am not sure which one is
more important. In such cases I tend to just accept whatever a proposed
patch contains.
Kevin
[Qemu-devel] [PATCH 3/5] qemu-nbd: add --discard option, Paolo Bonzini, 2013/02/08
[Qemu-devel] [PATCH 5/5] pc: add compatibility machine types for 1.4, Paolo Bonzini, 2013/02/08
[Qemu-devel] [PATCH 4/5] blockdev: enable discard by default, Paolo Bonzini, 2013/02/08
Re: [Qemu-devel] [RFC PATCH 0/5] Make discard visible to the guests, Stefan Hajnoczi, 2013/02/13