[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 0/5] Implement CPU hotplug on Arm
- To: Mykyta Poturai <Mykyta_Poturai@xxxxxxxx>
- From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
- Date: Tue, 3 Feb 2026 16:35:07 +0000
- Accept-language: en-GB, en-US
- Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 4.158.2.129) smtp.rcpttodomain=epam.com smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=arm.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
- 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=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=5Dsx8fcTzIpeGLNGtyA/G+vgUB/g184wbUSnT6QPl4c=; b=rjeviEIkO3LpFoOhe7n5HIBEUwu2GwvCKDn3jXga9qgmRxYb3Z3g+ymxKj3+KQkgdFwMk9ua+GRjwbtynmOMYCHBRU4sfPEDW3oCU1nalHlrALrY5CNnR1o5zaXkgkxcz4l2dmpCP/44lB7KH0KiFp+F7xCb1tuIK0av6LYEnCuJDohSHLk4FE3umMfw+e3YBaaKAa/JcV5CAfoSN1QjeMksSFwV5i+paf39EZsMmGSdTWGTZjvJlyKdTLH/kNWzHo+qhhCXDuG8m+LCA5/CONQwgMPugwsfoehDaZwD0WN7VbPRTFKiysEi1m+MEo4MNWf9PVE6SIs0mJHlpunMsA==
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=5Dsx8fcTzIpeGLNGtyA/G+vgUB/g184wbUSnT6QPl4c=; b=K/3TO6Zj5Hmv3NcqThYhIVpeqKnHOUN8hqJ6pOkkx+u+FsmGyV8vjg51TGxzx/IK8khZYvo90e/IpR/HPwcbfcxgqGy1rbRJ8lkLt/VDFB1bdYw7xPfIINAGLxd5CTkX6CVZ35OEnlCPj1f1e+S8Xuq8ma8Y509lEz1YKJHZBpK19KodmpaR+LRPmfzdupjIshoduWbxnzT5Za8LV6wS7FP4DpYsUFWxXLoLr5hLIQyJwXHeSUcnbOOcoIH9RdHVV1vwH5BcsOFXKOSIT4wzQFfmW/atYukamTJyL0Ywf8D2Ra5XJYU2VpPB5meKnNtM34pn4BrUuuYE8JZozKd7YA==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=pass; b=uWFc3MevPKsWXbfbSuQwqx+6ivFXsAeJiwSu3DA89jqGUx4gHWTK+EbKCR9mhWPCYQKVQlugXbUmBBbImOLJFSBQFaH4ypaqEzFRyz2E4T3RDm3jl5hV02IkKjJDPNCJpLMo217uMofLVx0OMullhoIOI/eqxkroGvGRIvy7yOXmK5mUHRTBZde24dmq2w4VRQM70Snc/nMohIJjBCbbthvwCQURzxRyvOfndPKMPBg07fX4bfno6tb0YnlJdg1Li7gQC+m/gcuAYXNJVohFQvr+CRFSVRT0VZjmK65z0GBh9KXRlj0xb3HTtHqRyfMmjQtLiWu6m8+7gm2oIOCtkw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=nO6giaH8NnJDm7bQNx5PN87eQw+2ZtZIBuj9zo9FmeRgKksd6UFGoWDIuk+n2yqJY2IcsDDp5nPB4/OdazDpmhpEBbjFbRe/7771EJrYdaRZYWGRwRSBddyhgqxj3Y4cWycxaDoMHGbj6innM3HTog6WpFt3JhmRaR2g6r4ybumxMD0OfDx0EdIKs8qyjqZkzovvA14L0ys5wxHPYnxG2IJF8z2fRANTM2hs/FhpWKZyJjoyAaFYOukgUM7zMMujFayrBXaBFqxz530LGn1Rx9Uh/47g7o0zDlop8H4+Uf7K7Q7X1Ko9nUubOknfZW5EfZ3nFKqPBxMn4GDYuNX+5A==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Timothy Pearson <tpearson@xxxxxxxxxxxxxxxxxxxxx>, Alistair Francis <alistair.francis@xxxxxxx>, Connor Davis <connojdavis@xxxxxxxxx>, Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>, "Daniel P. Smith" <dpsmith@xxxxxxxxxxxxxxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>
- Delivery-date: Tue, 03 Feb 2026 16:36:38 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Nodisclaimer: true
- Thread-index: AQHchGj6Bf8iLxj9KkKeGyrpjQwglLVxTT0A
- Thread-topic: [PATCH v5 0/5] Implement CPU hotplug on Arm
Hi Mykyta,
> On 13 Jan 2026, at 09:45, Mykyta Poturai <Mykyta_Poturai@xxxxxxxx> wrote:
>
> This series implements support for CPU hotplug/unplug on Arm. To achieve this,
> several things need to be done:
>
> 1. XEN_SYSCTL_CPU_HOTPLUG_* calls implemented on Arm64.
> 2. Enabled building of xen-hptool.
> 3. Migration of irqs from dying CPUs implemented.
>
> Tested on QEMU.
Do you have a way to validate this on real hardware ?
Doing hotplug/unplug might have side effects (caches, coherency, interrupts,
etc) that
I would not be completely confident to have validated by only testing this in
qemu.
Could you explain how you validated this (what kind of tests you did).
I would be interested to know how the system behave in the following cases:
- stopping cpu with guests pinned to specific cores
- stopping a cpu with a guest pinned and only running on that one
- checking with passthrough devices (for irq balancing)
Then a general remark i have is that most of this code (if not all) is always
in Xen
even if CONFIG_CPU_HOTPLUG is not activated in xen config which will probably
generate some dead code. Could we protect some of the code not used when
HOTPLUG is not activated with if (is_enabled(CONFIG_CPU_HOTPLUG)) or
equivalent forms ?
Cheers
Bertrand
>
> v4->v5:
> * drop merged patches
> * combine "smp: Move cpu_up/down helpers to common code" with
> "arm/sysctl: Implement cpu hotplug ops"
> * see individual patches
>
> v3->v4:
> * add irq migration patches
> * see individual patches
>
> v2->v3:
> * add docs
>
> v1->v2:
> * see individual patches
>
> Mykyta Poturai (5):
> arm/irq: Keep track of irq affinities
> arm/irq: Migrate IRQs during CPU up/down operations
> arm/sysctl: Implement cpu hotplug ops
> tools: Allow building xen-hptool without CONFIG_MIGRATE
> docs: Document CPU hotplug
>
> SUPPORT.md | 1 +
> docs/misc/cpu-hotplug.txt | 50 +++++++++++++++++++++++++
> tools/libs/guest/Makefile.common | 2 +-
> tools/misc/Makefile | 2 +-
> xen/arch/arm/Kconfig | 1 +
> xen/arch/arm/gic-vgic.c | 2 +
> xen/arch/arm/include/asm/irq.h | 2 +
> xen/arch/arm/irq.c | 63 +++++++++++++++++++++++++++++++-
> xen/arch/arm/smp.c | 9 +++++
> xen/arch/arm/smpboot.c | 6 +++
> xen/arch/arm/vgic.c | 14 ++++++-
> xen/arch/ppc/stubs.c | 4 ++
> xen/arch/riscv/stubs.c | 5 +++
> xen/arch/x86/Kconfig | 1 +
> xen/arch/x86/include/asm/smp.h | 3 --
> xen/arch/x86/smp.c | 33 ++---------------
> xen/arch/x86/sysctl.c | 12 ++----
> xen/common/Kconfig | 3 ++
> xen/common/smp.c | 34 +++++++++++++++++
> xen/common/sysctl.c | 45 +++++++++++++++++++++++
> xen/include/xen/smp.h | 4 ++
> 21 files changed, 248 insertions(+), 48 deletions(-)
> create mode 100644 docs/misc/cpu-hotplug.txt
>
> --
> 2.51.2
|
