All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Zhai, Edwin" <edwin.zhai@intel.com>
To: Kasai Takanori <kasai.takanori@jp.fujitsu.com>
Cc: xen-devel <xen-devel@lists.xensource.com>
Subject: Re: [PATCH] qemu-dm monitor doesn't support the quit command
Date: Wed, 10 Jan 2007 11:34:27 +0800	[thread overview]
Message-ID: <20070110033427.GD6153@edwin-srv.sh.intel.com> (raw)
In-Reply-To: <103201c73463$e6af4000$dab2220a@VF03007L>

quit command is useful when we have no host console(e.g. qemu window on a remote 
X). and it's a normal exit that is different from xm destroy(by KILL sig).

maybe we can change "q" to "quit" to avoid typo? or provide confirm info?

BTW, "q" is most popular command when we debug qemu device model:)

On Wed, Jan 10, 2007 at 12:03:24PM +0900, Kasai Takanori wrote:
> Hi All,
> 
> I sent the following patches the other day.
> This patch makes qemu-dm monitor not support the quit command.
> Because there is a possibility of shutting down HVM Domain by mistake.
> We think that the HVM domain should shut down by xm shutdown/destroy 
> command.
> Could you apply this patch?
> 
> 
> Thanks,
> 
> --
> Takanori Kasai
> 
> 
> ----- Original Message ----- 
> From: "Kasai Takanori" <kasai.takanori@jp.fujitsu.com>
> To: "xen-devel" <xen-devel@lists.xensource.com>
> Sent: Monday, December 25, 2006 10:47 AM
> Subject: Re: [Xen-devel] [PATCH] qemu-dm monitor doesn't support the quit 
> command
> 
> 
> >Hi All,
> >
> >>It is natural that there is quit command in original QEMU.
> >>However, I don't think that the HVM domain on Xen is necessary.
> >>Because there is a possibility of shutting down GuestOS due to the 
> >>mistake.
> >>I think that I should prohibit the quit command of qemu-dm.
> >
> >So as to refuse the quit command, the patch was corrected.
> >If there is any problem, please let me know.
> >
> >Thanks
> >
> >--
> >Takanori Kasai


> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel


-- 
best rgds,
edwin

  reply	other threads:[~2007-01-10  3:34 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-22 13:07 [PATCH] qemu-dm monitor doesn't support the quit command Kasai Takanori
2006-12-25  1:47 ` Kasai Takanori
2007-01-10  3:03   ` Kasai Takanori
2007-01-10  3:34     ` Zhai, Edwin [this message]
2007-01-11 12:02       ` Kasai Takanori
2007-01-17  3:05         ` Kasai Takanori
2007-01-17  5:42           ` Anthony Liguori
2007-01-17  8:59             ` Kasai Takanori

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=20070110033427.GD6153@edwin-srv.sh.intel.com \
    --to=edwin.zhai@intel.com \
    --cc=kasai.takanori@jp.fujitsu.com \
    --cc=xen-devel@lists.xensource.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.