All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <aliguori@us.ibm.com>
To: Avi Kivity <avi@redhat.com>
Cc: kvm@vger.kernel.org
Subject: Re: [PATCH 0/2] Make savevm versioning compatible with upstream QEMU
Date: Thu, 30 Apr 2009 08:35:34 -0500	[thread overview]
Message-ID: <49F9A926.5050909@us.ibm.com> (raw)
In-Reply-To: <49F9A7D3.9040402@redhat.com>

Avi Kivity wrote:
> 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.

I need to add save/restore support to upstream QEMU so this is a good 
excuse to just merge the changes in KVM upstream.  So hopefully this 
will become a non issue.  If something arises and you need more savevm 
state, introduce a new section suffixed or prefixed with kvm.  
Alternatively, ask and I can reserve an ID upstream.

For virtio-net, we just need to get the vnet stuff merged upstream.

-- 
Regards,

Anthony Liguori


      reply	other threads:[~2009-04-30 13:35 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 ` [PATCH 0/2] Make savevm versioning compatible with upstream QEMU Avi Kivity
2009-04-30 13:35   ` Anthony Liguori [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=49F9A926.5050909@us.ibm.com \
    --to=aliguori@us.ibm.com \
    --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.