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

Re: [Xen-devel] [PATCH] softtsc for PV domains


  • To: Dan Magenheimer <dan.magenheimer@xxxxxxxxxx>, "Xen-Devel (E-mail)" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
  • Date: Thu, 27 Aug 2009 17:00:54 +0100
  • Cc:
  • Delivery-date: Thu, 27 Aug 2009 09:01:29 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AconLM4SGyD35hZJSA+xtJljnNSXlwAAsBZd
  • Thread-topic: [Xen-devel] [PATCH] softtsc for PV domains

On 27/08/2009 16:40, "Dan Magenheimer" <dan.magenheimer@xxxxxxxxxx> wrote:

>> Changeset 20128 is a fixed and simplified version of this.
>> Please take a
>> look and test.

> Looks good and seems to work.  I'm not sure why you wanted
> to move the frequency info from the "t" debug key to the
> new "s" debug key... and I'm a bit worried that since the "s"
> key only registers if booted with softtsc, someone else will
> use the "s" key for something else and cause a conflict
> that won't be noticed until later.

The arch_ func thing was ugly and also broke ia64.

It'd be better probably to collect stats per-domain and then hook into
arch_dump_domain_info().

 -- Keir



_______________________________________________
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®.