Re: [PATCHv2 03/27] build: also check qemu_protocol for on-the-wire stability

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

 



On Fri, Jul 08, 2011 at 01:25:45PM -0600, Eric Blake wrote:
> Since we are going to add some libvirt-qemu.so entry points in
> 0.9.4, we might as well start checking for RPC stability, just
> as for libvirt.so.
> 
> * src/Makefile.am (PROTOCOL_STRUCTS): New variable.
> (remote_protocol-structs): Rename...
> (%_protocol-structs): ...and make more generic.
> * src/qemu_protocol-structs: New file.
> ---
> 
> v2: new patch, first suggested here:
> https://www.redhat.com/archives/libvir-list/2011-July/msg00433.html
> 
>  src/Makefile.am           |   20 +++++++++++---------
>  src/qemu_protocol-structs |   14 ++++++++++++++
>  2 files changed, 25 insertions(+), 9 deletions(-)
>  create mode 100644 src/qemu_protocol-structs
> 
> diff --git a/src/Makefile.am b/src/Makefile.am
> index cd8a7e9..bd965de 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -211,16 +211,18 @@ EXTRA_DIST +=  $(REMOTE_DRIVER_PROTOCOL) \
>  r1 = (?:/\* \d+ \*/\n)?
>  r2 = /\* <[[:xdigit:]]+> \S+:\d+ \*/
> 
> -.PHONY: remote_protocol-structs
> +PROTOCOL_STRUCTS = \
> +	$(srcdir)/remote_protocol-structs \
> +	$(srcdir)/qemu_protocol-structs
>  if WITH_REMOTE
>  # The .o file that pdwtags parses is created as a side effect of running
>  # libtool; but from make's perspective we depend on the .lo file.
> -remote_protocol-structs: libvirt_driver_remote_la-remote_protocol.lo
> +%_protocol-structs: libvirt_driver_remote_la-%_protocol.lo
>  	$(AM_V_GEN)if (pdwtags --help) > /dev/null 2>&1; then		\
> -	  pdwtags --verbose libvirt_driver_remote_la-remote_protocol.$(OBJEXT) \
> +	  pdwtags --verbose $(<:.lo=.$(OBJEXT))				\
>  	    | perl -0777 -n						\
>  		-e 'foreach my $$p (split m!\n\n$(r1)$(r2)\n!) {'	\
> -		-e '  if ($$p =~ /^struct remote_/) {'			\
> +		-e '  if ($$p =~ /^struct (remote|qemu)_/) {'		\
>  		-e '    $$p =~ s!\t*/\*.*?\*/!!sg;'			\
>  		-e '    $$p =~ s!\s+\n!\n!sg;'				\
>  		-e '    $$p =~ s!\s+$$!!;'				\
> @@ -233,7 +235,7 @@ remote_protocol-structs: libvirt_driver_remote_la-remote_protocol.lo
>  		-e '  print "/* -*- c -*- */\n";'			\
>  		-e '}'							\
>  		-e 'END {'						\
> -		-e '  if ($$n < 300) {'					\
> +		-e '  if ($$n < 3) {'					\
>  		-e '    warn "WARNING: your pdwtags program is too old\n";' \
>  		-e '    warn "WARNING: skipping the $@ test\n";'	\
>  		-e '    warn "WARNING: install dwarves-1.3 or newer\n";' \
> @@ -248,12 +250,12 @@ remote_protocol-structs: libvirt_driver_remote_la-remote_protocol.lo
>  	  echo 'WARNING: install the dwarves package to get pdwtags' >&2; \
>  	fi
>  else !WITH_REMOTE
> -# This generated file must live in git, because it cannot be re-generated
> +# These generated files must live in git, because they cannot be re-generated
>  # when configured --without-remote.
> -remote_protocol-structs:
> +$(srcdir)/%_protocol-structs:
>  endif
> -EXTRA_DIST += remote_protocol-structs
> -check-local: remote_protocol-structs
> +EXTRA_DIST += $(PROTOCOL_STRUCTS)
> +check-local: $(PROTOCOL_STRUCTS)
> 
>  # Mock driver, covering domains, storage, networks, etc
>  TEST_DRIVER_SOURCES =						\
> diff --git a/src/qemu_protocol-structs b/src/qemu_protocol-structs
> new file mode 100644
> index 0000000..e93e8bf
> --- /dev/null
> +++ b/src/qemu_protocol-structs
> @@ -0,0 +1,14 @@
> +/* -*- c -*- */
> +struct remote_nonnull_domain {
> +        remote_nonnull_string      name;
> +        remote_uuid                uuid;
> +        int                        id;
> +};
> +struct qemu_monitor_command_args {
> +        remote_nonnull_domain      dom;
> +        remote_nonnull_string      cmd;
> +        int                        flags;
> +};
> +struct qemu_monitor_command_ret {
> +        remote_nonnull_string      result;
> +};

ACK


If we want real paranoia we can do  src/rpc/virnetprotocol.x too,
though that should basically never be changed by normal patches.

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
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]