Re: [Xen-devel] [PATCH] tools: revert to using /var and /etc/

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Tue, 29 Jan 2013, Ian Campbell wrote:
> On Mon, 2013-01-28 at 16:49 +0000, Stefano Stabellini wrote:
> > On Mon, 28 Jan 2013, Ian Campbell wrote:
> > > On Mon, 2013-01-28 at 10:39 +0000, Sander Eikelenboom wrote:
> > > > Another thing is that:
> > > >         $QEMU_XEN -xen-domid 0 -xen-attach -name dom0 -nographic -M
> > > > xenpv -daemonize -monitor /dev/null -pidfile $QEMU_PIDFILE
> > > > 
> > > > Doesn't seem to start with upstream qemu, combination of -nographic
> > > > and -daemonize doesn't seem to be allowed anymore.
> > > > (not a problem for me, because i don't seem to be using it as
> > > > diskbackend, but for a 4.3 release ...) 
> > > 
> > > Stefano, is this expected?
> >  
> > No, it is not. Sander, thanks for the report.
> > This is the reason:
> > 
> > /* According to documentation and historically, -nographic redirects
> >  * serial port, parallel port and monitor to stdio, which does not work
> >  * with -daemonize.  We can redirect these to null instead, but since
> >  * -nographic is legacy, let's just error out.
> >  * We disallow -nographic only if all other ports are not redirected
> >  * explicitly, to not break existing legacy setups which uses
> >  * -nographic _and_ redirects all ports explicitly - this is valid
> >  * usage, -nographic is just a no-op in this case.
> >  */
> > 
> > If fact it looks like that the appended patch fixes it, can you please
> > confirm it?
> 
> Is this going to break people using older versions of upstream qemu or
> should this change be compatible?

They are compatible

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel