[Xen-devel] Re: [PATCH][v7] PV extension of HVM(hybrid) support in Xen

[Tim Deegan <Tim.Deegan@xxxxxxxxxx>]

At 02:11 +0000 on 11 Mar (1268273506), Sheng Yang wrote:
> > Shouldn't this be gated on (d->hvm_pv_enabled & XEN_HVM_PV_CLOCK_ENABLED)
> > rather than (eip != 0)?
> 
> Um... I think no other HVM should call this, and evtchn shouldn't work
> without PV clocksource... And it have two meaning here: one is setting
> up the start up IP for AP, another is initial PV clock.  I would
> update this to get it more clear.

Yes, I was just talking about the last line, where you set the offset to
zero.  Since you have introduced a flag that says whether the
clocksource is enabled, you should use it. 
 
> > Maybe use cpuid_edx() here?
> 
> Um? It's not native cpuid, but the one configuration file specific.

Yes, you're quite right.  I misread the code. 

Thanks, 

Tim.

-- 
Tim Deegan <Tim.Deegan@xxxxxxxxxx>
Principal Software Engineer, XenServer Engineering
Citrix Systems UK Ltd.  (Company #02937203, SL9 0BG)

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel