Re: [Libvirt-announce] Availability of 0.10.0 release candidate 2

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

 



-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

On Tue, Aug 28, 2012 at 04:05:36PM +0800, Daniel Veillard thus spake:
>  I don't like too much the idea of adding kernel defines in our headers.
>Basically this is all due to commits
>d57567940153147f4d43875fb92c3030b3178b03 and
>080bf330e3749d94ebe094f8deca0e3e67d3f2fe
>
>  The fact that it breaks only in daemon.c, i.e. after the library
>  compiled is a good infocation that the guards
>#if WITH_VIRTUALPORT in src/util/virnetdevvportprofile.c
>#if WITH_MACVTAP     in src/util/virnetdevmacvlan.c
>and
>#if defined(__linux__) && defined(HAVE_LIBNL) && defined(IFLA_VF_MAX)
> in src/util/virnetdev.c
>
>seems sufficient to avoid the portability problem in those files
>and the remaining problem is really about daemon/libvirtd.c and
>I would suggest a fix along those lines:
>
>Daniel
>
>diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c
>index 19dd26b..352d4fe 100644
>--- a/daemon/libvirtd.c
>+++ b/daemon/libvirtd.c
>@@ -1312,17 +1312,21 @@ int main(int argc, char **argv) {
>         goto cleanup;
>     }
>
>+#if defined(__linux__) && defined(NETLINK_ROUTE)
>     /* Register the netlink event service for NETLINK_ROUTE */
>     if (virNetlinkEventServiceStart(NETLINK_ROUTE, 0) < 0) {
>         ret = VIR_DAEMON_ERR_NETWORK;
>         goto cleanup;
>     }
>+#endif
>
>+#if defined(__linux__) && defined(NETLINK_KOBJECT_UEVENT)
>     /* Register the netlink event service for NETLINK_KOBJECT_UEVENT */
>     if (virNetlinkEventServiceStart(NETLINK_KOBJECT_UEVENT, 1) < 0) {
>         ret = VIR_DAEMON_ERR_NETWORK;
>         goto cleanup;
>     }
>+#endif
>
>     /* Run event loop. */
>     virNetServerRun(srv);
>-- 
>Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
>daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
>http://veillard.com/ | virtualization library  http://libvirt.org/
>

This is building as expected with your new patch for FreeBSD. Here is a link
to the build logs.


https://redports.org/buildarchive/20120828164025-20179/

- -jgh

- -- 
Jason Helfman
System Administrator
experts-exchange.com
http://www.experts-exchange.com/M_4830110.html
E4AD 7CF1 1396 27F6 79DD  4342 5E92 AD66 8C8C FBA5
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.19 (FreeBSD)

iF4EAREIAAYFAlA8+WYACgkQXpKtZoyM+6WK9wD9HXb5AybTrroQJTLseuRLvr7p
scRUSIA7lfSFM7s2lxUA/0B5X878TeDFk9IUaXgOWOsbjt6z28+m/1LuH9XcHHHa
=MCSJ
-----END PGP SIGNATURE-----

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