Re: [Xen-devel] [PATCH 21/21] xenstored: Add stub domain builder

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2012-01-23 at 14:26 +0000, Stefano Stabellini wrote:
> On Mon, 23 Jan 2012, Ian Campbell wrote:
> > On Fri, 2012-01-20 at 20:47 +0000, Daniel De Graaf wrote:
> > > +int main(int argc, char** argv)
> > > +{
> > > + xc_interface *xch;
> > > + struct xs_handle *xsh;
> > > + char buf[16];
> > > + int rv;
> > > +
> > > + if (argc != 4) {
> > > +         printf("Use: %s <xenstore-kernel> <memory_mb> <flask-label>\n", 
> > > argv[0]);
> > > +         return 2;
> > > + }
> > > +
> > > + xch = xc_interface_open(NULL, NULL, 0);
> > > + if (!xch) return 1;
> > > +
> > > + rv = build(xch, argv);
> > > +
> > > + xc_interface_close(xch);
> > > +
> > > + if (rv) return 1;
> > 
> > Did you consider forking a daemon at this point to sit and drain the
> > domains console ring into a log file? (instead of/as well as your patch
> > 08/21).
> 
> I don't know if there are any benefits in basing this stub domain builder
> on libxl but if it was based on libxl you could just add a pv console
> device with consback = LIBXL_CONSOLE_BACKEND_IOEMU and then set output =
> "file:/path/to/file", and you would have your logging going to that
> file.

This builder cannot use xenstore and therefore cannot use xenconsoled
(which would block waiting for xenstored) either.

Ian.


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