[Xen-devel] [PATCH 11/24] [xen-unstable.hg] xenstored cmdline opts to tell it what port and grant ref to use to access dom0's xenstore page

[Alex Zeffertt <alex.zeffertt@xxxxxxxxxxxxx>]

Adds three flags to xenstored for it to initialize dom0's connection
from another domain.

TODO: This doesn't make too much sense anymore becuase we're not doing a
local_dom_init for stubdoms anyway. It also makes the local_domid flag
redundant.

Signed-off-by: Diego Ongaro <diego.ongaro@xxxxxxxxxx>
Signed-off-by: Alex Zeffertt <alex.zeffertt@xxxxxxxxxxxxx>
---

diff -r e9bc9e4ab2fb tools/xenstore/xenstored_core.c
--- a/tools/xenstore/xenstored_core.c   Fri Aug 01 16:50:37 2008 +0100
+++ b/tools/xenstore/xenstored_core.c   Fri Aug 01 16:50:39 2008 +0100
@@ -55,6 +55,10 @@
 #include "hashtable.h"
 
 extern int xce_handle; /* in xenstored_domain.c */
+
+unsigned int local_domid = 0;
+int dom0_grant_ref = -1;
+evtchn_port_t dom0_port = -1;
 
 static bool verbose = false;
 LIST_HEAD(connections);
@@ -1753,6 +1757,9 @@
        { "preserve-local", 0, NULL, 'L' },
        { "verbose", 0, NULL, 'V' },
        { "watch-nb", 1, NULL, 'W' },
+       { "local-domid", 1, NULL, 'x' },
+       { "dom0-grant-ref", 1, NULL, 'y' },
+       { "dom0-port", 1, NULL, 'z' },
        { NULL, 0, NULL, 0 } };
 
 extern void dump_conn(struct connection *conn); 
@@ -1813,6 +1820,15 @@
                        break;
                case 'W':
                        quota_nb_watch_per_domain = strtol(optarg, NULL, 10);
+                       break;
+               case 'x':
+                       local_domid = strtol(optarg, NULL, 10);
+                       break;
+               case 'y':
+                       dom0_grant_ref = strtol(optarg, NULL, 10);
+                       break;
+               case 'z':
+                       dom0_port = strtol(optarg, NULL, 10);
                        break;
                }
        }
diff -r e9bc9e4ab2fb tools/xenstore/xenstored_domain.c
--- a/tools/xenstore/xenstored_domain.c Fri Aug 01 16:50:37 2008 +0100
+++ b/tools/xenstore/xenstored_domain.c Fri Aug 01 16:50:39 2008 +0100
@@ -578,26 +578,75 @@
 {
 }
 
+static int local_dom_init(void)
+{
+       extern unsigned int local_domid;
+       evtchn_port_t port;
+       struct domain *domain;
+
+       port = xenbus_evtchn();
+       if (port == -1) {
+               barf_perror("xenbus_evtchn()");
+               return -1;
+       }
+
+       domain = new_domain(NULL, local_domid, port); 
+       if (domain == NULL) {
+               barf_perror("new_domain");
+               return -1;
+       }
+
+       domain->interface = xenbus_map();
+       if (domain->interface == NULL) {
+               barf_perror("xenbus_map()");
+               return -1;
+       }
+
+       talloc_steal(domain->conn, domain); 
+
+       if (xc_evtchn_notify(xce_handle, domain->port) == -1) {
+               barf_perror("notify local domain failed");
+               return -1;
+       }
+
+       return 0; 
+}
+
 static int dom0_init(void) 
 { 
-       evtchn_port_t port;
+       extern unsigned int local_domid;
+       extern evtchn_port_t dom0_port;
+       extern int dom0_grant_ref;
        struct domain *dom0;
 
-       port = xenbus_evtchn();
-       if (port == -1)
+       /* if we're in dom0, local_dom_init() has already done this */
+       if (local_domid == 0)
+               return 0;
+
+       if (dom0_port == -1 || dom0_grant_ref == -1)
                return -1;
 
-       dom0 = new_domain(NULL, 0, port); 
-       if (dom0 == NULL)
+       dom0 = new_domain(NULL, 0, dom0_port);
+       if (dom0 == NULL) {
+               barf_perror("new_domain dom0");
                return -1;
+       }
 
-       dom0->interface = xenbus_map();
-       if (dom0->interface == NULL)
+       dom0->interface = xc_gnttab_map_grant_ref(*xcg_handle,
+                                                 0,
+                                                 dom0_grant_ref,
+                                                 PROT_READ|PROT_WRITE);
+       if (dom0->interface == NULL) {
+               barf_perror("xc_gnttab_map_grant_ref dom0");
                return -1;
+       }
 
-       talloc_steal(dom0->conn, dom0); 
+       talloc_steal(dom0->conn, dom0);
 
-       xc_evtchn_notify(xce_handle, dom0->port); 
+       if (xc_evtchn_notify(xce_handle, dom0->port) == -1) {
+               barf_perror("notify dom0 failed");
+               return -1;
+       }
 
        return 0; 
 }
@@ -633,6 +682,9 @@
 
        if (xce_handle < 0)
                barf_perror("Failed to open evtchn device");
+
+       if (local_dom_init() != 0)
+               barf_perror("Failed to initialize local domain's state");
 
        if (dom0_init() != 0) 
                barf_perror("Failed to initialize dom0 state"); 


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel