Re: [ANNOUNCE] kvm-79 release

[Avi Kivity <avi@redhat.com>]

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