Re: [Qemu-devel] [PATCH v7] powerpc: add PVR mask support
On 23.10.2013, at 07:57, Andreas Färber afaer...@suse.de wrote: Am 27.09.2013 09:05, schrieb Alexey Kardashevskiy: IBM POWERPC processors encode PVR as a CPU family in higher 16 bits and a CPU version in lower 16 bits. Since there is no significant change in behavior between versions, there is no point to add every single CPU version in QEMU's CPU list. Also, new CPU versions of already supported CPU won't break the existing code. This adds PVR value/mask support for KVM, i.e. for -cpu host option. As CPU family class name for POWER7 is POWER7-family, there is no need to touch aliases. Cc: Andreas Färber afaer...@suse.de Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru As promised to Paul, using the Hackathon timeslot to review this: Reviewed-by: Andreas Färber afaer...@suse.de Thanks, applied to ppc-next-1.8 Alex
Re: [Qemu-devel] [PATCH v7] powerpc: add PVR mask support
Am 27.09.2013 09:05, schrieb Alexey Kardashevskiy: IBM POWERPC processors encode PVR as a CPU family in higher 16 bits and a CPU version in lower 16 bits. Since there is no significant change in behavior between versions, there is no point to add every single CPU version in QEMU's CPU list. Also, new CPU versions of already supported CPU won't break the existing code. This adds PVR value/mask support for KVM, i.e. for -cpu host option. As CPU family class name for POWER7 is POWER7-family, there is no need to touch aliases. Cc: Andreas Färber afaer...@suse.de Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru As promised to Paul, using the Hackathon timeslot to review this: Reviewed-by: Andreas Färber afaer...@suse.de and sorry for the delay. Regards, Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
Re: [Qemu-devel] [PATCH v7] powerpc: add PVR mask support
On 10/14/2013 02:04 PM, Alexey Kardashevskiy wrote: On 09/30/2013 09:30 PM, Alexander Graf wrote: On 27.09.2013, at 10:05, Alexey Kardashevskiy wrote: IBM POWERPC processors encode PVR as a CPU family in higher 16 bits and a CPU version in lower 16 bits. Since there is no significant change in behavior between versions, there is no point to add every single CPU version in QEMU's CPU list. Also, new CPU versions of already supported CPU won't break the existing code. This adds PVR value/mask support for KVM, i.e. for -cpu host option. As CPU family class name for POWER7 is POWER7-family, there is no need to touch aliases. Cc: Andreas Färber afaer...@suse.de Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru Looks reasonable to me, but I'll wait for an ack from Andreas. Ping, anyone? Ping, anyone, please? -- Alexey
Re: [Qemu-devel] [PATCH v7] powerpc: add PVR mask support
On 09/30/2013 09:30 PM, Alexander Graf wrote: On 27.09.2013, at 10:05, Alexey Kardashevskiy wrote: IBM POWERPC processors encode PVR as a CPU family in higher 16 bits and a CPU version in lower 16 bits. Since there is no significant change in behavior between versions, there is no point to add every single CPU version in QEMU's CPU list. Also, new CPU versions of already supported CPU won't break the existing code. This adds PVR value/mask support for KVM, i.e. for -cpu host option. As CPU family class name for POWER7 is POWER7-family, there is no need to touch aliases. Cc: Andreas Färber afaer...@suse.de Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru Looks reasonable to me, but I'll wait for an ack from Andreas. Ping, anyone? -- Alexey
Re: [Qemu-devel] [PATCH v7] powerpc: add PVR mask support
On 27.09.2013, at 10:05, Alexey Kardashevskiy wrote: IBM POWERPC processors encode PVR as a CPU family in higher 16 bits and a CPU version in lower 16 bits. Since there is no significant change in behavior between versions, there is no point to add every single CPU version in QEMU's CPU list. Also, new CPU versions of already supported CPU won't break the existing code. This adds PVR value/mask support for KVM, i.e. for -cpu host option. As CPU family class name for POWER7 is POWER7-family, there is no need to touch aliases. Cc: Andreas Färber afaer...@suse.de Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru Looks reasonable to me, but I'll wait for an ack from Andreas. Alex
