qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 0/4] block: add optional 'speed' parameter to bl


From: Luiz Capitulino
Subject: Re: [Qemu-devel] [PATCH 0/4] block: add optional 'speed' parameter to block-stream
Date: Mon, 23 Apr 2012 17:46:20 -0300

On Mon, 23 Apr 2012 14:44:27 -0600
Eric Blake <address@hidden> wrote:

> On 04/23/2012 01:38 PM, Anthony Liguori wrote:
> >>> I like the approach of adding a new optional parameter; that is a
> >>> workable solution for libvirt (that is, libvirt can assume that if the
> >>> spelling 'block-stream' is available, then so is the optional 'speed'
> >>> parameter, if this patch is included in time for qemu 1.1).
> >>
> >> The optional parameter is fine for me too, but we recently had a
> >> discussion where it was decided that we should add new commands instead
> >> of extending existing ones.
> >>
> >> Anthony, I believe we're already taking that position, right?
> > 
> > Correct.  It's impossible to tell whether a given version of QEMU
> > supports extra parameters if we just add them to existing commands.
> 
> Thankfully, block-stream is not an existing command, and we've already
> created block-stream for qemu 1.1 as the new command that is explicitly
> different from the block_stream of RHEL 6.2.  Now is the time to get
> this fix in, before we are stuck with bad semantics.

Right, Paolo clarified this as well.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]