All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Juan Quintela <quintela@redhat.com>
Subject: [Qemu-devel] Re: [PATCH 03/26] Remove SaveVM v2 support
Date: Fri, 11 Sep 2009 10:37:58 -0500	[thread overview]
Message-ID: <4AAA6ED6.6080604@codemonkey.ws> (raw)
In-Reply-To: <alpine.DEB.2.00.0909111542580.2687@kaball-desktop>

Stefano Stabellini wrote:
> No one complained because the people affected by this issue probably
> don't follow qemu development so closely to have realized that their old
> images won't work anymore.
> I do believe that fixing v2 compatibility is important for a large
> number of qemu users (including kvm and xen).
>   

kvm never supported live migration with v2.  As I think I mentioned in 
another note, I don't want to burden the VMState refactoring with 
supporting a format that qemu doesn't support upstream.  That said, if 
you want to add back a version of it that makes use of the new 
infrastructure, I would be willing to take that.

In fact, if you're willing to do the cleanup work along with Juan, we 
can avoid removing it.

> That said, I don't want to fight over this more than necessary, so as
> long as you keep the per device loading functions in C we can maintain
> v2 compatibility downstream if upstream is not interested.
>   

The per device loading functions will be switching to VMState.

Regards,

Anthony Liguori

  reply	other threads:[~2009-09-11 15:38 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-10  1:04 [Qemu-devel] [PATCH 00/26] VMState: port several pc devices to vmstate Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 01/26] ram: remove support for loading v1 Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 02/26] ram: Remove SaveVM Version 2 support Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 03/26] Remove SaveVM v2 support Juan Quintela
2009-09-10 17:41   ` Stefano Stabellini
2009-09-10 17:43     ` [Qemu-devel] " Juan Quintela
2009-09-10 18:15       ` Stefano Stabellini
2009-09-10 18:22         ` Anthony Liguori
2009-09-11 14:05           ` Stefano Stabellini
2009-09-11 14:28             ` Juan Quintela
2009-09-11 15:32               ` Stefano Stabellini
2009-09-11 15:37                 ` Anthony Liguori [this message]
2009-09-11 15:48                 ` Juan Quintela
2009-09-11 17:59                   ` Stefano Stabellini
2009-09-17 11:40                   ` Stefano Stabellini
2009-09-10  1:04 ` [Qemu-devel] [PATCH 04/26] timers: remove useless check Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 05/26] Unexport ticks_per_sec variable. Create get_ticks_per_sec() function Juan Quintela
2009-09-10  1:20   ` malc
2009-09-10  1:57     ` [Qemu-devel] " Juan Quintela
2009-09-10  2:21       ` malc
2009-09-10 16:44         ` Juan Quintela
2009-09-10 17:02           ` malc
2009-09-10 17:38             ` Anthony Liguori
2009-09-10 21:31               ` malc
2009-09-10 22:08                 ` Anthony Liguori
2009-09-10 23:10                   ` malc
2009-09-10 23:33                     ` Juan Quintela
2009-09-11  5:49                     ` Amit Shah
2009-09-11 13:00                     ` Markus Armbruster
2009-09-11 15:34                     ` Anthony Liguori
2009-09-11 15:55                       ` Juan Quintela
2009-09-11 15:58                       ` Jan Kiszka
2009-11-09 16:29                       ` Paul Brook
2009-09-10 17:39             ` Juan Quintela
2009-09-10 22:16               ` Paolo Bonzini
2009-09-10 23:11                 ` malc
2009-09-11  9:04   ` Jan Kiszka
2009-09-11  9:31     ` Juan Quintela
2009-09-11  9:37       ` Jan Kiszka
2009-09-11 10:15         ` Juan Quintela
2009-09-11 10:26           ` Jan Kiszka
2009-09-10  1:04 ` [Qemu-devel] [PATCH 06/26] timers: Createt TimersState and put all timers state there Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 07/26] timers: move them to VMState Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 08/26] vmstate: add sensible arguments to vmstate_unregister() Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 09/26] vmstate: rename run_after_load() -> post_load() Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 10/26] vmstate: Add pre_load() hook Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 11/26] vmstate: Add pre/post_save() hooks Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 12/26] vmstate: port cpu_comon Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 13/26] vmstate: port fw_cfg device Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 14/26] vmstate: port i8259 device Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 15/26] vmstate: add support for uint8_t equal Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 16/26] vmstate: port fdc device Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 17/26] vmstate: add support for arrays of uint16_t Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 18/26] vmstate: port dma device Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 19/26] vmstate: port vmmouse device Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 20/26] vmstate: port pckbd device Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 21/26] vmstate: add uint64 array support Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 22/26] vmstate: port ioapic device Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 23/26] hpet: it is imposible that qemu_timer field is NULL at this point Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 24/26] vmstate: port hpet device Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 25/26] vmstate: port serial device Juan Quintela
2009-09-10  1:04 ` [Qemu-devel] [PATCH 26/26] vmstate: port cirrus_vga device Juan Quintela

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=4AAA6ED6.6080604@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=stefano.stabellini@eu.citrix.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.