From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Brook Subject: Re: [PATCH] qemu: virtio save/load bindings Date: Tue, 26 May 2009 11:20:29 +0100 Message-ID: <200905261120.30344.paul@codesourcery.com> References: <20090525133405.GA6166@redhat.com> <4A1BA998.8040803@codemonkey.ws> Mime-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Cc: "Michael S. Tsirkin" , Avi Kivity , qemu-devel@nongnu.org, Carsten Otte , kvm@vger.kernel.org, Rusty Russell , virtualization@lists.linux-foundation.org, Christian Borntraeger , Blue Swirl To: Anthony Liguori Return-path: Received: from mail.codesourcery.com ([65.74.133.4]:35249 "EHLO mail.codesourcery.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752105AbZEZKUa (ORCPT ); Tue, 26 May 2009 06:20:30 -0400 In-Reply-To: <4A1BA998.8040803@codemonkey.ws> Content-Disposition: inline Sender: kvm-owner@vger.kernel.org List-ID: > > - /* FIXME: load/save binding. */ > > - //pci_device_save(&vdev->pci_dev, f); > > - //msix_save(&vdev->pci_dev, f); > > qdev regressed save/restore? What else is broken right now from the > qdev commit? > > I'm beginning to think committing in the state it was in was a mistake. > Paul, can you put together a TODO so that we know all of the things that > have regressed so we can get things back into shape? Sorry, this one apparently slipped past. I#'d intended to fix it, but apparently never ported that bit of the patch. Paul