From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:36389) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UfTdc-0006rg-JE for qemu-devel@nongnu.org; Thu, 23 May 2013 07:24:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UfTdX-0007cr-8n for qemu-devel@nongnu.org; Thu, 23 May 2013 07:24:52 -0400 Date: Thu, 23 May 2013 16:54:29 +0530 From: Amit Shah Message-ID: <20130523112429.GB10951@amit.redhat.com> References: <1369240371-21253-1-git-send-email-mdroth@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1369240371-21253-1-git-send-email-mdroth@linux.vnet.ibm.com> Subject: Re: [Qemu-devel] [PATCH v2] wdt_i6300esb: fix vmstate versioning List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Michael Roth Cc: peter.maydell@linaro.org, nick@bytemark.co.uk, lersek@redhat.com, qemu-devel@nongnu.org, qemu-stable@nongnu.org On (Wed) 22 May 2013 [11:32:51], Michael Roth wrote: > When this VMSD was introduced it's version fields were set to > sizeof(I6300State), making them essentially random from build to build, > version to version. > > To fix this, we lock in a high version id and low minimum version id to > support old->new migration from all prior versions of this device's > state. This should work since the device state has not changed since > its introduction. > > The potentially breaks migration from 1.5+ to 1.5, but since the > versioning was essentially random prior to this patch, new->old > migration was not consistently functional to begin with. > > Reported-by: Nicholas Thomas > Suggested-by: Peter Maydell > Cc: qemu-stable@nongnu.org > Signed-off-by: Michael Roth > --- > v2: > * Fixed s/except/accept/ typo (Laszlo) Reviewed-by: Amit Shah Amit