[libvirt] [PATCH] qemu: fix my typo at commit 74e034964c32edb1732d0ff7642f3977f3587d72

Eric Blake eblake at redhat.com
Fri Feb 3 16:35:40 UTC 2012

On 02/03/2012 09:09 AM, Laine Stump wrote:
> On 01/31/2012 10:22 PM, Eric Blake wrote:
>> On 01/31/2012 05:54 PM, Taku Izumi wrote:
>>> Fix my typo at
>>>    commit 74e034964c32edb1732d0ff7642f3977f3587d72
>>> "disk->rawio == -1" indicates that this value is not
>>> specified. So in case of this, domain must not
>>> be tainted.
>>> Signed-off-by: Taku Izumi<izumi.taku at jp.fujitsu.com>
>>> ---
>>>   src/qemu/qemu_domain.c |    2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>> ACK and pushed.
> Sorry I didn't notice this mail earlier. Actually, in the version of the
> code that was pushed, the concept of using "-1" as the default value was
> scrapped in favor of having a "rawio_specified" bool in the structure
> that defaulted to false. This way the structure doesn't need any special
> initialization (the lack of which was causing multiple make check
> failures with the original patch). So, rawio will only ever be set to 0
> or 1 - it will never be -1.
> This means that the code originally committed actually works correctly.
> The modification in this "fix" also works, though.

And that still makes me wonder if we want to switch to a three-state
enum, default, off, and on, and compare disk->rawio ==
VIR_DOMAIN_DISK_RAWIO_ON, rather than tracking a secondary bool.  Oh well.

Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120203/f88d986f/attachment-0001.sig>

More information about the libvir-list mailing list