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

Re: [PATCH v6 4/6] xen/cpupool: Create different cpupools at boot time


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
  • Date: Fri, 8 Apr 2022 09:06:13 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=HzI+bN9oppV40ggV3ApoXTUWt1JVb6XB+aCZy2YBP7k=; b=ks9ShoUsJr3JYbcRm5FM5kuA5sByTNE4X+JOHOJJLYaj3R2ddxAHALwAc1p0SsWn4y0ChjBqif/cSNyEfRGz7FDM6PiySya1Yim+HQ3E0W/txmeGzrdbOuJFiEALr8wL7MQ3TFa9MjvmXkTfdOGeJb9w7qXU1kM0uiQqU6tjlg0yyiNPnid2xzcK5fk38yc5TJAimSK69EE7/9vVg/VjW9cNYWa4Dp0Um095CXuvf6QWYe5tv6WayOD9WhIxbpsxsQcWagxsjcjEkz1UIwQK9dcb2NoYbk1Cdui/XsZ6UquHGdluiaJBRPy9dRX0M306Tm0GZC2b4IkI9v/h7r2WfQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=M48TuKoFofsoc/vPdUCeIW+6RsHunDpeAl2r+vvZTwb986O9kfP/b3Arx5ZHDTdknFqs0b8NULdcNr7b6Tzp44sNhCQbpQEUyddkdPUejwje1Br+yjaaLZSpvKsit3867l8sX7MTd8nF8IfxWQX/I9PMXXk9CLXinO1yZC1lsrxTmf6/dubD4DaQ1ME+oY2ME36xrU917AAy8uXaYekoGbpQjo0Z776+qKfjCuos0ulOWVccJS0IVb0aHNXtQi6+ZM+78ldv/HozTTwQtOehI0eeNkoJewOR4u/Lhx81j0JEFyoh+4naKWnvG3s8ktQyrmcZyJo3HFpfxq1QYdeXog==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Wei Chen <Wei.Chen@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Dario Faggioli <dfaggioli@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Fri, 08 Apr 2022 09:06:35 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHYSyUg2T6WkJDiXECW3myZV3L//qzltq2AgAAC0gA=
  • Thread-topic: [PATCH v6 4/6] xen/cpupool: Create different cpupools at boot time


> On 8 Apr 2022, at 09:56, Jan Beulich <jbeulich@xxxxxxxx> wrote:
> 
> On 08.04.2022 10:45, Luca Fancellu wrote:
>> Introduce a way to create different cpupools at boot time, this is
>> particularly useful on ARM big.LITTLE system where there might be the
>> need to have different cpupools for each type of core, but also
>> systems using NUMA can have different cpu pools for each node.
>> 
>> The feature on arm relies on a specification of the cpupools from the
>> device tree to build pools and assign cpus to them.
>> 
>> ACPI is not supported for this feature.
>> 
>> Documentation is created to explain the feature.
>> 
>> Signed-off-by: Luca Fancellu <luca.fancellu@xxxxxxx>
>> Reviewed-by: Juergen Gross <jgross@xxxxxxxx>
> 
> This looks to be not in line with ...
> 
>> ---
>> Changes in v6:
>> - Changed docs, return if booted with ACPI in btcpupools_dtb_parse,
>> panic if /chosen does not exists. Changed commit message (Julien)
>> - Add Juergen R-by for the xen/common/sched part that didn't change
> 
> ... what you say here. What's the scope of Jürgen's R-b? If it has
> restricted scope, you need to retain that restriction for committers
> to know.

Hi Jan,

Sorry about that, I’ve just refreshed my memory with sending-patches.pandoc and 
I see
I should have added Juergen's R-by with # area.

It’s the first time I retain an R-by for just a part of the commit, I will 
remember it for the next
time.

Cheers,
Luca 

> 
> Jan


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.