All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bertrand Marquis <Bertrand.Marquis@arm.com>
To: Julien Grall <julien@xen.org>
Cc: Luca Fancellu <Luca.Fancellu@arm.com>,
	Xen-devel <xen-devel@lists.xenproject.org>,
	Wei Chen <Wei.Chen@arm.com>,
	Stefano Stabellini <sstabellini@kernel.org>,
	Volodymyr Babchuk <Volodymyr_Babchuk@epam.com>,
	Andrew Cooper <andrew.cooper3@citrix.com>,
	George Dunlap <george.dunlap@citrix.com>,
	Jan Beulich <jbeulich@suse.com>, Wei Liu <wl@xen.org>,
	Juergen Gross <jgross@suse.com>,
	Dario Faggioli <dfaggioli@suse.com>
Subject: Re: [PATCH v3 4/6] xen/cpupool: Create different cpupools at boot time
Date: Tue, 22 Mar 2022 08:35:08 +0000	[thread overview]
Message-ID: <9ABBD292-A77F-4CDC-B096-546516114D42@arm.com> (raw)
In-Reply-To: <108443e0-c7d7-af6d-2183-b1bd48b0f288@xen.org>

Hi Julien,

> On 21 Mar 2022, at 18:44, Julien Grall <julien@xen.org> wrote:
> 
> Hi Bertrand,
> 
> On 21/03/2022 17:19, Bertrand Marquis wrote:
>>> On 21 Mar 2022, at 17:36, Julien Grall <julien@xen.org> wrote:
>>>> So I don’t know why on x86 we must have cpu0 in cpupool0, maybe x86 maintainer have more knowledge about that and
>>>> I can put a comment here.
>>> 
>>> On Arm, we are not yet supporting all the CPU features that x86 supports (e.g. CPU hotplug, suspend/resume...). So I a am bit concerned that the restriction is just not there yet (or possibly hidden).
>>> 
>>> Therefore, before lifting the restriction on Arm (and other arch), I would like us to understand why it is necessary on x86.
>>> 
>>> We may not have the answer quickly, so is it going to be a problem to keep the restriction on Arm?
>> I am ok to keep the limitation to have dom0 always running on cpu0.
>> Only limitation I can see is that on a big little system, dom0 needs to stay on the type of core of the first booted core.
> 
> Where does this limitation come from?

If dom0 must run on core0 and core0 is Little then you cannot build a system where dom0 is running on big cores.
If the limitation is not there, you can build such a configuration without any dependency to the boot core type.

Cheers
Bertrand

> 
> Cheers,
> 
> -- 
> Julien Grall


  reply	other threads:[~2022-03-22  8:35 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-18 15:25 [PATCH v3 0/6] Boot time cpupools Luca Fancellu
2022-03-18 15:25 ` [PATCH v3 1/6] tools/cpupools: Give a name to unnamed cpupools Luca Fancellu
2022-03-18 15:25 ` [PATCH v3 2/6] xen/sched: create public function for cpupools creation Luca Fancellu
2022-03-18 15:25 ` [PATCH v3 3/6] xen/sched: retrieve scheduler id by name Luca Fancellu
2022-03-18 15:25 ` [PATCH v3 4/6] xen/cpupool: Create different cpupools at boot time Luca Fancellu
2022-03-18 16:12   ` Julien Grall
2022-03-21 15:58     ` Luca Fancellu
2022-03-21 16:36       ` Julien Grall
2022-03-21 17:19         ` Bertrand Marquis
2022-03-21 17:44           ` Julien Grall
2022-03-22  8:35             ` Bertrand Marquis [this message]
2022-03-22  8:47               ` Bertrand Marquis
2022-03-22  9:16                 ` Julien Grall
2022-03-22  9:28                   ` Bertrand Marquis
2022-03-22  9:52         ` Luca Fancellu
2022-03-22 14:01           ` Julien Grall
2022-03-23 13:58             ` Luca Fancellu
2022-03-23 18:53               ` Julien Grall
2022-03-21  9:10   ` Jan Beulich
2022-03-21 15:51     ` Luca Fancellu
2022-03-21 23:45   ` Stefano Stabellini
2022-03-22  8:49     ` Luca Fancellu
2022-03-22 17:40       ` Stefano Stabellini
2022-03-18 15:25 ` [PATCH v3 5/6] arm/dom0less: assign dom0less guests to cpupools Luca Fancellu
2022-03-18 16:18   ` Julien Grall
2022-03-22 17:19     ` Luca Fancellu
2022-03-21  9:04   ` Jan Beulich
2022-03-21 23:45   ` Stefano Stabellini
2022-03-18 15:25 ` [PATCH v3 6/6] xen/cpupool: Allow cpupool0 to use different scheduler Luca Fancellu

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=9ABBD292-A77F-4CDC-B096-546516114D42@arm.com \
    --to=bertrand.marquis@arm.com \
    --cc=Luca.Fancellu@arm.com \
    --cc=Volodymyr_Babchuk@epam.com \
    --cc=Wei.Chen@arm.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=dfaggioli@suse.com \
    --cc=george.dunlap@citrix.com \
    --cc=jbeulich@suse.com \
    --cc=jgross@suse.com \
    --cc=julien@xen.org \
    --cc=sstabellini@kernel.org \
    --cc=wl@xen.org \
    --cc=xen-devel@lists.xenproject.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.