- Devel - libvirt List Archives

[Libvir] bug #384443 - packages for virt-install and virt-manager
by Richard W.M. Jones 13 Nov '07

13 Nov '07

1 1

[Libvir] [PATCH] Make qemud's install-init rule work in a non-srcdir build.
by Jim Meyering 13 Nov '07

13 Nov '07

When building from other than the source directory, several things currently fail. This one fails because $(srcdir)/libvirtd.init doesn't exist. The libvirtd.init file we want to install is the one just built, in the current directory. Here's the fix: Make qemud's install-init rule work in a non-srcdir build. * qemud/Makefile.am: In the install-init, remove an unneeded $(srcdir)/ prefix. --- qemud/Makefile.am | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/qemud/Makefile.am b/qemud/Makefile.am index 506fde5..3da6265 100644 --- a/qemud/Makefile.am +++ b/qemud/Makefile.am @@ -95,7 +95,7 @@ remote_dispatch_proc_switch.h: remote_generate_stubs.pl remote_protocol.x if LIBVIRT_INIT_SCRIPTS_RED_HAT install-init: libvirtd.init mkdir -p $(DESTDIR)$(sysconfdir)/rc.d/init.d - $(INSTALL_SCRIPT) $(srcdir)/libvirtd.init $(DESTDIR)$(sysconfdir)/rc.d/init.d/libvirtd + $(INSTALL_SCRIPT) libvirtd.init $(DESTDIR)$(sysconfdir)/rc.d/init.d/libvirtd mkdir -p $(DESTDIR)$(sysconfdir)/sysconfig $(INSTALL_SCRIPT) $(srcdir)/libvirtd.sysconf $(DESTDIR)$(sysconfdir)/sysconfig/libvirtd -- 1.5.3.5.643.g40e25

2 2

[Libvir] bug #384300 - libvirt package for Debian
by Richard W.M. Jones 13 Nov '07

13 Nov '07

This is a libvirt package for Debian. It's derived from the Ubuntu libvirt package, rebuilt for Debian testing. I'm not the greatest Debian packager in the world, so any comments / feedback are welcome. The previous patch to the base Xen package is also required before this can be built. Source: http://et.redhat.com/~rjones/debian-libvirt/libvirt_0.3.3.orig.tar.gz http://et.redhat.com/~rjones/debian-libvirt/libvirt_0.3.3-1_i386.changes http://et.redhat.com/~rjones/debian-libvirt/libvirt_0.3.3-1.dsc http://et.redhat.com/~rjones/debian-libvirt/libvirt_0.3.3-1.diff.gz Binary for i386: http://et.redhat.com/~rjones/debian-libvirt/libvirt-bin_0.3.3-1_i386.deb http://et.redhat.com/~rjones/debian-libvirt/libvirt-dev_0.3.3-1_i386.deb http://et.redhat.com/~rjones/debian-libvirt/libvirt0_0.3.3-1_i386.deb http://et.redhat.com/~rjones/debian-libvirt/python-libvirt_0.3.3-1_i386.deb There is a Debian ITP for libvirt here: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=384300 Rich. -- Emerging Technologies, Red Hat - http://et.redhat.com/~rjones/ Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SL4 1TE, United Kingdom. Registered in England and Wales under Company Registration No. 03798903

1 0

[Libvir] [PATCH] bug #402249 - include <xen/*.h> header files
by Richard W.M. Jones 13 Nov '07

13 Nov '07

When you build Xen from source, the "make install" rule installs extra header files such as <xen/xen.h> and <xen/dom0_ops.h> which are all required for building libvirt. However the Debian package doesn't run the install rule, but instead runs (effectively) "make -C tools install" and that only installs a partial set of header files. In particular none of the vital public Xen header files are installed, making it impossible to build libvirt, and probably other stuff as well. This patch fixes that by copying the required files by hand: http://et.redhat.com/~rjones/debian-libvirt/debian-xen-includes.patch (It is similar to what the Fedora RPM does, faced with the same situation). There is a bug which is somewhat related to this issue: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=402249 Rich. -- Emerging Technologies, Red Hat - http://et.redhat.com/~rjones/ Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SL4 1TE, United Kingdom. Registered in England and Wales under Company Registration No. 03798903

1 0

[Libvir] [PATCH] conftest: fix transposed size and count arguments to fwrite.
by Jim Meyering 12 Nov '07

12 Nov '07

I noticed conftest fail and tracked it down to this: conftest: fix transposed size and count arguments to fwrite. * tests/conftest.c: Include <string.h> and <errno.h>. Also include strerror in diagnostic. --- tests/conftest.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/conftest.c b/tests/conftest.c index 604e410..9c8e3a6 100644 --- a/tests/conftest.c +++ b/tests/conftest.c @@ -1,6 +1,8 @@ #include <unistd.h> #include <stdlib.h> #include <stdio.h> +#include <string.h> +#include <errno.h> #include "conf.h" int main(int argc, char **argv) { @@ -25,8 +27,8 @@ int main(int argc, char **argv) { exit(3); } virConfFree(conf); - if (fwrite(buffer, len, 1, stdout) != len) { - fprintf(stderr, "Write failed\n"); + if (fwrite(buffer, 1, len, stdout) != len) { + fprintf(stderr, "Write failed: %s\n", strerror (errno)); exit(1); } exit(0); -- 1.5.3.5.643.g40e25

2 2

[Libvir] Question about specifications of virsh start
by Masayuki Sunou 12 Nov '07

12 Nov '07

