xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Keir Fraser <keir.xen@gmail.com>
To: "Tian, Kevin" <kevin.tian@intel.com>,
	Juergen Gross <juergen.gross@ts.fujitsu.com>
Cc: "Liu, Jinsong" <jinsong.liu@intel.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	"mark.langsdorf@amd.com" <mark.langsdorf@amd.com>
Subject: Re: [PATCH] use per-cpu variables in cpufreq
Date: Tue, 31 May 2011 08:31:20 +0100	[thread overview]
Message-ID: <CA0A57D8.1B4DE%keir.xen@gmail.com> (raw)
In-Reply-To: <625BA99ED14B2D499DC4E29D8138F1505CA61C06B3@shsmsx502.ccr.corp.intel.com>

On 31/05/2011 02:51, "Tian, Kevin" <kevin.tian@intel.com> wrote:

>> 
>> Well, there is a difference between logical and physical cpu hotplug. Xen is
>> capable of bringing CPUs online and offline without them actually being
>> physically plugged/unplugged from the mainboard. Indeed our physical hotplug
>> support is relatively new and I would suspect not much used (and it supports
>> only physical insertion, not removal!).
>> 
>> Frankly there are a number of questions around CPUs that are physically
>> plugged in after boot:
>>  * How does per-CPU ACPI state like PM info get set up?
> 
> A hotplug-able cpu still has an ACPI CPU object in DSDT table, which may exist
> in original DSDT table, or dynamically loaded later upon hotplug event. Once
> dom0 ACPI recognizes a new CPU object, it will notify Xen about discovered
> pm information.

Thanks. I guessed there was probably some magic for this.

 -- Keir

>>  * In a system where TSCs are otherwise all perfectly in sync, does the
>> firmware help us by setting up the new CPUs' TSCs likewise?
> 
> Here what firmware can do is similar to what Xen can do, which can't ensure
> you a truly synchronized TSC on new CPU.
> 
> Thanks
> Kevin
> 

  reply	other threads:[~2011-05-31  7:31 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-27 11:11 [PATCH] use per-cpu variables in cpufreq Juergen Gross
2011-05-27 13:11 ` Keir Fraser
2011-05-27 13:29   ` Juergen Gross
2011-05-28  7:52     ` Keir Fraser
2011-05-30  5:47       ` Juergen Gross
2011-05-30  9:45         ` Keir Fraser
2011-05-31  1:51           ` Tian, Kevin
2011-05-31  7:31             ` Keir Fraser [this message]
2011-05-31  7:37             ` Liu, Jinsong
2011-05-30  8:06       ` Tian, Kevin
2011-05-30 15:33         ` Liu, Jinsong
2011-06-10 19:00 ` Langsdorf, Mark
2011-06-14  9:04   ` Juergen Gross

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CA0A57D8.1B4DE%keir.xen@gmail.com \
    --to=keir.xen@gmail.com \
    --cc=jinsong.liu@intel.com \
    --cc=juergen.gross@ts.fujitsu.com \
    --cc=kevin.tian@intel.com \
    --cc=mark.langsdorf@amd.com \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).