Re: [Qemu-devel] [PATCH v2 0/8] target-sparc: Update to use VMStateDescription

[Peter Maydell]

On 15 January 2016 at 17:03, Mark Cave-Ayland
<address@hidden> wrote:
> I've just run through some SPARC32 tests with this patchset applied -
> testing a savevm/loadvm pair pre-patch and post-patch, and then a few
> random tests with various OSs at random points during boot and no
> regressions here, so:
>
> Tested-by: Mark Cave-Ayland <address@hidden>

Thanks.

> The only minor nit I could see during review was that both patches 7 and
> 8 bump the VMState revision for SPARC64, and given that before the last
> patch in the set nothing works then is it just worth bumping once to
> version 8 in patch 7 and let that be it?

My general feeling is that there isn't a significant shortage
of positive integers that would lead us to want to conserve them :-)
If you prefer only bumping once you can do that when you apply the
patches to your target-sparc queue if you like.

thanks
-- PMM