[PATCH] xen: Strip vcpu_switch_to_aarch64_mode() from PPC and RISC-V stubs

Andrew Cooper posted 1 patch 7 hours ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20260210163445.2796291-1-andrew.cooper3@citrix.com
xen/arch/ppc/stubs.c   | 5 -----
xen/arch/riscv/stubs.c | 5 -----
2 files changed, 10 deletions(-)
[PATCH] xen: Strip vcpu_switch_to_aarch64_mode() from PPC and RISC-V stubs
Posted by Andrew Cooper 7 hours ago
This is absolutely too much copy&paste from ARM.  All other stubs look
somewhat reasonable.

No functional change.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Oleksii Kurochko <oleksii.kurochko@gmail.com>
CC: Timothy Pearson <tpearson@raptorengineering.com>
CC: Anthony PERARD <anthony.perard@vates.tech>
CC: Michal Orzel <michal.orzel@amd.com>
CC: Jan Beulich <jbeulich@suse.com>
CC: Julien Grall <julien@xen.org>
CC: Roger Pau Monné <roger.pau@citrix.com>
CC: Stefano Stabellini <sstabellini@kernel.org>
---
 xen/arch/ppc/stubs.c   | 5 -----
 xen/arch/riscv/stubs.c | 5 -----
 2 files changed, 10 deletions(-)

diff --git a/xen/arch/ppc/stubs.c b/xen/arch/ppc/stubs.c
index f7f6e7ed97af..a333f06119a0 100644
--- a/xen/arch/ppc/stubs.c
+++ b/xen/arch/ppc/stubs.c
@@ -162,11 +162,6 @@ void arch_vcpu_destroy(struct vcpu *v)
     BUG_ON("unimplemented");
 }
 
-void vcpu_switch_to_aarch64_mode(struct vcpu *v)
-{
-    BUG_ON("unimplemented");
-}
-
 int arch_sanitise_domain_config(struct xen_domctl_createdomain *config)
 {
     BUG_ON("unimplemented");
diff --git a/xen/arch/riscv/stubs.c b/xen/arch/riscv/stubs.c
index acbfde79b5a7..d071c8b86d07 100644
--- a/xen/arch/riscv/stubs.c
+++ b/xen/arch/riscv/stubs.c
@@ -126,11 +126,6 @@ void arch_vcpu_destroy(struct vcpu *v)
     BUG_ON("unimplemented");
 }
 
-void vcpu_switch_to_aarch64_mode(struct vcpu *v)
-{
-    BUG_ON("unimplemented");
-}
-
 int arch_sanitise_domain_config(struct xen_domctl_createdomain *config)
 {
     BUG_ON("unimplemented");

base-commit: 2fa468919c39aac189623b6c580ce4ff8592d799
-- 
2.39.5


Re: [PATCH] xen: Strip vcpu_switch_to_aarch64_mode() from PPC and RISC-V stubs
Posted by Oleksii Kurochko 7 hours ago
On 2/10/26 5:34 PM, Andrew Cooper wrote:
> This is absolutely too much copy&paste from ARM.  All other stubs look
> somewhat reasonable.
>
> No functional change.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
> ---
> CC: Oleksii Kurochko <oleksii.kurochko@gmail.com>
> CC: Timothy Pearson <tpearson@raptorengineering.com>
> CC: Anthony PERARD <anthony.perard@vates.tech>
> CC: Michal Orzel <michal.orzel@amd.com>
> CC: Jan Beulich <jbeulich@suse.com>
> CC: Julien Grall <julien@xen.org>
> CC: Roger Pau Monné <roger.pau@citrix.com>
> CC: Stefano Stabellini <sstabellini@kernel.org>
> ---
>   xen/arch/ppc/stubs.c   | 5 -----
>   xen/arch/riscv/stubs.c | 5 -----
>   2 files changed, 10 deletions(-)
>
> diff --git a/xen/arch/ppc/stubs.c b/xen/arch/ppc/stubs.c
> index f7f6e7ed97af..a333f06119a0 100644
> --- a/xen/arch/ppc/stubs.c
> +++ b/xen/arch/ppc/stubs.c
> @@ -162,11 +162,6 @@ void arch_vcpu_destroy(struct vcpu *v)
>       BUG_ON("unimplemented");
>   }
>   
> -void vcpu_switch_to_aarch64_mode(struct vcpu *v)
> -{
> -    BUG_ON("unimplemented");
> -}
> -
>   int arch_sanitise_domain_config(struct xen_domctl_createdomain *config)
>   {
>       BUG_ON("unimplemented");
> diff --git a/xen/arch/riscv/stubs.c b/xen/arch/riscv/stubs.c
> index acbfde79b5a7..d071c8b86d07 100644
> --- a/xen/arch/riscv/stubs.c
> +++ b/xen/arch/riscv/stubs.c
> @@ -126,11 +126,6 @@ void arch_vcpu_destroy(struct vcpu *v)
>       BUG_ON("unimplemented");
>   }
>   
> -void vcpu_switch_to_aarch64_mode(struct vcpu *v)
> -{
> -    BUG_ON("unimplemented");
> -}
> -

These changes looks good to me:
  Reviewed-by: Oleksii Kurochko <oleksii.kurochko@gmail.com>

Thanks for cleaning it up.

~ Oleksii