Re: [Qemu-devel] [PATCH 1/2] target-arm: sort TCG cpreg list by 64bit id version

2013-10-25 Thread Peter Maydell
On 11 October 2013 18:38, Alvise Rigo wrote: > Both KVM and TCG populate the cpreg_list with 64 bit registers IDs, but in > the TCG side the cpreg_list is sorted using the 32 bit id version while in > the kvm side the 64 bit id version is used. > This patch makes the sorting of the cpreg_list co

Re: [Qemu-devel] [PATCH 1/2] target-arm: sort TCG cpreg list by 64bit id version

2013-10-14 Thread Peter Maydell
On 14 October 2013 11:45, alvise rigo wrote: > Sorry, I should have mentioned that an improper sorting of the cpreg_list > could lead to a migration failure when cpu_post_load considers an incoming > register as missing in the cpreg_indexes array. > However, as long as the two lists are exactly th

Re: [Qemu-devel] [PATCH 1/2] target-arm: sort TCG cpreg list by 64bit id version

2013-10-14 Thread alvise rigo
Sorry, I should have mentioned that an improper sorting of the cpreg_list could lead to a migration failure when cpu_post_load considers an incoming register as missing in the cpreg_indexes array. However, as long as the two lists are exactly the same, the problem does not occur. On Fri, Oct 11, 2

Re: [Qemu-devel] [PATCH 1/2] target-arm: sort TCG cpreg list by 64bit id version

2013-10-11 Thread Peter Maydell
On 12 October 2013 02:38, Alvise Rigo wrote: > Both KVM and TCG populate the cpreg_list with 64 bit registers IDs, but in > the TCG side the cpreg_list is sorted using the 32 bit id version while in > the kvm side the 64 bit id version is used. > This patch makes the sorting of the cpreg_list co

[Qemu-devel] [PATCH 1/2] target-arm: sort TCG cpreg list by 64bit id version

2013-10-11 Thread Alvise Rigo
Both KVM and TCG populate the cpreg_list with 64 bit registers IDs, but in the TCG side the cpreg_list is sorted using the 32 bit id version while in the kvm side the 64 bit id version is used. This patch makes the sorting of the cpreg_list consistent between KVM and TCG. Signed-off-by: Alvise R