[Public] > -----Original Message----- > From: Jan Beulich <[email protected]> > Sent: Wednesday, September 10, 2025 11:14 PM > To: Penny, Zheng <[email protected]> > Cc: Huang, Ray <[email protected]>; Stefano Stabellini > <[email protected]>; Julien Grall <[email protected]>; Bertrand Marquis > <[email protected]>; Orzel, Michal <[email protected]>; > Volodymyr Babchuk <[email protected]>; Andrew Cooper > <[email protected]>; Anthony PERARD <[email protected]>; > Roger Pau Monné <[email protected]>; Christopher Clark > <[email protected]>; Daniel P. Smith > <[email protected]>; [email protected] > Subject: Re: [PATCH v2 08/26] xen/domctl: wrap domain_soft_reset() with > CONFIG_MGMT_HYPERCALLS > > On 10.09.2025 09:38, Penny Zheng wrote: > > Function domain_soft_reset() is responsible for domain soft reset > > domctl-op, and shall be wrapped with CONFIG_MGMT_HYPERCALLS Tracking > > its calling chain, and the following functions shall also be wrapped > > with CONFIG_MGMT_HYPERCALLS: > > - grant_table_warn_active_grants() > > - argo_soft_reset() > > - arch_domain_soft_reset() > > Wrap XEN_DOMCTL_soft_reset-case transiently with > > CONFIG_MGMT_HYPERCALLS, and it will be removed when introducing > > CONFIG_MGMT_HYPERCALLS on the common/domctl.c in the last. > > > > Signed-off-by: Penny Zheng <[email protected]> > > --- > > v1 -> v2: > > - remove unnessary wrapping in stub.c > > - adapt to changes of "unify DOMCTL to MGMT_HYPERCALLS" > > - wrap XEN_DOMCTL_soft_reset-case transiently > > --- > > xen/arch/arm/domain.c | 2 ++ > > xen/arch/x86/domain.c | 2 ++ > > What about PPC and RISC-V? They have the function in stubs.c, but not adding > the > #ifdef there increases the chance that when the stubs are replaced by real > functions, the intended #ifdef might then be forgotten to add. >
As we are addressing concerns on the v1 about editing stubs.c files [1], I removed them all in this patch serie. If they are considered necessary now, I'll add them back in next version [1] https://lists.xenproject.org/archives/html/xen-devel/2025-08/msg00135.html > Jan
