[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] qemu-dm monitor doesn't support the quit command



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@xxxxxxxxxxxxxx>
> To: "xen-devel" <xen-devel@xxxxxxxxxxxxxxxxxxx>
> 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@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel


-- 
best rgds,
edwin

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.