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

[PATCH] xenbus: Allow PVH dom0 a non-local xenstore


  • To: Juergen Gross <jgross@xxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Oleksandr Tyshchenko <oleksandr_tyshchenko@xxxxxxxx>
  • From: Jason Andryuk <jason.andryuk@xxxxxxx>
  • Date: Sat, 3 May 2025 09:19:35 -0400
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=suse.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0)
  • 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=XThhfT990yCc2xQ1ru4NmFFbo4KjWfIRuJptyrW4mmc=; b=hpO0fQWOGgAUJaJJ5D7mbskayjEh4czOkyf2KZTquSqaNXOJpd3JWtzbSL+vbzAUJAa8eDS/L2jk0M3ALRe2gus2ccsMxPtPuJ/wtoO2mONAnys4xoceLfDhY29t/NJ+dXGl8B+cvUzbMUXwpl5ObYRbtqF7QL3rJxL7Bstmm61etixrFeoW+cE+gOeKvImhIRbGaQoY7WHBXvNgHfIS5hbX9uCxL2KaehuRgDbDvWQNkVMoEmD7bucQIGinvrX6umZtBwbmNKiIA3sBLxoWY7c08a8jZNQpeeeFSoh5bF0oepQan6X5UzMtuEr2mPcN01PLqrGywAcM/b3H3Tanvw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=RiERQ+EymohCNCMtfmRaaRpVE8FLKbghvFBZfvA5wZRUJ8gnODMworNqKSW9NJNegZWLq4Bzs+8tOrQ8Fx3WPtF+2/hRWx12VMOg3jjQBpXCqPLuWccpi0H9xu6nzNHUI3coDINDehvrtyPHhoLlFohMupyXEbDeJoMjFLhuMq/d+pLaRil+7hPohTGOeglBJszmk/+N/NWp1W1XAjcAw8XXz0TdyikmSMmhELE9DxBXSPLmx9rFZrwjCxCBCngxU4x0n/f0LzfWGRg+X30A8BP5NsbF9gJWik+BLkSCG+FUhUjnrK2CdTcp5jsSOqRluy1DnQEFqfgj99EoATM83g==
  • Cc: Jason Andryuk <jason.andryuk@xxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxx>, <linux-kernel@xxxxxxxxxxxxxxx>
  • Delivery-date: Sat, 03 May 2025 13:20:04 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Make xenbus_init() allow a non-local xenstore for a PVH dom0 - it is
currently forced to XS_LOCAL.  With Hyperlaunch booting dom0 and a
xenstore stubdom, dom0 can be handled as a regular XS_HVM following the
late init path.

Drop the use of xen_initial_domain() and just check for the event
channel instead.  This matches the PV case where there is no check for
initial domain.

Check the full 64bit HVM_PARAM_STORE_EVTCHN value to catch the off
chance that high bits are set for the 32bit event channel.

Signed-off-by: Jason Andryuk <jason.andryuk@xxxxxxx>
---
 drivers/xen/xenbus/xenbus_probe.c | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/drivers/xen/xenbus/xenbus_probe.c 
b/drivers/xen/xenbus/xenbus_probe.c
index 6d32ffb01136..7604f70ee108 100644
--- a/drivers/xen/xenbus/xenbus_probe.c
+++ b/drivers/xen/xenbus/xenbus_probe.c
@@ -966,9 +966,15 @@ static int __init xenbus_init(void)
        if (xen_pv_domain())
                xen_store_domain_type = XS_PV;
        if (xen_hvm_domain())
+       {
                xen_store_domain_type = XS_HVM;
-       if (xen_hvm_domain() && xen_initial_domain())
-               xen_store_domain_type = XS_LOCAL;
+               err = hvm_get_parameter(HVM_PARAM_STORE_EVTCHN, &v);
+               if (err)
+                       goto out_error;
+               xen_store_evtchn = (int)v;
+               if (!v)
+                       xen_store_domain_type = XS_LOCAL;
+       }
        if (xen_pv_domain() && !xen_start_info->store_evtchn)
                xen_store_domain_type = XS_LOCAL;
        if (xen_pv_domain() && xen_start_info->store_evtchn)
@@ -987,10 +993,6 @@ static int __init xenbus_init(void)
                xen_store_interface = gfn_to_virt(xen_store_gfn);
                break;
        case XS_HVM:
-               err = hvm_get_parameter(HVM_PARAM_STORE_EVTCHN, &v);
-               if (err)
-                       goto out_error;
-               xen_store_evtchn = (int)v;
                err = hvm_get_parameter(HVM_PARAM_STORE_PFN, &v);
                if (err)
                        goto out_error;
-- 
2.34.1




 


Rackspace

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