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] libxl: use named options for tsc_mode

To: Ian Campbell <ian.campbell@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: RE: [Xen-devel] [PATCH] libxl: use named options for tsc_mode
From: Dan Magenheimer <dan.magenheimer@xxxxxxxxxx>
Date: Thu, 10 Nov 2011 09:29:35 -0800 (PST)
Cc: ian.jackson@xxxxxxxxxx
Delivery-date: Thu, 10 Nov 2011 09:31:53 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <bc79b560aafa1e4dc42a.1320922519@xxxxxxxxxxxxxxxxxxxxxxxxx>
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>
References: <bc79b560aafa1e4dc42a.1320922519@xxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
> From: Ian Campbell [mailto:ian.campbell@xxxxxxxxxx]
> Subject: [Xen-devel] [PATCH] libxl: use named options for tsc_mode
> 
> # HG changeset patch
> # User Ian Campbell <ian.campbell@xxxxxxxxxx>
> # Date 1320922479 0
> # Node ID bc79b560aafa1e4dc42af00e6a326dc651b5636a
> # Parent  460b507e15f864dd6712f5040e36538d6e076ae4
> libxl: use named options for tsc_mode.
> 
> It seems that this knob is expoerted from the hypervisor as a raw
> integer (no symbolic names) documented in xen/include/asm-x86. Propagating 
> that
> all the way to the end user is hardly friendly (it's bad enough in the
> hypercall interface).

Thanks for looking at this!

> Add an enum at the libxl level with a hopefully descriptive set of names.
> Deprecate the use of an integer in xl cfg files.

We (Oracle) already have shipped cfg files that use the integer
so would prefer that the deprecation message be removed.  IMHO, to
the vast majority of users, the symbolic names will be gibberish
anyway and are likely to be misspelled.  For knowledgeable folk,
they are nice though, so maybe just support both?
 
> +   * `"always_emulate"`: guest rdtsc/p always emulated at 1GHz (kernel
> +     and user).

Many guests will "appear" to run at a slower "bogomips" so a word
or two more here may keep some from panicking.  Maybe:

guest rdtsc/p always emulated and the virtual TSC will appear to
increment (kernel and user) at a fixed 1GHz rate, regardless of
the PCPU HZ rate or power state; this will NOT affect underlying
CPU performance

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

<Prev in Thread] Current Thread [Next in Thread>