[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[RFC PATCH 2/4] xen/arm: Move make_hypervisor_node()


  • To: xen-devel@xxxxxxxxxxxxxxxxxxxx
  • From: Koichiro Den <den@xxxxxxxxxxxxx>
  • Date: Sun, 22 Jun 2025 00:11:59 +0900
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=valinux.co.jp; dmarc=pass action=none header.from=valinux.co.jp; dkim=pass header.d=valinux.co.jp; arc=none
  • 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=ZzEqMguDXik+OUbkQR4RTQfJmvwdGVxoIrMJVs4Ffn0=; b=o9UxXJ/2qRKq9Cb9bG94NUB87nl/G302ZWytSEfw4BXvPHRLbY55hDCeo2o/Kg5NjBTeIhH/tBoy59pXf9nBtSbb4cf71pmSFWvdpXy52saRgaADpkf2+9QB1OGdc3aY/BZLdd+bD+F2T0dDcWo7a8ml7HWivMd55CdpstKpjJ517oWsBRDX8k7rxuKa3Nk8LWSbul73HV36GrgJLEcWK99127URX3es/gYR9+0h3AMr8X2aW9yXUlZx/mb5Vjv/iDCEk8mznf8pVQ2uy7GpS6B2sWwsf6ilnlwggAKkMTzO1tr06Wp+GFFQDI3GRBNo7U/0FKruvEf+g+moOQTc8Q==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=E1BxLef7LVRWI6ip8AZdslSIgOJwKd9CyRVy54r3S0HT2611z56a3YOYYmKpeNi3AJXUc5tMJhIUDXZq2EVt4GYkMhB//ntuJ4r3LQwzg0g+RO/6wLWBO1bZZlxvCIPN2CjDa85Dn8Rko/U8KkTKjiDu08iyWhMxR0cfutQPSps9VKSRZ1EB42THbhvvEdDV/RxrZ5c8q7H1I3sRojXGX1uAhK2GU0VFR0E7eFWec65Ryl3WFIk951oC8Im1BmV+/6JpAATamHM6I+xJbgwjc9HBUHvvwq74huLjdPC7FOg0k0giQMob4Rbe/ZhHjbaSkiWl2e0ekbDF55kcs+V4Hw==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=valinux.co.jp;
  • Cc: Koichiro Den <den@xxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
  • Delivery-date: Sat, 21 Jun 2025 15:12:23 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Even though make_hypervisor_node() does not rely on the /reserved-memory
instantiation when calling find_unused_regions() (the wrapper introduced
in the previous commit), the next but one commit will use it for PV time
shared regions, in addition to the existing extended regions.

Move it as a prerequisite for the commit after next.

No functional changes intended.

Signed-off-by: Koichiro Den <den@xxxxxxxxxxxxx>
---
 xen/arch/arm/domain_build.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c
index 9d44b6fa9470..967ca6f375ca 100644
--- a/xen/arch/arm/domain_build.c
+++ b/xen/arch/arm/domain_build.c
@@ -1725,14 +1725,6 @@ static int __init handle_node(struct domain *d, struct 
kernel_info *kinfo,
          */
         evtchn_allocate(d);
 
-        /*
-         * The hypervisor node should always be created after all nodes
-         * from the host DT have been parsed.
-         */
-        res = make_hypervisor_node(d, kinfo, addrcells, sizecells);
-        if ( res )
-            return res;
-
         res = make_psci_node(kinfo->fdt);
         if ( res )
             return res;
@@ -1763,6 +1755,14 @@ static int __init handle_node(struct domain *d, struct 
kernel_info *kinfo,
             if ( res )
                 return res;
         }
+
+        /*
+         * The hypervisor node should always be created after all nodes
+         * from the host DT have been parsed.
+         */
+        res = make_hypervisor_node(d, kinfo, addrcells, sizecells);
+        if ( res )
+            return res;
     }
 
     res = fdt_end_node(kinfo->fdt);
-- 
2.48.1




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.