[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH] arm/domain: fix comment for arch_set_info_guest
Hi Luca, On 22/08/2022 07:56, Luca Fancellu wrote: On 5 Aug 2022, at 18:35, Julien Grall <julien@xxxxxxx> wrote: Hi Luca, On 05/08/2022 14:08, Luca Fancellu wrote:The function arch_set_info_guest is not reached anymore through VCPUOP_initialise on arm, update the comment. Signed-off-by: Luca Fancellu <luca.fancellu@xxxxxxx> --- Changes in v2: - rephrased comment to not list caller functions (Julien) --- xen/arch/arm/domain.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/xen/arch/arm/domain.c b/xen/arch/arm/domain.c index 2cd481979cf1..9db8a37a089c 100644 --- a/xen/arch/arm/domain.c +++ b/xen/arch/arm/domain.c @@ -885,9 +885,8 @@ static int is_guest_pv64_psr(uint64_t psr) #endif /* - * Initialise VCPU state. The context can be supplied by either the - * toolstack (XEN_DOMCTL_setvcpucontext) or the guest - * (VCPUOP_initialise) and therefore must be properly validated. + * Initialise vCPU state. The context may be supplied by an external entity, so + * we need to validate itNIT: Missing a full stop. This can be added on commit. Acked-by: Julien Grall <jgrall@xxxxxxxxxx>Hi Julien, Any plan to commit this one? Not an important change, just asking so that I can remove it from my watch list. Sorry for the delay. It is now pushed. Cheers, -- Julien Grall
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |