Re: [libvirt] [PATCH v2] Fix disability to run on systems with no PCI bus

2011-04-22 Thread Diego Elio Pettenò
Il giorno ven, 22/04/2011 alle 14.24 +0200, Michal Privoznik ha scritto: +/* On x390(x) system there is no PCI bus. Looks like a typo? -- Diego Elio Pettenò — Flameeyes http://blog.flameeyes.eu/ -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo

Re: [libvirt] [PATCH] qemu: Always reserves slot 0x02 for primary VGA.

2011-04-05 Thread Diego Elio Pettenò
collision for virtio-blk as well: https://bugs.gentoo.org/show_bug.cgi?id=362197 -- Diego Elio Pettenò — Flameeyes http://blog.flameeyes.eu/ -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] Entering 0.9.0 freeze week, RC1 version to test

2011-03-28 Thread Diego Elio Pettenò
Il giorno lun, 28/03/2011 alle 14.25 +0800, Daniel Veillard ha scritto: thanks for giving it a try, it seems to work fine here at least for basic tasks, All looks fine in Gentoo, now it's in testing, will report if I hit any issue. -- Diego Elio Pettenò — Flameeyes http

[libvirt] int-overflow test fails if /root is not writable and /root/.virsh does not exist yet

2011-03-17 Thread Diego Elio Pettenò
=359307 I guess it is worth looking at and fix it not to fail in this case? Thanks, -- Diego Elio Pettenò — Flameeyes http://blog.flameeyes.eu/ -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [libvirt-php] rewrite checking for libxml2

2011-02-21 Thread Diego Elio Pettenò
Il giorno lun, 21/02/2011 alle 09.51 +0800, Lyre ha scritto: +# PKG_PROG_PKG_CONFIG([MIN-VERSION]) +# -- I don't think you're supposed to inline pkg.m4 within aclocal... -- Diego Elio Pettenò — Flameeyes http://blog.flameeyes.eu/ -- libvir-list mailing list

Re: [libvirt] [libvirt-php] rewrite checking for libxml2

2011-02-21 Thread Diego Elio Pettenò
them, nor a single rm m4/pkg.m4 or similar (which you can do with libtool). -- Diego Elio Pettenò — Flameeyes http://blog.flameeyes.eu/ -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [libvirt-php] rewrite checking for libxml2

2011-02-21 Thread Diego Elio Pettenò
... if you were not to add it to the repository at all, it would require for pkg-config to be installed on the system when autotools are rebuilt, which is a decent request, and would otherwise work correctly. So in my opinion it's just better not to add pkg.m4 to libvirt-php at all. -- Diego Elio

Re: [libvirt] [libvirt-php] rewrite checking for libxml2

2011-02-21 Thread Diego Elio Pettenò
Elio Pettenò — Flameeyes http://blog.flameeyes.eu/ -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] Currently in freeze for 0.8.8, rc2 available

2011-02-14 Thread Diego Elio Pettenò
Il giorno lun, 14/02/2011 alle 12.22 +0800, Daniel Veillard ha scritto: Just a reminder that we are in feature freeze for 0.8.8, I just uploaded a new candidate version: All green in Gentoo, even tests working without workarounds this time! Thanks! -- Diego Elio Pettenò — Flameeyes http

[libvirt] commandtest fails when executed with preloaded libraries

2011-01-06 Thread Diego Elio Pettenò
for in the pre-generated log files. Any idea how I should go about to solve this beside adding LD_PRELOAD to the log files themselves for Gentoo? -- Diego Elio Pettenò — Flameeyes http://blog.flameeyes.eu/ -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo

[libvirt] [PATCH] commandtest: avoid printing loader-control variables from commandhelper

2011-01-06 Thread Diego Elio Pettenò
This avoids throwing the tests off if LD_LIBRARY_PATH or LD_PRELOAD or other variables are set. Signed-off-by: Diego Elio Pettenò flamee...@gmail.com --- tests/commandhelper.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/tests/commandhelper.c b/tests

Re: [libvirt] Back and suggested release schedule

2010-12-22 Thread Diego Elio Pettenò
, I'd go for 0.8.7 myself, after all the version should be intended more for users than developers, and there is no requirement to change, e.g. configuration files. -- Diego Elio Pettenò — Flameeyes http://blog.flameeyes.eu/ -- libvir-list mailing list libvir-list@redhat.com https

Re: [libvirt] Back and suggested release schedule

2010-12-22 Thread Diego Elio Pettenò
cannot be found by using either a git checkout or a git snapshot, and can only be found when using the output of make dist. -- Diego Elio Pettenò — Flameeyes http://blog.flameeyes.eu/ -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH 2/2] build: quote AC_LANG_PROGRAM to avoid warnings with autoconf 2.68

2010-12-13 Thread Diego Elio Pettenò
--- configure.ac |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index b5c995f..eb1425a 100644 --- a/configure.ac +++ b/configure.ac @@ -73,7 +73,7 @@ VERSION_SCRIPT_FLAGS=-Wl,--version-script= LIBVIRT_COMPILE_WARNINGS([maximum])

[libvirt] [PATCH 1/2] build: properly handle ./configure --with-libpcap

2010-12-13 Thread Diego Elio Pettenò
Without this fix, ./configure --with-libpcap will cause --with-libpcap=yes to be implicitly passed down, which cause yes/bin/pcap-config to be searched for rather than /usr/bin/pcap-config. Also output pcap: no when pcap is not found or disabled. --- configure.ac | 30

Re: [libvirt] [PATCH] build: avoid shadowing devname() on BSD systems

2010-12-03 Thread Diego Elio Pettenò
-access to my mac right now, or I'd be sending something myself). -- Diego Elio Pettenò — “Flameeyes” http://blog.flameeyes.eu/ If you found a .asc file in this mail and know not what it is, it's a GnuPG digital signature: http://www.gnupg.org/ -- libvir-list mailing list libvir-list@redhat.com

Re: [libvirt] Release of libvirt 0.8.6

2010-12-02 Thread Diego Elio Pettenò
RedHatco.? Why not warning the rest of the packagers, so that each run their tests? -- Diego Elio Pettenò — “Flameeyes” http://blog.flameeyes.eu/ If you found a .asc file in this mail and know not what it is, it's a GnuPG digital signature: http://www.gnupg.org/ -- libvir-list mailing list libvir

Re: [libvirt] Release of libvirt 0.8.6

2010-12-01 Thread Diego Elio Pettenò
macvtap references when macvtap is disabled at configure time. -- Diego Elio Pettenò — “Flameeyes” http://blog.flameeyes.eu/ If you found a .asc file in this mail and know not what it is, it's a GnuPG digital signature: http://www.gnupg.org/ signature.asc Description: This is a digitally signed

Re: [libvirt] Release of libvirt 0.8.6

2010-12-01 Thread Diego Elio Pettenò
-traffic) mailinglist for packagers to test the release? We'd then have time to test whether the code is fine for all of us or not. [Speaking as the Gentoo maintainer of libvirt] -- Diego Elio Pettenò — “Flameeyes” http://blog.flameeyes.eu/ If you found a .asc file in this mail and know not what

Re: [libvirt] Release of libvirt 0.8.6

2010-12-01 Thread Diego Elio Pettenò
Il giorno gio, 02/12/2010 alle 05.47 +1100, Justin Clift ha scritto: Concept wise, do you reckon something like this would work: Yes that looks perfect to me. -- Diego Elio Pettenò — “Flameeyes” http://blog.flameeyes.eu/ If you found a .asc file in this mail and know not what it is, it's

Re: [libvirt] how to attach PCI/USB device

2010-11-02 Thread Diego Elio Pettenò
be able to test login with that stuff without them ;) -- Diego Elio Pettenò — “Flameeyes” http://blog.flameeyes.eu/ If you found a .asc file in this mail and know not what it is, it's a GnuPG digital signature: http://www.gnupg.org/ -- libvir-list mailing list libvir-list@redhat.com https

[libvirt] [PATCH 1/2] qemu: add the USB devices to the cgroup whitelist

2010-11-01 Thread Diego Elio Pettenò
Make sure that the QEmu process within the cgroup can access the device file for the USB device that has to be connected to the virtual domain. --- src/qemu/qemu_driver.c | 55 1 files changed, 55 insertions(+), 0 deletions(-) diff --git

[libvirt] [PATCH 2/2] misc: encode the Emacs suggstions from HACKING into a .dir-locals.el file

2010-11-01 Thread Diego Elio Pettenò
With this file in place, opening any source file in libvirt will set up Emacs for proper indentation. --- .dir-locals.el |8 1 files changed, 8 insertions(+), 0 deletions(-) create mode 100644 .dir-locals.el diff --git a/.dir-locals.el b/.dir-locals.el new file mode 100644 index

Re: [libvirt] [PATCH] qemu: add the USB devices to the cgroup whitelist

2010-11-01 Thread Diego Elio Pettenò
Emacs users would get it out of the box? ;) -- Diego Elio Pettenò — “Flameeyes” http://blog.flameeyes.eu/ If you found a .asc file in this mail and know not what it is, it's a GnuPG digital signature: http://www.gnupg.org/ signature.asc Description: This is a digitally signed message part

Re: [libvirt] how to attach PCI/USB device

2010-11-01 Thread Diego Elio Pettenò
are enabled. Not sure about the PCI stuff sincerely, but at least USB with the current git head works fine for me (finally). -- Diego Elio Pettenò — Flameeyes http://blog.flameeyes.eu/ -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] xen: work with ia64 MAX_VIRT_CPUS of 64

2010-10-30 Thread Diego Elio Pettenò
: https://bugs.gentoo.org/show_bug.cgi?id=343353 -- Diego Elio Pettenò — “Flameeyes” http://blog.flameeyes.eu/ If you found a .asc file in this mail and know not what it is, it's a GnuPG digital signature: http://www.gnupg.org/ signature.asc Description: This is a digitally signed message part

Re: [libvirt] Release of libvirt-0.8.5

2010-10-30 Thread Diego Elio Pettenò
of it (it doesn't apply cleanly); you can find the one I've been applying at http://git.overlays.gentoo.org/gitweb/?p=proj/libvirt.git;a=patch;h=844ba61193015624dcf753b1eed2f62ae5d66685 -- Diego Elio Pettenò — “Flameeyes” http://blog.flameeyes.eu/ If you found a .asc file in this mail and know

Re: [libvirt] Release of libvirt-0.8.5

2010-10-30 Thread Diego Elio Pettenò
=proj/libvirt.git;a=patch;h=8547fa3c9c93ddabc2ac1d0f44b03d52a8decc84 In gentoo I have a 0.8.5-r1 that applies those two and the one I sent for USB/cgroup interactions (since I needed it for my stuff and it was reported in Gentoo as a bug already). -- Diego Elio Pettenò — “Flameeyes” http

[libvirt] [PATCH] build: cleanup declaration of xen tests.

2010-10-29 Thread Diego Elio Pettenò
Only build statstest and reconnect if they should be used, so add them behind the ENABLE_XEN_TESTS conditional. --- tests/Makefile.am | 10 ++ 1 files changed, 6 insertions(+), 4 deletions(-) diff --git a/tests/Makefile.am b/tests/Makefile.am index 9720b6e..20b6805 100644 ---

Re: [libvirt] Release of libvirt-0.8.5

2010-10-29 Thread Diego Elio Pettenò
Il giorno ven, 29/10/2010 alle 17.11 +0200, Daniel Veillard ha scritto: So the release it out ! And hopefully we are back on our 'end of month' release cycle. The libvirt-0.8.5 release is available from 0.8.5 has broken tests when disabling Xen support (see my patch about that). -- Diego Elio

[libvirt] [PATCH] qemu: add the USB devices to the cgroup whitelist

2010-10-29 Thread Diego Elio Pettenò
Add a new interface to hostusb.h to add an USB device abstraction to a cgroup whitelist; then use it both when attaching a new USB device and when adding it to the commandline so that the device can be accessed by the QEmu-specific cgroup. --- src/libvirt_private.syms |1 +

[libvirt] [PATCH] Don't use %.3d format for bus/addr of USB devices.

2010-10-26 Thread Diego Elio Pettenò
When using 0-prefixed numbers, QEmu will interpret them as octal numbers (as C convention says); this means that if you attach a device that has addr 10 (decimal) you're going to attach a different device. --- src/qemu/qemu_conf.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)

[libvirt] cgroups and USB passthrough devices don't work together

2010-10-26 Thread Diego Elio Pettenò
the device… -- Diego Elio Pettenò — “Flameeyes” http://blog.flameeyes.eu/ If you found a .asc file in this mail and know not what it is, it's a GnuPG digital signature: http://www.gnupg.org/ signature.asc Description: This is a digitally signed message part -- libvir-list mailing list libvir-list

[libvirt] [PATCH] build-sys: only build the test programs during the check phase.

2010-08-10 Thread Diego Elio Pettenò
This avoids building the tests when testing libvirt is not the aim. --- tests/Makefile.am | 26 +- 1 files changed, 13 insertions(+), 13 deletions(-) diff --git a/tests/Makefile.am b/tests/Makefile.am index 71c2c74..1dc7f66 100644 --- a/tests/Makefile.am +++

[libvirt] [PATCH] build-sys: distribute the man pages in the tarball

2010-08-09 Thread Diego Elio Pettenò
This slightly reduces the requirements for building libvirt. The man pages are deleted with make maintainer-clean. --- daemon/Makefile.am |5 +++-- tools/Makefile.am |5 +++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/daemon/Makefile.am b/daemon/Makefile.am index

[libvirt] [PATCH 1/2] build-sys: fix build when daemon is disabled by not installing libvirtd.8

2010-08-05 Thread Diego Elio Pettenò
Since the rule to build libvirtd.8 is within the WITH_LIBVIRTD conditional, so declare the man page in there as well. Without this change, build without daemon will fail. --- .gnulib|2 +- daemon/Makefile.am |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git

[libvirt] [PATCH 2/2] build-sys: distribute the man pages in the tarball

2010-08-05 Thread Diego Elio Pettenò
This slightly reduces the requirements for building libvirt. --- daemon/Makefile.am |4 ++-- tools/Makefile.am |4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/daemon/Makefile.am b/daemon/Makefile.am index dbf0ac3..74ad67b 100644 --- a/daemon/Makefile.am +++

[libvirt] [PATCH 1/4] Avoid searching for windres when not building for cygwin or mingw.

2010-02-24 Thread Diego Elio Pettenò
Just checking for a windres tool might hit even on Linux systems when building for Linux (e.g.: when using Gentoo and having built binutils with multitarget support), and will then fail to link properly at the end of the build. Check the host string before deciding whether to look for windres or

[libvirt] [PATCH 3/4] Add a simple macro to check in which library dlopen() is, and use it.

2010-02-24 Thread Diego Elio Pettenò
Since on not all systems dlopen() is in libdl, add a very simple macro that allows to search for it and sets a DLOPEN_LIBS variable with its value to be used. Use the macro instead of hardcoding -ldl for the drivers-as-modules case. --- configure.ac |4 ++-- m4/libdl.m4 | 44

[libvirt] [PATCH 4/4] Fix build of the VirtualBox driver when using --no-add-needed.

2010-02-24 Thread Diego Elio Pettenò
With the recent changes to the linking defaults in Fedora 13 (namely enabling --no-add-needed behaviour by default), we have to pass the dlopen()-providing libraries directly at the link of the module; use the newly introduced macro to look for it and add it to the Makefile. --- configure.ac|

[libvirt] [PATCH 1/2] Use AC_SEARCH_LIBS to find the library to use for dlopen().

2010-02-24 Thread Diego Elio Pettenò
Instead of using AC_CHECK_LIB and hardcoding -ldl, search for the library needed to get dlopen() and then use the cached value. --- configure.ac |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 5b1eb5f..a3f28c4 100644 --- a/configure.ac

[libvirt] [PATCH] Avoid searching for windres when not building for cygwin or mingw.

2010-02-12 Thread Diego Elio Pettenò
Just checking for a windres tool might hit even on Linux systems when building for Linux (e.g.: when using Gentoo and having built binutils with multitarget support), and will then fail to link properly at the end of the build. Check the host string before deciding whether to look for windres or

[libvirt] [PATCH 1/2] Standardise ./configure --help options reporting.

2010-01-17 Thread Diego Elio Pettenò
Always use AC_HELP_STRING for the help text for options at ./configure, so that the output is properly aligned. Use proper quadrigraphs for outputting the brackets. Always use autoconf-style [default=$foo] output to state the default, both where it was stated before and where it wasn't. This

[libvirt] [PATCH 2/2] Change detection of xen so that it's actually automatic rather than forced.

2010-01-17 Thread Diego Elio Pettenò
This ensures that ./configure will work fine if xen development packages are not around, rather than fail. When passing ./configure --with-xen, the lack of xen development packages become fatal. --- configure.in | 60 - 1 files changed, 34

[libvirt] [PATCH 1/4] Avoid creating special ./configure options for htmldir.

2010-01-16 Thread Diego Elio Pettenò
Request use of autoconf 2.63 or later, so that --htmldir (and the htmldir variable) are defined. This makes deciding where to install the HTML documentation more in line with the autoconf standard and other packages. --- configure.in | 12 +--- docs/Makefile.am | 14 +++---

[libvirt] [PATCH 3/4] Change detection of xen so that it's actually automatic rather than forced.

2010-01-16 Thread Diego Elio Pettenò
This ensures that ./configure will work fine if xen development packages are not around, rather than fail. When passing ./configure --with-xen, the lack of xen development packages become fatal. --- configure.in | 60 - 1 files changed, 34

[libvirt] [PATCH 0/4] A bit of configure clear-up

2010-01-16 Thread Diego Elio Pettenò
Since on the whole it seems like the configure could see some cleanup, as it seem to use lots of older constructs and repeated code, I tried doing some basic cleaning. My changes also require autoconf-2.63… it could be something older as well, but 2.63 is new enough I'd say. If nobody has reasons

[libvirt] [PATCH 4/4] Remove definition of m4_foreach_w for autoconf 2.59.

2010-01-16 Thread Diego Elio Pettenò
Depends on the previous patch requiring autoconf 2.63 or later. --- configure.in |7 --- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/configure.in b/configure.in index 18b5429..385c48b 100644 --- a/configure.in +++ b/configure.in @@ -49,13 +49,6 @@ AC_PROG_CPP

[libvirt] [PATCH 2/4] Standardise options reporting on ./configure --help

2010-01-16 Thread Diego Elio Pettenò
Always use AS_HELP_STRING to define the help string for the options, fix proper quoting and quadrigraphs for brackets, and always use the [default=$foo] syntax to show the default (in line with autoconf's own defaults). --- configure.in | 98

[libvirt] [PATCH] Disallow building Python bindings if shared libraries are disabled.

2010-01-08 Thread Diego Elio Pettenò
Patch concept by Jim Meyering j...@meyering.net, moved before the Python testing, and changed the message a bit to explain the problem. --- configure.in |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/configure.in b/configure.in index 3f2f8ff..5275412 100644 ---

[libvirt] [PATCH 2/3] Don't re-invent the HTML directory destination.

2009-12-23 Thread Diego Elio Pettenò
Instead of adding a --with-html-dir option to the configure, require autoconf 2.61 or later that already provides the --htmldir option. --- configure.in | 12 +--- docs/Makefile.am | 14 +++--- 2 files changed, 8 insertions(+), 18 deletions(-) diff --git a/configure.in

[libvirt] [PATCH] Disable building of static Python module.

2009-12-23 Thread Diego Elio Pettenò
Python modules are loaded at runtime so the static version of it is not really needed, this avoids duplicating the build for the PIC and non-PIC cases. --- python/Makefile.am |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/python/Makefile.am b/python/Makefile.am index

[libvirt] [PATCH] Allow versioned pc machines for QEmu

2009-12-11 Thread Diego Elio Pettenò
* docs/schemas/domain.rng: add pattern for pc-0.10, pc-0.11 and similar machine types --- docs/schemas/domain.rng |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/docs/schemas/domain.rng b/docs/schemas/domain.rng index d1d3efb..dd3f732 100644 ---