[libvirt] Re: [PATCH] Fix raw storage volume creation for allocation < capacity.

Cole Robinson crobinso at redhat.com
Mon Jun 22 17:19:41 UTC 2009


Daniel Veillard wrote:
> On Thu, Jun 18, 2009 at 03:13:06PM -0400, Cole Robinson wrote:
>> Cole Robinson wrote:
>>> CreateXMLFrom changes accidentally caused all raw volume creation to be
>>> fully allocated (as though allocation == capacity). Fix this.
>>>
>>> Also force CreateXMLFrom to maintain the previous behavior: sparseness
>>> should still be maintained since we search for holes when copying, and the
>>> clone behavior hasn't been tested with anything but the broken behavior.
> [...]
>>> +    /* Make sure allocation is at least as large as the destination cap,
>>> +     * to make absolutely sure we copy all possible contents */
>>> +    if (newvol->allocation < origvol->capacity;
>>> +        newvol->allocation = origvol->capacity;
>>> +
>> Of course the tweak I make at the last second is completely invalid
>> syntax :(. Besides needing the obvious fix above, the patch does work as
>> expected.
> 
>   Heh, ACK to tyhe fixed fix :-)
> 
> Daniel
> 

Thanks, applied with the above syntax fix.

- Cole




More information about the libvir-list mailing list