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

Re: [PATCH 10/15] x86/hyperlaunch: locate dom0 initrd with hyperlaunch


  • To: Jason Andryuk <jason.andryuk@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • From: "Daniel P. Smith" <dpsmith@xxxxxxxxxxxxxxxxxxxx>
  • Date: Wed, 11 Dec 2024 10:49:56 -0500
  • Arc-authentication-results: i=1; mx.zohomail.com; dkim=pass header.i=apertussolutions.com; spf=pass smtp.mailfrom=dpsmith@xxxxxxxxxxxxxxxxxxxx; dmarc=pass header.from=<dpsmith@xxxxxxxxxxxxxxxxxxxx>
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1733932201; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=4mNkgdDLw9V4heCgs6q03+jt8fQN8SUF8+4WfCQLRMs=; b=Arg85pjk9G9nR6gwu8Y9D8I8NPE55+VrM8tSHK2LPScIiWph2P8T+VdP5YX/mfjUETqNdChNFSSP7oX0Yy7r4bqTec8kGvFEdDBy2DNW62YdmjYPcP5nUCV224bC+BhY1pJg80KNUFSJVT6GK5Wka2JCvaY+xoW/My2duIzRjt0=
  • Arc-seal: i=1; a=rsa-sha256; t=1733932201; cv=none; d=zohomail.com; s=zohoarc; b=hO5jdAoh7s6+2XYrGfHwetCAiwBcyJGNScpc1+cIiRQe1qYtZivX9te5Sn60oSib9uKICTOQuz913brDCc/UADCQqwl+oIsOPn72S6u5TmmYDQYvPA6OY6WaZyQzdB2MOgbj8mQ2Uf77Whvq9dvoPiKNAJgPngxaUUsbaienpd8=
  • Cc: christopher.w.clark@xxxxxxxxx, stefano.stabellini@xxxxxxx, Jan Beulich <jbeulich@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Delivery-date: Wed, 11 Dec 2024 15:50:15 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 11/25/24 18:34, Jason Andryuk wrote:
On 2024-11-23 13:20, Daniel P. Smith wrote:
Look for a subnode of type `multiboot,ramdisk` within a domain node. If
found, process the reg property for the MB1 module index.

Signed-off-by: Daniel P. Smith <dpsmith@xxxxxxxxxxxxxxxxxxxx>
---
  xen/arch/x86/domain_builder/fdt.c | 25 ++++++++++++++++++++++
  xen/arch/x86/setup.c              | 35 +++++++++++++++++--------------
  2 files changed, 44 insertions(+), 16 deletions(-)

diff --git a/xen/arch/x86/domain_builder/fdt.c b/xen/arch/x86/ domain_builder/fdt.c
index f8ddb11b339e..bc8054a80ec1 100644
--- a/xen/arch/x86/domain_builder/fdt.c
+++ b/xen/arch/x86/domain_builder/fdt.c
@@ -152,6 +152,31 @@ static int __init process_domain_node(
                  if ( ret > 0 )
                      bd->kernel->fdt_cmdline = true;
              }
+
+            continue;
+        }
+        if ( fdt_node_check_compatible(fdt, node, "multiboot,ramdisk") == 0 )

I think
         continue;
     }
     if

should change to

     } else if

?

Yah, I can make it a nested set of ifs.

Also "module,ramdisk"/"module,index"?

Yes, will be updated to new format.

v/r,
dps




 


Rackspace

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