WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: "Xu, Dongxiao" <dongxiao.xu@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: RE: [Xen-devel][PATCH 01/02] pvrdtscp: move the place of write_rdtscp_aux()
From: Dan Magenheimer <dan.magenheimer@xxxxxxxxxx>
Date: Fri, 11 Dec 2009 06:51:01 -0800 (PST)
Cc: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Delivery-date: Fri, 11 Dec 2009 06:52:46 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <EADF0A36011179459010BDF5142A457501D13FE7E8@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
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.

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