[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: Luca Fancellu <luca.fancellu@xxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Fri, 8 Apr 2022 11:01:46 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.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=2l5XVfc3HnYN8zk/7bqg4CjtKczWdpVE3hLQeczu80Q=; b=fpOExsJSSI1nd3sMf1qsfxm/GIqskvsRSNcJCuBVgdhNeXs+0jjMeY/2KOHP87fWOok23npVPPBWq3bs7LOkNtnYrem0U60Lbd59vhu5MCskIZzemvLoSWRwvb6u+ZaXG8kyxEz7D+Odj6iLLMlkVMqBL7VFnt0yIGtH5G6gPGy/2iFPoMvyZ0otvoCqYoHm/RukkpEvGfYpumPWdJjb53kdx+pT4sszxlDCB7sq1PEXwMeLUnxClMR8UNFnAbhMcGWykNBNbKJE0nFhB2NqNikErdKvUpJR/MdaG0JCDTcy+PglSOvQyMsFybWZEQZXh4NBPYgobAn8rK0LfHSTXw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=kUd+0J/HFdQRI2AGQ+s2o6fIerCPFwAVWHthepyg9s6FfBJz9mPIwRu/V1kGtgYe9QH7YZGP4vqkZwOTvNJW95nTrlEVluOAgzO44kdcHmV4abmEFlZfMABkkD3DZ273ULgbJClCdO6PjoVe5AFsymVMv7lD6z15ukne9XTKCozypn45yoeEwt8+6bBUg0iwxyTUtNXvIiSgk2e08KVo+r+irALJoSyIqo905NxtOQpg+TgCgluQbBV1OcE5QaXch/NLcle/7E8/caP7Pnl+/CKfRRKWPhF0/sVKXahYQqtWa7O2eEmZXbPySM43cHP4x4y0l/C4dFPgmfMxGQiP2w==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: bertrand.marquis@xxxxxxx, 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
  • Delivery-date: Fri, 08 Apr 2022 09:02:17 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 08.04.2022 10:45, Luca Fancellu wrote:
> ---
>  docs/misc/arm/device-tree/cpupools.txt | 140 +++++++++++++++++
>  xen/arch/arm/include/asm/smp.h         |   3 +
>  xen/common/Kconfig                     |   7 +
>  xen/common/Makefile                    |   1 +
>  xen/common/boot_cpupools.c             | 207 +++++++++++++++++++++++++
>  xen/common/sched/cpupool.c             |  12 +-
>  xen/include/xen/sched.h                |  14 ++
>  7 files changed, 383 insertions(+), 1 deletion(-)
>  create mode 100644 docs/misc/arm/device-tree/cpupools.txt
>  create mode 100644 xen/common/boot_cpupools.c

Under whose maintainership is the new file to fall? Without an
addition to ./MAINTAINERS and without the file being placed in
xen/common/sched/, it'll be REST maintainers, which I think would
better be avoided. Would it perhaps make sense to have this as
xen/common/sched/boot.c, allowing other boot-only code to
potentially be moved there over time? This would then also avoid
me asking about the underscore in the file name: Underscores are
a somewhat artificial thing for use in places where dashes can't
be used. Yet in the file system dashes are fine, and dashes are
(slightly) easier to type.

Jan




 


Rackspace

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