[libvirt] [PATCHv3] qemu: reduce file padding requirements

[Eric Blake]

On 06/17/2010 01:32 AM, Laine Stump wrote:
> On 06/10/2010 03:31 PM, Eric Blake wrote:
>> Followup to https://bugzilla.redhat.com/show_bug.cgi?id=599091,
>> commit 20206a4b, to reduce disk waste in padding.
>>
>> * src/qemu/qemu_monitor.h (QEMU_MONITOR_MIGRATE_TO_FILE_BS): Drop
>> back to 512.
>> (QEMU_MONITOR_MIGRATE_TO_FILE_TRANSFER_SIZE): New macro.
>> * src/qemu/qemu_driver.c (qemudDomainSaveFlag): Update comment.
>> * src/qemu/qemu_monitor_text.c (qemuMonitorTextMigrateToFile): Use
>> two invocations of dd to output non-aligned large blocks.
>> * src/qemu/qemu_monitor_json.c (qemuMonitorJSONMigrateToFile):
>> Likewise.
>> ---
>>
>> v2: avoid truncating regular file destination
>>
>> v3: change minimum block size from 512 to 4k, to avoid unaligned
>> RMW cycles on newer disks with 4k sectors
>>    
> 
> Okay, I've finally gotten around to trying this out, and in addition to
> reducing the file size, it also works!
> 
> ACK
> 
> (Note: I haven't tested saving to a block device, so I'm taking the word
> of the dd documentation that this works as well ;-)

Thanks; pushed.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

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