Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-12 Thread Alexey Kardashevskiy
On 07/10/2015 07:19 PM, Andrea Bolognani wrote: On Fri, 2015-07-10 at 15:19 +1000, Alexey Kardashevskiy wrote: If I'm reading the kernel source[1] correctly, there are actually subtle differences other than the number of cores: #define CPU_FTRS_POWER8 (/* Bunch of features here */) #d

Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-10 Thread Andrea Bolognani
On Fri, 2015-07-10 at 15:19 +1000, Alexey Kardashevskiy wrote: > > > If I'm reading the kernel source[1] correctly, there are actually > > subtle > > differences other than the number of cores: > > > >#define CPU_FTRS_POWER8 (/* Bunch of features here */) > >#define CPU_FTRS_POWER8E (CPU_

Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-09 Thread Alexey Kardashevskiy
On 07/09/2015 08:03 PM, Andrea Bolognani wrote: On Thu, 2015-07-09 at 12:35 +1000, Alexey Kardashevskiy wrote: Does "all versions of POWER8" include things like POWER8E, POWER8NVL and "POWER8 DD1", as one of the variations is known in the kernel source? Can we safely migrate guests eg. from a

Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-09 Thread Andrea Bolognani
On Thu, 2015-07-09 at 12:35 +1000, Alexey Kardashevskiy wrote: > > > Does "all versions of POWER8" include things like POWER8E, > > POWER8NVL > > and "POWER8 DD1", as one of the variations is known in the kernel > > source? Can we safely migrate guests eg. from a POWER8 v1.0 host to > > a POWER8E

Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-08 Thread Alexey Kardashevskiy
On 07/09/2015 02:35 AM, Andrea Bolognani wrote: On Wed, 2015-07-08 at 17:02 +1000, Alexey Kardashevskiy wrote: I think it's because libvirt likes to specify a specific CPU - and if it gets the new PVR from the host, qemu won't understand it. A specific CPU in this case is "POWER8", I added t

Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-08 Thread Andrea Bolognani
On Wed, 2015-07-08 at 17:02 +1000, Alexey Kardashevskiy wrote: > > > > > I think it's because libvirt likes to specify a specific CPU - > > > > and if > > > > it gets the new PVR from the host, qemu won't understand it. > > > > > > A specific CPU in this case is "POWER8", I added this > > > speci

Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-08 Thread Alexey Kardashevskiy
On 07/08/2015 04:45 PM, David Gibson wrote: On Wed, Jul 08, 2015 at 04:40:27PM +1000, Alexey Kardashevskiy wrote: On 07/08/2015 03:37 PM, David Gibson wrote: On Wed, Jul 08, 2015 at 02:10:28PM +1000, Alexey Kardashevskiy wrote: On 07/08/2015 01:43 PM, David Gibson wrote: From: Andrea Bolognan

Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-07 Thread David Gibson
On Wed, Jul 08, 2015 at 04:40:27PM +1000, Alexey Kardashevskiy wrote: > On 07/08/2015 03:37 PM, David Gibson wrote: > >On Wed, Jul 08, 2015 at 02:10:28PM +1000, Alexey Kardashevskiy wrote: > >>On 07/08/2015 01:43 PM, David Gibson wrote: > >>>From: Andrea Bolognani > >>> > >>>Add a missing PVR valu

Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-07 Thread Alexey Kardashevskiy
On 07/08/2015 03:37 PM, David Gibson wrote: On Wed, Jul 08, 2015 at 02:10:28PM +1000, Alexey Kardashevskiy wrote: On 07/08/2015 01:43 PM, David Gibson wrote: From: Andrea Bolognani Add a missing PVR value for the POWER8E v2.1 CPU. Information taken >from the kernel cputable. Signed-off-by

Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-07 Thread David Gibson
On Wed, Jul 08, 2015 at 02:10:28PM +1000, Alexey Kardashevskiy wrote: > On 07/08/2015 01:43 PM, David Gibson wrote: > >From: Andrea Bolognani > > > >Add a missing PVR value for the POWER8E v2.1 CPU. Information taken > >from the kernel cputable. > > > >Signed-off-by: Andrea Bolognani > >Signed-o

Re: [Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-07 Thread Alexey Kardashevskiy
On 07/08/2015 01:43 PM, David Gibson wrote: From: Andrea Bolognani Add a missing PVR value for the POWER8E v2.1 CPU. Information taken from the kernel cputable. Signed-off-by: Andrea Bolognani Signed-off-by: David Gibson --- target-ppc/cpu-models.c | 4 +++- target-ppc/cpu-models.h | 1 +

[Qemu-devel] [PATCH] target-ppc: Add POWER8E_v2.1 CPU model.

2015-07-07 Thread David Gibson
From: Andrea Bolognani Add a missing PVR value for the POWER8E v2.1 CPU. Information taken from the kernel cputable. Signed-off-by: Andrea Bolognani Signed-off-by: David Gibson --- target-ppc/cpu-models.c | 4 +++- target-ppc/cpu-models.h | 1 + 2 files changed, 4 insertions(+), 1 deletion(-