[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] add support for protocol driver create_options
From: |
MORITA Kazutaka |
Subject: |
[Qemu-devel] Re: [PATCH] add support for protocol driver create_options |
Date: |
Mon, 24 May 2010 15:21:35 +0900 |
User-agent: |
Wanderlust/2.14.0 (Africa) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (Gojō) APEL/10.7 Emacs/22.3 (x86_64-pc-linux-gnu) MULE/5.0 (SAKAKI) |
At Fri, 21 May 2010 13:40:31 +0200,
Kevin Wolf wrote:
>
> Am 20.05.2010 07:36, schrieb MORITA Kazutaka:
> > This patch enables protocol drivers to use their create options which
> > are not supported by the format. For example, protcol drivers can use
> > a backing_file option with raw format.
> >
> > Signed-off-by: MORITA Kazutaka <address@hidden>
>
> Hm, this is not stackable, right? Though I do see that making it
> stackable would require some bigger changes, so maybe we can get away
> with claiming that this approach covers everything that happens in practice.
>
> If we accept that this is the desired behaviour, the code looks good to me.
>
As you say, this patch isn't stackable; we must specify a image name
with at most 1 format and 1 protocol. I cannot think of a situation
where we want to use more than one protocol to create qemu images, so
this seems to be enough to me.
Thanks,
Kazutaka