[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