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] [RFC] [PATCH] use "reliable" tsc properly when available

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, "Xen-Devel (E-mail)" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [RFC] [PATCH] use "reliable" tsc properly when available, but verify
From: Dan Magenheimer <dan.magenheimer@xxxxxxxxxx>
Date: Tue, 29 Sep 2009 11:45:49 -0700 (PDT)
Cc:
Delivery-date: Tue, 29 Sep 2009 11:46:30 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C6E80E9D.16004%keir.fraser@xxxxxxxxxxxxx>
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
> > I'd feel better running check_tsc_warp (at least) once
> > on ALL processors, even if X86_FEATURE_TSC_RELIABLE
> > is set.  Is that OK?  Or should we just trust it under
> > exactly the same circumstances as Linux does?
> 
> Well, what does trusting it imply for us right now? Do you 
> have another
> patch in mind, even if we decide against check_tsc_warp?
> 
> I mean, I'm not unhappy with our TSC management code as it is. I'm not
> greatly excited to mess with it yet again.

Well, if it is indeed reliable, there's no reason that
Xen should write_tsc, though it might be interesting
to check_tsc_warp on various hardware before and after
Xen does write_tsc (to see if Xen is breaking anything).

But I could certainly code the patch to just "measure
and report" for now and we could revisit Xen's use of
write_tsc at a later time if that's your preference.

I definitely have more patches in mind for pvrdtsc but
may not get back to them for a few weeks, so was
hoping to validate tsc reliability on some machines
in the meantime.

Thanks,
Dan

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