All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Eric Blake <eblake@redhat.com>
Cc: quintela@redhat.com, qemu-devel@nongnu.org,
	stefanb@linux.vnet.ibm.com, mdroth@linux.vnet.ibm.com,
	agraf@suse.de, aliguori@amazon.com, afaerber@suse.de,
	"Dr. David Alan Gilbert" <dgilbert@redhat.com>
Subject: Re: [Qemu-devel] [RFC PATCH v2 00/16] visitor+BER migration format
Date: Wed, 7 May 2014 12:57:34 +0300	[thread overview]
Message-ID: <20140507095734.GB12259@redhat.com> (raw)
In-Reply-To: <53580D27.2080507@redhat.com>

On Wed, Apr 23, 2014 at 12:57:43PM -0600, Eric Blake wrote:
> On 04/23/2014 11:54 AM, Michael S. Tsirkin wrote:
> > On Wed, Apr 23, 2014 at 11:47:21AM -0600, Eric Blake wrote:
> >> On 04/23/2014 11:16 AM, Dr. David Alan Gilbert wrote:
> >>> The one thing that the environment variable does make nice and easy,
> >>> for dev, is using it with existing test setups - e.g. running virt-test
> >>> in BER mode or existing mode.
> >> Hmm. Maybe a compromise - an environment variable determines the default
> >> state of the QMP capability (for use in running the testsuite in one
> >> mode or the other for all tests that don't explicitly set a mode), but
> >> the QMP command still allows toggling the state at runtime (the
> >> environment variable doesn't lock us out from changing away from the
> >> startup default).
> > 
> > Eric, could you explain the need for the runtime thing?
> 
> I thought I did - when migrating across machines to what we know is an
> older qemu, we want the old migration format; but when migrating to a
> file to be reloaded by the current (or newer) qemu, we want the new
> format since it is more robust.

It's more robust for adding new functionality/new machine types but for
old one I think we can assume it describes it sufficiently.

> But libvirt can't know up front whether
> the user starting a qemu instance has plans down the road of calling
> 'virsh save' (migrate to file) or 'virsh migrate' (migrate across
> machines) first.
> 
> On the other hand, you have a point that upstream tends to not care
> about migration from new qemu to old (that's one of the value-added
> tasks of downstream distros), and that we could merely get away with
> using the -m machine name as the witness of which form to use, with no
> need for an environment variable or any other runtime switch.
> 
> -- 
> Eric Blake   eblake redhat com    +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
> 

  parent reply	other threads:[~2014-05-07  9:58 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1398271069-22057-1-git-send-email-dgilbert@redhat.com>
     [not found] ` <1398271069-22057-6-git-send-email-dgilbert@redhat.com>
2014-05-07  9:47   ` [Qemu-devel] [RFC PATCH v2 05/16] Header/constant/types fixes for visitors Michael S. Tsirkin
2014-05-07 10:33     ` Dr. David Alan Gilbert
     [not found] ` <1398271069-22057-2-git-send-email-dgilbert@redhat.com>
2014-05-07  9:50   ` [Qemu-devel] [RFC PATCH v2 01/16] Visitor: Add methods for migration format use Michael S. Tsirkin
2014-05-07 10:23     ` Dr. David Alan Gilbert
2014-05-07 10:32       ` Michael S. Tsirkin
     [not found] ` <5357EF56.4010703@redhat.com>
     [not found]   ` <20140423171622.GG2516@work-vm>
     [not found]     ` <87sip3dvsj.fsf@blackfin.pond.sub.org>
     [not found]       ` <20140424082059.GB2459@work-vm>
     [not found]         ` <20140424082923.GA31845@redhat.com>
     [not found]           ` <87wqeduc0d.fsf@blackfin.pond.sub.org>
2014-05-06 18:58             ` [Qemu-devel] [RFC PATCH v2 00/16] visitor+BER migration format Dr. David Alan Gilbert
2014-05-06 20:26               ` Michael S. Tsirkin
2014-05-07  5:49                 ` Markus Armbruster
2014-05-07  9:22                   ` Dr. David Alan Gilbert
     [not found]     ` <5357FCA9.8040801@redhat.com>
     [not found]       ` <20140423175410.GA28308@redhat.com>
     [not found]         ` <53580D27.2080507@redhat.com>
2014-05-07  9:57           ` Michael S. Tsirkin [this message]
     [not found] ` <1398271069-22057-16-git-send-email-dgilbert@redhat.com>
2014-05-07 10:02   ` [Qemu-devel] [RFC PATCH v2 15/16] Wire in BER visitors Michael S. Tsirkin
2014-05-07 10:08     ` Dr. David Alan Gilbert

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=20140507095734.GB12259@redhat.com \
    --to=mst@redhat.com \
    --cc=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=aliguori@amazon.com \
    --cc=dgilbert@redhat.com \
    --cc=eblake@redhat.com \
    --cc=mdroth@linux.vnet.ibm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=stefanb@linux.vnet.ibm.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.