[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH 18/19] target-ppc: Enhance the CPU no
From: |
Andreas Färber |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH 18/19] target-ppc: Enhance the CPU node labels for the guest device tree for pseries. |
Date: |
Mon, 08 Jul 2013 18:45:27 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130620 Thunderbird/17.0.7 |
Hi,
Am 08.07.2013 17:49, schrieb Prerna Saxena:
> On 07/08/2013 02:32 PM, Andreas Färber wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> Am 08.07.2013 03:09, schrieb David Gibson:
>>> On Sat, Jul 06, 2013 at 11:54:15PM +1000, Alexey Kardashevskiy
>>> wrote:
>>>> @@ -1342,6 +1346,13 @@ static void
>>>> ppc_spapr_init(QEMUMachineInitArgs *args)
>>>> register_savevm_live(NULL, "spapr/htab", -1, 1,
>>>> &savevm_htab_handlers, spapr);
>>>>
>>>> + /* Ensure that cpu_model is correctly reflected for a KVM
>>>> guest */ + if (kvm_enabled() && !strcmp(cpu_model, "host")) {
>>>> + asm ("mfpvr %0" + : "=r"(pvr)); +
>>>> cpu_model = ppc_cpu_alias_by_pvr(pvr);
>>>
>>> This needs to be protected by an ifdef CONFIG_KVM or similar. If
>>> the compiler optimization level is turned down, so that it doesn't
>>> recognize that the kvm_enabled() is always false, then this could
>>> attempt to compile the ppc asm instructions on an x86 (or
>>> whatever) host.
>>
>> This hunk can be completely replaced by QOM mechanisms - just didn't
>> get to replying yet...
>
> Sorry I already sent out a v2, and only then saw your message. Could you
> pls explain how I could use QOM to replace this code block ?
Well, in short the thing is it has not much to do with KVM. The
KVM-specific host-powerpc64-cpu type is derived from the one you're
looking for and thus you can use object_class_get_parent() to obtain the
parent type and look at its name - stripping "-" TYPE_POWERPC_CPU from
it should be much more efficient but will give you the detailed name
including revision. I was planning to propose an alternative patch for that.
Replacing a concrete model name with its simpler alias is a secondary
issue (separate patch) that is not specific to KVM or -cpu host. Compare
-cpu POWER8_v1.0 printing .../address@hidden/... presumably.
Further, Alex has already applied a patch of his working around the
alias table being a rather archaic construct, not intended for frequent
use. Instead of adding even more functions that iterate it, we should
turn it into a hashtable for efficient lookup.
(Note that the cpu_model_str field may contain more than just the model
name, it is otherwise unused in softmmu and I was therefore preparing a
patch to ban its use to linux-user solely, so the type name seems the
most reliable indicator we have and as a bonus no PVR needed for it.)
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
- [Qemu-ppc] [PATCH 12/19] pseries: savevm support for pseries machine, (continued)
- [Qemu-ppc] [PATCH 12/19] pseries: savevm support for pseries machine, Alexey Kardashevskiy, 2013/07/06
- [Qemu-ppc] [PATCH 13/19] pseries: savevm support for PCI host bridge, Alexey Kardashevskiy, 2013/07/06
- [Qemu-ppc] [PATCH 14/19] target-ppc: Add helper for KVM_PPC_RTAS_DEFINE_TOKEN, Alexey Kardashevskiy, 2013/07/06
- [Qemu-ppc] [PATCH 15/19] pseries: Support for in-kernel XICS interrupt controller, Alexey Kardashevskiy, 2013/07/06
- [Qemu-ppc] [PATCH 16/19] pseries: savevm support with KVM, Alexey Kardashevskiy, 2013/07/06
- [Qemu-ppc] [PATCH 17/19] target-ppc: Add POWER8 v1.0 CPU model, Alexey Kardashevskiy, 2013/07/06
- [Qemu-ppc] [PATCH 18/19] target-ppc: Enhance the CPU node labels for the guest device tree for pseries., Alexey Kardashevskiy, 2013/07/06
- [Qemu-ppc] [PATCH v2 18/19] target-ppc: Enhance the CPU node labels for the guest device tree for pseries., Prerna Saxena, 2013/07/08
[Qemu-ppc] [PATCH 19/19] spapr-pci: rework MSI/MSIX, Alexey Kardashevskiy, 2013/07/06
[Qemu-ppc] [PATCH 08/19] pseries: savevm support for PAPR VIO logical tty, Alexey Kardashevskiy, 2013/07/06
Re: [Qemu-ppc] [PATCH 00/19 v4] spapr: migration, pci, msi, power8, Anthony Liguori, 2013/07/29