[libvirt] [PATCH 4/6 V4] inject-nmi: Implementing the remote protocol

[Eric Blake]

On 05/10/2011 02:26 AM, Lai Jiangshan wrote:
> ---
>  daemon/remote_generator.pl   |    2 +-
>  src/remote/remote_driver.c   |    2 +-
>  src/remote/remote_protocol.x |    8 +++++++-
>  src/remote_protocol-structs  |    4 ++++
>  4 files changed, 13 insertions(+), 3 deletions(-)
> 
> diff --git a/daemon/remote_generator.pl b/daemon/remote_generator.pl
> index 062ccc1..74fa769 100755
> --- a/daemon/remote_generator.pl
> +++ b/daemon/remote_generator.pl
> @@ -35,7 +35,7 @@ sub name_to_ProcName {
>      @elems = map { $_ =~ s/Nwfilter/NWFilter/; $_ =~ s/Xml/XML/;
>                     $_ =~ s/Uri/URI/; $_ =~ s/Uuid/UUID/; $_ =~ s/Id/ID/;
>                     $_ =~ s/Mac/MAC/; $_ =~ s/Cpu/CPU/; $_ =~ s/Os/OS/;
> -                   $_ } @elems;
> +                   $_ =~ s/Nmi/NMI/; $_ } @elems;

Do we really want yet another difference in spelling?  But cleaning that
up affects more than just your new API, so I'll leave it for a later patch.

> +++ b/src/remote/remote_protocol.x
> @@ -817,6 +817,11 @@ struct remote_domain_undefine_args {
>      remote_nonnull_domain dom;
>  };
>  
> +struct remote_domain_inject_nmi_args {
> +    remote_nonnull_domain dom;
> +    unsigned int flags;
> +};
> +
>  struct remote_domain_set_vcpus_args {

This file layout...

> +++ b/src/remote_protocol-structs
> @@ -1435,3 +1435,7 @@ struct remote_message_header {
>          u_int                      serial;
>          remote_message_status      status;
>  };
> +struct remote_domain_inject_nmi_args {
> +        remote_nonnull_domain      dom;
> +        unsigned int               flags;
> +};

didn't match this layout - I had to adjust the structs to come in the
same order.  On Fedora, if you install the 'dwarves' package, then 'make
check' would catch this for you.  Also, dwarves settles on u_int rather
than unsigned int.

-- 
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/20110510/bb1f5152/attachment-0001.sig>