[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3] xen_console: support the new extended xensto
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [PATCH v3] xen_console: support the new extended xenstore protocol |
Date: |
Thu, 30 Jun 2011 18:05:37 +0100 |
User-agent: |
Alpine 2.00 (DEB 1167 2008-08-23) |
On Thu, 30 Jun 2011, Alexander Graf wrote:
> On 06/29/2011 01:12 PM, address@hidden wrote:
> > From: Stefano Stabellini<address@hidden>
> >
> > 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