[libvirt] Bugfix for RHEL 5.4+ ?? -- Re: Synchronous commands block virsh?
Daniel Veillard
veillard at redhat.com
Mon Feb 1 14:34:46 UTC 2010
On Fri, Jan 22, 2010 at 01:11:01PM -0500, Chris Lalancette wrote:
> On 01/20/2010 06:18 AM, Andreas Kurz wrote:
> > Hi all,
> >
> > Thanks for the information Daniel!
> >
> > On Wednesday 20 January 2010 11:55:41 Daniel P. Berrange wrote:
> >> On Mon, Jan 18, 2010 at 01:52:15PM +0100, Andreas Kurz wrote:
> >>> Hi list,
> >>>
> >>> I found this thread from December:
> >>>
> >>> http://www.mail-archive.com/libvir-list@redhat.com/msg18595.htm
> >>>
> >>> .... that starting with version 0.7.4 libvirtd will no longer block
> >>> other API calls, which would be extremely useful in HA clusters in
> >>> combination with live migration.
> >>>
> >>> Will this fix find its way into RHEL 5?
> >>
> >> The fixes for that are *incredibly* invasive & thus have very high risk
> >> to stability. So that's not the kind of thing that's likely to be approved
> >> for a RHEL5 minor update.
> >
> > I tried to compile libvirtd 0.7.5 for RHEL5 to test it in our lab an ran into
> > major troubles when it comes to KVM integration .... do you see a realistic
> > change that such a new Version is compileable on RHEL5 systems?
>
> Hm, interesting. I just compiled libvirt git head (effectively 0.7.6) on my
> RHEL-5 box here, and it worked just fine with one minor tweak. The minor
> tweak I had to make was to make the qemu driver understand that qemu-kvm
> lives in /usr/libexec. That's the following patch:
>
> diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
> index c227fe1..2ba8366 100644
> --- a/src/qemu/qemu_conf.c
> +++ b/src/qemu/qemu_conf.c
> @@ -797,7 +797,8 @@ qemudCapsInitGuest(virCapsPtr caps,
> */
> if (STREQ(info->arch, hostmachine) ||
> (STREQ(hostmachine, "x86_64") && STREQ(info->arch, "i686"))) {
> - const char *const kvmbins[] = { "/usr/bin/qemu-kvm", /* Fedora */
> + const char *const kvmbins[] = { "/usr/libexec/qemu-kvm", /* RHEL */
> + "/usr/bin/qemu-kvm", /* Fedora */
> "/usr/bin/kvm" }; /* Upstream .spec */
>
> for (i = 0; i < ARRAY_CARDINALITY(kvmbins); ++i) {
>
>
> If that doesn't fix it for you, what kind of error messages are you getting?
I think we should push that patch, but I though we were now using
$PATH to lookup executable. In any case /usr/libexec/ is not in the
default path so this still sounds right,
ACK,
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel at veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/
More information about the libvir-list
mailing list