|
|
|
|
|
|
|
|
|
|
xen-devel
[Xen-devel] Re: [PATCH v3] xen_console: support the new extended xenstor
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
|
|
|
|
|