c/s 20255 passed my test, too.
Thanks a lot,
Kouya
Christoph Egger writes:
> On Friday 25 September 2009 16:22:39 Keir Fraser wrote:
> > Applied as c/s 20255. I messed with it some more, so perhaps worth checking
> > the bug still is gone. ;-)
>
> I retested and the bug is still gone.
>
> Christoph
>
> > -- Keir
> >
> > On 25/09/2009 13:25, "Kouya Shimura" <kouya@xxxxxxxxxxxxxx> wrote:
> > > Keir,
> > > Okay, attached is a new patch per your advice.
> > > I tested it carefully. It works for me.
> > >
> > > Christoph,
> > > Could you please test it?
> > >
> > > Thanks,
> > > Kouya
> > >
> > > Keir Fraser writes:
> > >> On 25/09/2009 10:32, "Kouya Shimura" <kouya@xxxxxxxxxxxxxx> wrote:
> > >>> I've forgotten VCPU switch. Indeed, it's pretty fragile. :-)
> > >>> What about modifying pt_restore_timer() not to advance pt->scheduled?
> > >>> We should rename pending_intr_nr to pending_intr(boolean).
> > >>
> > >> It's not clear to me that the old logic around pt->pending_intr_nr and
> > >> pt->scheduled really needs to change that drastically. The more we mess
> > >> with this code the more likely we are to break stuff, as it's hard to
> > >> get good coverage of the timer_modes and various ways guest Oses may
> > >> manage their timers.
> > >>
> > >> I would prefer to see whether moving the update of pt->scheduled out of
> > >> pt_intr_post() and back into the timer_fn() works. That moves us back
> > >> towards what we had before your patches, and therefore I'm more
> > >> comfortable with it.
> > >>
> > >> -- Keir
> > >
> > > Signed-off-by: Kouya Shimura <kouya@xxxxxxxxxxxxxx>
>
>
>
> --
> ---to satisfy European Law for business letters:
> Advanced Micro Devices GmbH
> Karl-Hammerschmidt-Str. 34, 85609 Dornach b. Muenchen
> Geschaeftsfuehrer: Andrew Bowd, Thomas M. McCoy, Giuliano Meroni
> Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
> Registergericht Muenchen, HRB Nr. 43632
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
Home