[libvirt] [PATCH] Don't list capabilities entries if emulators can't be accessed
Daniel P. Berrange
berrange at redhat.com
Thu Aug 28 10:52:43 UTC 2008
On Mon, Aug 25, 2008 at 01:16:39PM -0400, Cole Robinson wrote:
> Daniel P. Berrange wrote:
> > On Wed, Aug 20, 2008 at 12:51:05PM -0400, Cole Robinson wrote:
> >
> >> diff --git a/src/qemu_conf.c b/src/qemu_conf.c
> >> index dc9e42a..0328cc1 100644
> >> --- a/src/qemu_conf.c
> >> +++ b/src/qemu_conf.c
> >> @@ -230,6 +230,10 @@ qemudCapsInitGuest(virCapsPtr caps,
> >> virCapsGuestPtr guest;
> >> int i;
> >>
> >> + /* Check for existance of base emulator */
> >> + if (access(info->binary, X_OK) == -1)
> >> + return 0;
> >> +
> >
> > This isn't right - this means that if KVM is installed, but QEMU is
> > not installed you won't get any capabilities.
> >
> > Basically we need todo all the access() checks for QEMU, KVM, /dev/kvm
> > up-front. And then generated the capabilites if either QEMU or KVM is
> > available.
> >
>
> Okay, I think this patch solves the issues. We check upfront
> for the base emulator and potential kvm emulators (qemu-kvm,
> and /usr/bin/kvm for ubuntu/upstream .spec). If nothing is
> found, just return. If only the base emulator is found, skip
> kvm even if /dev/kvm exists. If only kvm bin is found, add
> the base emulator capabilities only if emulator and host arch
> matches.
ACK, this looks good now.
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