Re: [PATCH v3 01/18] blockcopy: allow larger buf-size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Aug 30, 2014 at 22:02:19 -0600, Eric Blake wrote:
> While qemu definitely caps granularity to 64 MiB, it places no
> limits on buf-size.  On a machine beefy enough for lots of
> memory, a buf-size larger than 2 GiB is feasible, so we should
> pass a 64-bit parameter.
> 
> * include/libvirt/libvirt.h.in (VIR_DOMAIN_BLOCK_COPY_BUF_SIZE):
> Allow 64 bits.
> 
> Signed-off-by: Eric Blake <eblake@xxxxxxxxxx>
> ---
>  include/libvirt/libvirt.h.in | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/libvirt/libvirt.h.in b/include/libvirt/libvirt.h.in
> index 9358314..a64f597 100644
> --- a/include/libvirt/libvirt.h.in
> +++ b/include/libvirt/libvirt.h.in
> @@ -2678,8 +2678,8 @@ typedef enum {
>   * VIR_DOMAIN_BLOCK_COPY_BUF_SIZE:
>   * Macro for the virDomainBlockCopy buffer size tunable: it represents
>   * how much data in bytes can be in flight between source and destination,
> - * as an unsigned int. Specifying 0 is the same as omitting this parameter,
> - * to request the hypervisor default.
> + * as an unsigned long long. Specifying 0 is the same as omitting this
> + * parameter, to request the hypervisor default.
>   */
>  #define VIR_DOMAIN_BLOCK_COPY_BUF_SIZE "buf-size"

ACK, we need this in 1.2.8.

Jirka

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]