All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Roger Pau Monné" <roger.pau@citrix.com>
To: Sander Eikelenboom <linux@eikelenboom.it>
Cc: "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	Ian Jackson <Ian.Jackson@eu.citrix.com>,
	Ian Campbell <Ian.Campbell@citrix.com>,
	Stefano Stabellini <Stefano.Stabellini@eu.citrix.com>
Subject: Re: [PATCH] tools: revert to using /var and /etc/
Date: Mon, 28 Jan 2013 18:54:04 +0000	[thread overview]
Message-ID: <5106C94C.2050205@citrix.com> (raw)
In-Reply-To: <1557888518.20130128194603@eikelenboom.it>

On 28/01/13 18:46, Sander Eikelenboom wrote:
> 
> Monday, January 28, 2013, 5:49:28 PM, you 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?
> 
> Confirmed thx !

Running Qemu using the -daemonize and -nographic options used to left
the console tty in a bad state (which could be reverted by using 'stty
sane'). Do you know if this is also fixed?

  reply	other threads:[~2013-01-28 18:54 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-25 13:10 [PATCH] tools: revert to using /var and /etc/ Ian Campbell
2013-01-25 13:41 ` Roger Pau Monné
2013-01-25 16:15   ` Ian Campbell
2013-01-27 21:49     ` Sander Eikelenboom
2013-01-28 10:25       ` Ian Campbell
2013-01-28 10:39         ` Sander Eikelenboom
2013-01-28 10:50           ` Ian Campbell
2013-01-28 16:49             ` Stefano Stabellini
2013-01-28 18:46               ` Sander Eikelenboom
2013-01-28 18:54                 ` Roger Pau Monné [this message]
2013-01-29 11:26                   ` Stefano Stabellini
2013-01-29 11:30               ` Ian Campbell
2013-01-29 11:33                 ` Stefano Stabellini
2013-01-28 18:48             ` [PATCH] xencommons: redirect serial and parallel to /dev/null Stefano Stabellini
2013-02-05 11:35               ` Ian Campbell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5106C94C.2050205@citrix.com \
    --to=roger.pau@citrix.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=Stefano.Stabellini@eu.citrix.com \
    --cc=linux@eikelenboom.it \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.