All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin Wolf <kwolf@suse.de>
To: Yuji Shimada <shimada-yxb@necst.nec.co.jp>
Cc: xen-devel@lists.xensource.com,
	Ian Jackson <Ian.Jackson@eu.citrix.com>,
	Yosuke Iwamatsu <y-iwamatsu@ab.jp.nec.com>
Subject: Re: [PATCH] Log qemu-dm's message with date, time and pid
Date: Mon, 30 Jun 2008 11:45:45 +0200	[thread overview]
Message-ID: <4868AB49.9050109@suse.de> (raw)
In-Reply-To: <20080630134036.F31B.SHIMADA-YXB@necst.nec.co.jp>

Yuji Shimada schrieb:
> I agree with developing proper logging functions. But without these
> functions, I think ioemu-logger is a good solution.
> 
> We can reboot xend by the following command.
>   # /etc/init.d/xend restart
> Running of guest domains must not be affected by rebooting xend.

This seems to be a valid point.

> I'm sure that ioemu-logger dying is rare case. But there is no guarantee
> that it doesn't die.
> If xend starts ioemu-logger, xend loses one end of a pipe by rebooting
> xend. After that, xend can't associates stdin for ioemu-logger with the
> pipe when xend restarts ioemu-logger after ioemu-logger dies. This
> problem happens because the life cycle of xend and qemu-dm is different.
> So I think to start ioemu-logger by qemu-dm is more simple.
> 
> Using named pipe (FIFO), it's possible to start ioemu-logger by xend.
> But sorry, I don't know about python well. Could anyone develope it?

I'm under the impression that you're completely missing my (and Ian's)
point. Actually, I'm not talking about having ioemu-logger started by
xend. I'm talking about not having ioemu-logger at all (at least not as
a new external tool). Xen has already too many of these small tools,
IMHO, and we shouldn't add even more to them without need.

So, please implement it in qemu-dm instead of an external tool, if xend
is not possible.

Kevin

  reply	other threads:[~2008-06-30  9:45 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-26  5:02 [PATCH] Log qemu-dm's message with date, time and pid Yosuke Iwamatsu
2008-06-26 11:20 ` Kevin Wolf
2008-06-26 14:46 ` Ian Jackson
2008-06-26 16:38   ` Daniel P. Berrange
2008-07-01 17:38     ` Ian Jackson
2008-07-01 18:10       ` Daniel P. Berrange
2008-07-01 20:06       ` John Levon
2008-07-02 10:26         ` Ian Jackson
2008-07-02 10:41           ` Daniel P. Berrange
2008-07-02 15:20             ` Brendan Cully
2008-06-27  8:19   ` Yuji Shimada
2008-06-27  9:10     ` Kevin Wolf
2008-06-27 13:40       ` Ian Jackson
2008-06-30  8:21         ` Yuji Shimada
2008-06-30  9:45           ` Kevin Wolf [this message]
2008-07-01  8:01             ` Yuji Shimada

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=4868AB49.9050109@suse.de \
    --to=kwolf@suse.de \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=shimada-yxb@necst.nec.co.jp \
    --cc=xen-devel@lists.xensource.com \
    --cc=y-iwamatsu@ab.jp.nec.com \
    /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.