Re: [Qemu-devel] [PATCH 0/2] support migration/save/load on AArch64 CPUs

[Pranavkumar Sawargaonkar]

Hi PMM,

On 5 December 2014 at 19:41, Peter Maydell <address@hidden> wrote:
> These patches implement support for migration/save/load on AArch64
> CPUs. The first one from Alex (with some mangling from me) just
> moves the sysreg sync code we have for 32-bit across to 64-bit.
> The second fills in the gaps in the main CPU vmstate struct.
>
> Notes:
>  * currently doing 'from cold' incoming migration with -loadvm
>    or -incoming command line arguments is broken if using the VGIC;
>    this is a kernel bug, fixed by this patch:
>    http://www.spinics.net/lists/arm-kernel/msg383736.html
>  * Pranav: patch 1 here is what you need to rebase the bigendian
>    virtio patch on so it can use a single common hook for 32 and
>    64 bit without hacks to work around missing register sync

Thanks for the patch.
I have applied this patch and called cpu_synchronize_state() to get
sctlr_el1 value in env->cp15.c1_sys  during virtio endianness
determination while ARM64 guest was booting , but value getting read
is 0.
Do I need to do anything else ?

Thanks,
Pranav

>  * Christoffer: patch 1 is also the one which moves reset around
>
> Alex Bennée (1):
>   target-arm/kvm: make reg sync code common between kvm32/64
>
> Peter Maydell (1):
>   target-arm: Support save/load for 64 bit CPUs
>
>  target-arm/kvm.c     | 98 
> ++++++++++++++++++++++++++++++++++++++++++++++++++++
>  target-arm/kvm32.c   | 94 ++-----------------------------------------------
>  target-arm/kvm64.c   | 24 +++++++------
>  target-arm/kvm_arm.h | 22 ++++++++++++
>  target-arm/machine.c | 22 ++++++++++--
>  5 files changed, 156 insertions(+), 104 deletions(-)
>
> --
> 1.9.1
>