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

[Xen-devel] Re: [PATCH v3] xen_console: support the new extended xenstore protocol



On Thu, 30 Jun 2011, Alexander Graf wrote:
> On 06/29/2011 01:12 PM, stefano.stabellini@xxxxxxxxxxxxx wrote:
> > From: Stefano Stabellini<stefano.stabellini@xxxxxxxxxxxxx>
> >
> > Since CS 21994 on xen-unstable.hg and CS
> > 466608f3a32e1f9808acdf832a5843af37e5fcec on qemu-xen-unstable.git, few
> > changes have been introduced to the PV console xenstore protocol, as
> > described by the document docs/misc/console.txt under xen-unstable.hg.
> >
> >  From the Qemu point of view, very few modifications are needed to
> > correctly support the protocol: read from xenstore the "output" node
> > that tell us what the output of the PV console is going to be.
> > In case the output is a tty, write to xenstore the device name.
> >
> > Changes in v2:
> >
> > - fix error paths: free malloc'ed strings and close the xenstore
> > connection before returning;
> >
> > - remove useless snprintf in xenstore_store_pv_console_info if i == 0.
> >
> > Changes in v3:
> >
> > - replace xs_daemon_open/xs_daemon_close with xs_open/xs_close.
> 
> 
> On SLES11SP1:
> 
> cc1: warnings being treated as errors
> /studio/tmp/agraf/xen-all.c: In function âstore_dev_infoâ:
> /studio/tmp/agraf/xen-all.c:755: error: implicit declaration of function 
> âxs_openâ
> /studio/tmp/agraf/xen-all.c:755: error: nested extern declaration of 
> âxs_openâ
> /studio/tmp/agraf/xen-all.c:755: error: assignment makes pointer from 
> integer without a cast
> /studio/tmp/agraf/xen-all.c:784: error: implicit declaration of function 
> âxs_closeâ
> /studio/tmp/agraf/xen-all.c:784: error: nested extern declaration of 
> âxs_closeâ

I'll send an update with two simple implementations of xs_close and
xs_open in case xen < 4.1
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

 


Rackspace

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