[libvirt] [PATCH 4/5] setmem: implement the remote protocol to address the new API
Eric Blake
eblake at redhat.com
Thu Mar 10 22:09:09 UTC 2011
On 03/02/2011 01:13 AM, Taku Izumi wrote:
>
> This patch implements the remote protocol to address the new API.
>
> Index: libvirt-git/src/remote/remote_driver.c
> ===================================================================
> --- libvirt-git.orig/src/remote/remote_driver.c
> +++ libvirt-git/src/remote/remote_driver.c
> @@ -2445,6 +2445,33 @@ done:
> }
>
> static int
> +remoteDomainSetMemoryFlags (virDomainPtr domain, unsigned long memory, unsigned int
> flags)
The extra newline here caused git to complain that your patch was
corrupt. You had several other cases where the patch was botched.
After fixing those, the patch still failed indentation in
remote_protocol-structs. (Hmm - why do we create that file with TABs in
the first place? I should fix that in a followup)
> Index: libvirt-git/src/remote_protocol-structs
> ===================================================================
> --- libvirt-git.orig/src/remote_protocol-structs
> +++ libvirt-git/src/remote_protocol-structs
> @@ -340,6 +340,11 @@ struct remote_domain_set_memory_args {
> remote_nonnull_domain dom;
> uint64_t memory;
> };
> +struct remote_domain_set_memory_flags_args {
> + remote_nonnull_domain dom;
> + uint64_t memory;
> + u_int flags;
> +};
Plus, when I reran rpcgen, it changed the spacing in remote_protocol.c.
I pushed after fixing those issues.
--
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/20110310/0f9d0d42/attachment-0001.sig>
More information about the libvir-list
mailing list