[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code
From: |
Nikunj A Dadhania |
Subject: |
Re: [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code |
Date: |
Mon, 30 Mar 2015 11:04:19 +0530 |
User-agent: |
Notmuch/0.17+27~gae47d61 (http://notmuchmail.org) Emacs/24.3.1 (x86_64-redhat-linux-gnu) |
Nikunj A Dadhania <address@hidden> writes:
> David Gibson <address@hidden> writes:
>
>> On Mon, Mar 30, 2015 at 01:18:01PM +1100, Alexey Kardashevskiy wrote:
>>> On 03/27/2015 08:49 PM, Nikunj A Dadhania wrote:
>>> >Each hardware instance has a platform unique location code. The OF
>>> >device tree that describes a part of a hardware entity must include
>>> >the “ibm,loc-code” property with a value that represents the location
>>> >code for that hardware entity.
>>> >
>>> >Introduce an hcall to populate ibm,loc-code.
>>> >1) PCI passthru devices need to identify with its own ibm,loc-code
>>> > available on the host.
>>> >2) Emulated devices encode as following: qemu_<name>:<slot>.<fn>
>>> >
>>> >Signed-off-by: Nikunj A Dadhania <address@hidden>
>> [snip]
>>> >diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h
>>> >index af71e8b..95157ac 100644
>>> >--- a/include/hw/ppc/spapr.h
>>> >+++ b/include/hw/ppc/spapr.h
>>> >@@ -310,7 +310,10 @@ typedef struct sPAPREnvironment {
>>> > #define KVMPPC_H_LOGICAL_MEMOP (KVMPPC_HCALL_BASE + 0x1)
>>> > /* Client Architecture support */
>>> > #define KVMPPC_H_CAS (KVMPPC_HCALL_BASE + 0x2)
>>> >-#define KVMPPC_HCALL_MAX KVMPPC_H_CAS
>>> >+#define KVMPPC_H_RTAS_UPDATE (KVMPPC_HCALL_BASE + 0x3)
>>> >+#define KVMPPC_H_REPORT_MC_ERR (KVMPPC_HCALL_BASE + 0x4)
>>> >+#define KVMPPC_H_GET_LOC_CODE (KVMPPC_HCALL_BASE + 0x5)
>>> >+#define KVMPPC_HCALL_MAX KVMPPC_H_GET_LOC_CODE
>>>
>>>
>>> Please add only relevant codes. And what happened to patches adding
>>> H_RTAS_UPDATE and H_REPORT_MC_ERR?
>>>
>>> Also (it is probably a very stupid question but still :) ), why are all
>>> these callbacks - hypercalls, not RTAS calls? The hypercalls are numbered in
>>> sPAPR and we kind of stealing numbers from that space while we are
>>> allocating RTAS tokens ourselves and have more freedom.
>>
>> Also, I thought the plan was to remove PCI device enumeration from
>> SLOF and move it to qemu (since we need to partially do that for
>> hotplug).
>
> For me it was a short term plan.
Sorry, I meant PCI device enumeration removal from SLOF was a long term
plan.
> IMHO, this cant be done in short time.
>
>> That removes the need for the hcall entirely.
>>
>> --
>> David Gibson | I'll have my music baroque, and my code
>> david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_
>> _other_
>> | _way_ _around_!
>> http://www.ozlabs.org/~dgibson
- [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code, Nikunj A Dadhania, 2015/03/27
- Re: [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code, Alexey Kardashevskiy, 2015/03/29
- Re: [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code, David Gibson, 2015/03/29
- Re: [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code, Alexey Kardashevskiy, 2015/03/29
- Re: [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code, Nikunj A Dadhania, 2015/03/30
- Re: [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code,
Nikunj A Dadhania <=
- Re: [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code, Alexey Kardashevskiy, 2015/03/30
- Re: [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code, Nikunj A Dadhania, 2015/03/30
Re: [Qemu-ppc] [PATCH v2] spapr: populate ibm,loc-code, Nikunj A Dadhania, 2015/03/30