public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Daniel Gollub <gollub@b1-systems.de>
To: Avi Kivity <avi@redhat.com>
Cc: kvm@vger.kernel.org
Subject: Re: [PATCH] qemu-kvm: Handle -no-shutodwn
Date: Wed, 20 May 2009 14:20:00 +0200	[thread overview]
Message-ID: <200905201420.01237.gollub@b1-systems.de> (raw)
In-Reply-To: <4A13EEB4.9090703@redhat.com>

On Wednesday 20 May 2009 01:51:16 pm Avi Kivity wrote:
> Daniel Gollub wrote:
> > Plain QEMU has the parameter -no-shutdown. This avoids termination of the
> > qemu process when VM got shutdown (e.g. to still use the QEMU-Monitor
> > with stopped VM). This parameter has no effect on qemu-kvm, today.
> >
> > This patch introduces identical handling, as in qemu, of -no-shutdown for
> > qemu-kvm:
> >
> >  * termination of qemu-kvm process on a VM shutdown get only avoided once
> >  * second shutdown of VM cause termination of qemu-kvm (like in qemu)
> >
> >  
>
> Applied, thanks.
>
> What is the reason for the wierd semantics?

No idea.

> I'd expect -no-shutdown to
> keep functioning even after the first shutdown.

I would prefer something like this as well. But this way it's 1:1 the same 
behavior as with plain QEMU. I'll prepare a propsoal + patch for changing this 
in QEMU. If this gets accepted i'll changed it for qemu-kvm as well.

Best Regards,
Daniel

-- 
Daniel Gollub                        Geschaeftsfuehrer: Ralph Dehner
FOSS Developer                       Unternehmenssitz:  Vohburg
B1 Systems GmbH                      Amtsgericht:       Ingolstadt
Mobil: +49-(0)-160 47 73 970         Handelsregister:   HRB 3537
EMail: gollub@b1-systems.de          http://www.b1-systems.de

Adresse: B1 Systems GmbH, Osterfeldstraße 7, 85088 Vohburg
http://pgpkeys.pca.dfn.de/pks/lookup?op=get&search=0xED14B95C2F8CA78D

      reply	other threads:[~2009-05-20 12:23 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-19 21:28 [PATCH] qemu-kvm: Handle -no-shutodwn Daniel Gollub
2009-05-20 11:51 ` Avi Kivity
2009-05-20 12:20   ` Daniel Gollub [this message]

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=200905201420.01237.gollub@b1-systems.de \
    --to=gollub@b1-systems.de \
    --cc=avi@redhat.com \
    --cc=kvm@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox