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

Re: [libvirt] Bugfix for RHEL 5.4+ ?? -- Re: Synchronous commands block virsh?



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?

-- 
Chris Lalancette


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