All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: kvm@vger.kernel.org
Subject: Re: [PATCH 0/2] Make savevm versioning compatible with upstream QEMU
Date: Thu, 30 Apr 2009 16:29:55 +0300	[thread overview]
Message-ID: <49F9A7D3.9040402@redhat.com> (raw)
In-Reply-To: <1241038430-7444-1-git-send-email-aliguori@us.ibm.com>

Anthony Liguori wrote:
> Right now, there is no way savevm versioning can be compatible with upstream
> QEMU because KVM adds fields to existing savevm structures without incrementing
> the versions.
>
> If you assume that KVM will eventually merge into upstream QEMU, this means that
> eventually KVM is going to have to break backwards compatibility with itself
> to resolve this issue in a non-graceful way.
>
> So let's do that now instead of doing it later when the situation is only worse.
>
> I'm happy to allocate particular version identifiers for KVM to avoid future
> conflicts.  I believe we should try to eliminate the existing differences so
> that we can converge in the future on a common versioning scheme.
>   

Applied both, thanks.

I think we can avoid the need to synchronize too much by saving 
kvm-specific state for device "x" using id "x-kvm"; this allows the two 
to evolve independently.  Of course it's much better to avoid divergence 
in the first place, but this isn't always possible.

-- 
error compiling committee.c: too many arguments to function


  parent reply	other threads:[~2009-04-30 13:29 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-29 20:53 [PATCH 0/2] Make savevm versioning compatible with upstream QEMU Anthony Liguori
2009-04-29 20:53 ` [PATCH 1/2] Increment virtio-net savevm version to avoid conflict " Anthony Liguori
2009-04-30  4:25   ` Alex Williamson
2009-04-30 13:39     ` Anthony Liguori
2009-04-29 20:53 ` [PATCH 2/2] Increment version id for CPU save state Anthony Liguori
2009-04-30 13:29 ` Avi Kivity [this message]
2009-04-30 13:35   ` [PATCH 0/2] Make savevm versioning compatible with upstream QEMU Anthony Liguori

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=49F9A7D3.9040402@redhat.com \
    --to=avi@redhat.com \
    --cc=aliguori@us.ibm.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.