Hi I have a question about specifications of virsh start. It is necessary to remove "type=ioemu" from the configuration file of Xen, when HVM domain installed Para Virtualized Driver use unmodified driver on Xen 3.0.3. But,libvirt always adds "type=ioemu" to vif of HVM domain in xm_internal.c. Therefore virsh start cannot start correctly HVM domain installed Para Virtualized Driver. Could you teach why does libvirt disregard setting of configuration file and set type=ioemu ? Thanks, Masayuki Sunou.

3 4

[Libvir] [PATCH] missing printf attributes
by Jim Meyering 08 Nov '07

08 Nov '07

I stumbled across two more missing printf attributes: 2007-11-08 Jim Meyering <meyering(a)redhat.com> * src/virsh.c (vshPrintExtra, vshDebug): Use gcc's printf attribute. Attaching differently, in an attempt to defy mailman-2.1.5:

2 1

[Libvir] Re: [collectd] collectd libvirtstats plugin
by Florian Forster 08 Nov '07

08 Nov '07

Hi Rich, as promised, I've taken a more in-depth look at the `libvirtstats' plugin (I've adopted this as a working title for now and will do the renaming on the weekend or so..) and it looks very good, thanks again :) The `_virDomainInfo' structure has the members `memory' and `maxMem'. Can one calculate `unused' or `free' (or whatever the preferred term in this particular case is) memory using `maxMem - memory'? Would that be an interesting metric? I'm not sure what the best way to store the CPU usage is. Is the `cpuTime' member of the `_virDomainInfo' structure the same as the sum over all `(struct _virVcpuInfo).cpuTime'? Does collecting the total make sense in this case? I'd much rather use the same `type' for ``normal'' CPU statistics and this case which would be much simpler if we didn't collect the total.. I've done some minor things while looking through the code. It basically a mix of coding style and personal convention. Since there's no Debian package yet I didn't install try to compile it yet. I'll install the library and fix the errors I've made as soon as I have some more spare time. The things I've changed are basically: - Rather use `sizeof (array);' than the `DATA_MAX_NAME_LEN' define. - NULL-terminate the strings unconditionally after `strncpy' or `snprintf' was used. - Rather use -- 8< -- if (status != 0) { handle_error (); } -- >8 -- than -- 8< -- if (status == -1) { handle_error (); } -- >8 -- On Mon, Nov 05, 2007 at 02:00:12PM +0000, Richard W.M. Jones wrote: > In the types.db, I have set the max for virt_cpu_total to be > 256,000,000,000. [...] Therefore I have assumed here that > max CPUs = 256. That's basically the reason why I'd rather collect each virtual CPU in a separate file and ``simply'' combine the graphs. This will work for n+1 CPUs ;) > I'm using CSV output, which works, but it seems to report the > aggregate counter values instead of the differences. Is that to be > expected? Yes, that is expected. The changes I've made for the next feature release (4.3) include an easy way to get the rates for a counter value. So I will certainly add an option to store rates instead. Regards, -octo -- Florian octo Forster Hacker in training GnuPG: 0x91523C3D http://verplant.org/

2 1

Re: [Libvir] [PATCH] * src/virsh.c (vshCommandOptInt): Correct indentation
by Jim Meyering 08 Nov '07

08 Nov '07

Well, mailman-2.1.5 mangled yet another of my messages. [ I know why, and have reported the bug. Maybe we'll get a fixed/newer version of mailman. ] But at least the patch got through. Here's the part that it removed: Here's a tiny patch to fix indentation, since I'm about to change the code in that area, I'm submitting this first, so we don't mix syntax-only changes with ones that change semantics. The last two times I tried to send an attachment, mailman removed them. I'll try one more time, with this one: Jim Meyering <meyering(a)redhat.com> wrote: > Signed-off-by: Jim Meyering <meyering(a)redhat.com> > --- > ChangeLog | 4 ++++ > src/virsh.c | 10 +++++----- > 2 files changed, 9 insertions(+), 5 deletions(-) > > diff --git a/ChangeLog b/ChangeLog > index ce68401..a6de485 100644 > --- a/ChangeLog > +++ b/ChangeLog > @@ -1,3 +1,7 @@ > +2007-11-08 Jim Meyering <meyering(a)redhat.com> > + > + * src/virsh.c (vshCommandOptInt): Correct indentation > + > Wed Nov 7 14:38:43 CET 2007 Daniel Veillard <veillard(a)redhat.com> > > * docs/libvir.html docs/site.xsl: small updates with references > diff --git a/src/virsh.c b/src/virsh.c > index 0615a7d..852dea2 100644 > --- a/src/virsh.c > +++ b/src/virsh.c > @@ -1,7 +1,7 @@ > /* > * virsh.c: a Xen shell used to exercise the libvirt API > * > - * Copyright (C) 2005 Red Hat, Inc. > + * Copyright (C) 2005, 2007 Red Hat, Inc. > * > * See COPYING.LIB for the License of this software > * > @@ -3956,10 +3956,10 @@ vshCommandOptInt(vshCmd * cmd, const char *name, int *found) > > if ((arg != NULL) && (arg->data != NULL)) { > res = strtol(arg->data, &end_p, 10); > - if ((arg->data == end_p) || (*end_p!= 0)) > - num_found = FALSE; > - else > - num_found = TRUE; > + if ((arg->data == end_p) || (*end_p!= 0)) > + num_found = FALSE; > + else > + num_found = TRUE; > } > if (found) > *found = num_found;

2 1

[Libvir] fix misleading indentation
by Jim Meyering 08 Nov '07

08 Nov '07

Here's a tiny patch to fix indentation, since I'm about to change the code in that area, I'm submitting this first, so we don't mix syntax-only changes with ones that change semantics. The last two times I tried to send an attachment, mailman removed them. I'll try one more time, with this one:

1 0