[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [PATCH v2] xen/arm: Reorder SCI resource cleanup in domain destruction
Move the SCI (System Control and Management Interface) resource cleanup earlier in the domain_relinquish_resources() sequence to ensure proper cleanup ordering during domain destruction. The SCI cleanup is now performed before TEE (Trusted Execution Environment) cleanup rather than after P2M mapping cleanup. This reordering ensures that SCI resources are properly released before other subsystems that might depend on them are torn down. This change addresses potential resource cleanup dependencies where SCI resources need to be released before P2M mappings are cleaned up, preventing potential issues during domain destruction on ARM platforms with SCI support. Fixes: e2cc10867b (xen/arm: add generic SCI subsystem, 2025-09-04) Signed-off-by: Oleksii Moisieiev <oleksii_moisieiev@xxxxxxxx> --- Changes in v2: - rearrange enum by placing PROG_sci before PROG_tee - add "Fixes:" tag xen/arch/arm/domain.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/xen/arch/arm/domain.c b/xen/arch/arm/domain.c index 1a8585d02b..e36719bce4 100644 --- a/xen/arch/arm/domain.c +++ b/xen/arch/arm/domain.c @@ -1042,6 +1042,7 @@ static int relinquish_memory(struct domain *d, struct page_list_head *list) */ enum { PROG_pci = 1, + PROG_sci, PROG_tee, PROG_xen, PROG_page, @@ -1049,7 +1050,6 @@ enum { PROG_p2m_root, PROG_p2m, PROG_p2m_pool, - PROG_sci, PROG_done, }; @@ -1090,6 +1090,11 @@ int domain_relinquish_resources(struct domain *d) return ret; #endif + PROGRESS(sci): + ret = sci_relinquish_resources(d); + if ( ret ) + return ret; + PROGRESS(tee): ret = tee_relinquish_resources(d); if (ret ) @@ -1109,10 +1114,6 @@ int domain_relinquish_resources(struct domain *d) ret = relinquish_p2m_mapping(d); if ( ret ) return ret; - PROGRESS(sci): - ret = sci_relinquish_resources(d); - if ( ret ) - return ret; PROGRESS(p2m_root): /* -- 2.34.1
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |