[libvirt] [PATCH 2/2]: Call udevsettle in the appropriate places
Daniel P. Berrange
berrange at redhat.com
Thu Nov 27 16:21:13 UTC 2008
On Thu, Nov 27, 2008 at 08:53:49AM +0100, Chris Lalancette wrote:
> Daniel P. Berrange wrote:
> Index: configure.in
> ===================================================================
> RCS file: /data/cvs/libvirt/configure.in,v
> retrieving revision 1.187
> diff -u -r1.187 configure.in
> --- a/configure.in 25 Nov 2008 15:48:11 -0000 1.187
> +++ b/configure.in 27 Nov 2008 07:44:51 -0000
> @@ -115,11 +115,17 @@
> [/sbin:/usr/sbin:/usr/local/sbin:$PATH])
> AC_PATH_PROG([BRCTL], [brctl], [brctl],
> [/sbin:/usr/sbin:/usr/local/sbin:$PATH])
> +AC_PATH_PROG([UDEVADM], [udevadm], [],
> + [/sbin:/usr/sbin:/usr/local/sbin:$PATH])
>
> AC_DEFINE_UNQUOTED([DNSMASQ],["$DNSMASQ"],
> [Location or name of the dnsmasq program])
> AC_DEFINE_UNQUOTED([BRCTL],["$BRCTL"],
> [Location or name of the brctl program (see bridge-utils)])
> +if test -n "$UDEVADM"; then
> + AC_DEFINE_UNQUOTED([UDEVADM],["$UDEVADM"],
> + [Location or name of the udevadm program])
> +fi
ACK.
> +void virStorageBackendWaitForDevices(virConnectPtr conn)
> +{
> +#ifdef UDEVADM
> + const char *const settleprog[] = { UDEVADM, "settle", NULL };
> + int exitstatus;
> +
> + if (access(UDEVADM, X_OK) != 0)
> + return;
> +
> + /*
> + * NOTE: we ignore errors here; this is just to make sure that any device
> + * nodes that are being created finish before we try to scan them.
> + * If this fails for any reason, we still have the backup of polling for
> + * 5 seconds for device nodes.
> + */
> + virRun(conn, settleprog, &exitstatus);
> +#endif
> +}
This will generate a compiler warning when UDEVADM is not defined.
Better to have the #ifdef around the entire function, and then do
a no-op decl with ATTRIBUTE_UNUSED
#else
void virStorageBackendWaitForDevices(virConnectPtr conn ATTRIBUTE_UNUSED) {}
#endif
Daniel
--
|: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :|
|: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|
More information about the libvir-list
mailing list