All of lore.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 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.