From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=34458 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Q2NOk-0001R1-2M for qemu-devel@nongnu.org; Wed, 23 Mar 2011 08:42:51 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Q2NOh-0002fs-OQ for qemu-devel@nongnu.org; Wed, 23 Mar 2011 08:42:48 -0400 Received: from mail-gx0-f173.google.com ([209.85.161.173]:46194) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Q2NOh-0002ff-LN for qemu-devel@nongnu.org; Wed, 23 Mar 2011 08:42:47 -0400 Received: by gxk26 with SMTP id 26so4051645gxk.4 for ; Wed, 23 Mar 2011 05:42:46 -0700 (PDT) Message-ID: <4D89EABE.50204@codemonkey.ws> Date: Wed, 23 Mar 2011 07:42:38 -0500 From: Anthony Liguori MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH 11/11] test-vmstate: add test case to verify we don't change VMState References: <1300839376-22520-1-git-send-email-aliguori@us.ibm.com> <1300839376-22520-12-git-send-email-aliguori@us.ibm.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Jan Kiszka , qemu-devel@nongnu.org, Juan Quintela On 03/23/2011 05:22 AM, Peter Maydell wrote: > On 23 March 2011 00:16, Anthony Liguori wrote: >> + if (old_version != new_version) { >> + g_error("Version %d of device `%s' is available in QEMU, but schema still reports %d, please update schema.\n", >> + new_version, device, old_version); >> + } > Might be nice for these "please update" error messages to > include a pointer to a docs file explaining in more detail > how to do that? > (also>80 char line ;-)) Ack. >> diff --git a/vmstate/schema.json b/vmstate/schema.json >> new file mode 100644 >> index 0000000..23483ab >> --- /dev/null >> +++ b/vmstate/schema.json >> @@ -0,0 +1,1176 @@ >> +{ >> + "cpu": { >> + "mcg_cap": "uint64", >> + "a20_mask": "int32", >> + "tsc_offset": "uint64", > This schema file appears to be board-specific (or at least > x86-specific) -- shouldn't the cpu/board/whatever name > be in the filename, so we have scope to expand the test > to checking migration issues for other platforms too? It's not really. Every VMStateDescription that is builtin into the tree is in the file. That said, the only target where the CPU is currently described by VMStateDescription is target-i386. Right now the file is generated via i386-softmmu. There may be a few devices left out because they are either not compiled into i386-softmmu or are target specific. We could complicate things further by trying to run against every target and then building a union of all target outputs but I'm not sure it's worth the effort at this stage. > (I don't care much about ARM migration breakages just at the > moment but I suspect that it will be becoming more important > by this time next year...) > > Also since this looks like an autogenerated file that's going > to be going into version control maybe it should have a > comment header at the top of the "autogenerated, do not edit > by hand!" type. JSON doesn't support comments.. I can add comment parsing to our parser though. Regards, Anthony Liguori > -- PMM >