Martin Vogt wrote:
Hello,


[...]
Notes:
If you use the modules bundled with kvm-79, you can use any version
of Linux from 2.6.16 upwards.

Nope.
I tested kvm-79 on a SLES 10 with 2.6.16,
in the hope that my previously reported bug:

"hrtimer_start: Unknown symbol in module"

was fixed. (It wasnt)

Now I have this "one-liner patch(tm)" here which works for me:


diff -u external-module-compat-comm.h.org external-module-compat-comm.h
--- external-module-compat-comm.h.org   2008-11-12 15:51:55.000000000 +0100
+++ external-module-compat-comm.h       2008-11-12 15:52:05.000000000 +0100
@@ -601,7 +601,7 @@

 static inline void hrtimer_start_expires(struct hrtimer *timer, int mode)
 {
-       hrtimer_start(timer, timer->expires, mode);
+       hrtimer_start_p(timer, timer->expires, mode);
 }



Is this a correct fix?


It is correct -- I added hrtimer_start_expires without taking 2.6.16 into account.

- I could boot suse 11.1 beta4 with it(which didnt work with kvm-77)

Im really not sure if this works corretly and I dont want have broken
modules in my kernel :(

Is hrtimer_start_p already initialized when  hrtimer_start_expires is
called? (help)

Yes, hrtimer_start_p is initialized very early on.

Please send a signed-off-by line and I will apply.


--
error compiling committee.c: too many arguments to function

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to