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

RE: [Xen-devel][PATCH 01/02] pvrdtscp: move the place of write_rdtscp_aux()



Dan Magenheimer wrote:
> This patch is wrong for TSC_MODE_PVRDTSCP.  The original
> code should be left in __update_vcpu_system_time and
> the context switch code should write to TSC_AUX only
> if tsc_mode is *NOT* TSC_MODE_PVRDTSCP because the
> implementation of full rdtscp functionality and
> TSC_MODE_PVRDTSCP mode are incompatible.

Could you detail the wrong place? Appreciate for this!
In my patch, I have done save/restore for guest/host 
TSC_AUX MSR, so it will not break PVRDTSC logic. 
 
> 
> But I continue to question if this should be done at all.
> See other thread.
> 
>> -----Original Message-----
>> From: Xu, Dongxiao [mailto:dongxiao.xu@xxxxxxxxx]
>> Sent: Friday, December 11, 2009 4:54 AM
>> To: xen-devel@xxxxxxxxxxxxxxxxxxx
>> Cc: Keir Fraser
>> Subject: [Xen-devel][PATCH 01/02] pvrdtscp: move the place of
>> write_rdtscp_aux() 
>> 
>> 
>>  Pvrdtscp: move write_rdtscp_aux() to paravirt_ctxt_switch_to()
>>  - Currently write_rdtscp_aux() is placed in
>> update_vcpu_system_time(), which
>> is called by schedule() before context_switch(). This will
>> break the HVM guest
>> TSC_AUX state because at this point, MSR hasn't beed saved
>> for HVM guests.So
>> put the function in the point when a PV vcpu is really scheduled in.
>> 
>> Signed-off-by: Dongxiao Xu <dongxiao.xu@xxxxxxxxx>
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.