[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4] ppc: introduce CPUPPCState::cpu_dt_id and CP
From: |
Alexey Kardashevskiy |
Subject: |
Re: [Qemu-devel] [PATCH v4] ppc: introduce CPUPPCState::cpu_dt_id and CPUState::kvm_cpu_id |
Date: |
Sat, 16 Nov 2013 04:02:06 +1100 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.0 |
On 16.11.2013 2:11, Andreas Färber wrote:
> Am 15.11.2013 06:14, schrieb Alexey Kardashevskiy:
>> diff --git a/include/qom/cpu.h b/include/qom/cpu.h
>> index 7739e00..52fc76d 100644
>> --- a/include/qom/cpu.h
>> +++ b/include/qom/cpu.h
>> @@ -197,6 +197,7 @@ struct CPUState {
>> bool kvm_vcpu_dirty;
>> struct KVMState *kvm_state;
>> struct kvm_run *kvm_run;
>> + int kvm_cpu_id;
>>
>> /* TODO Move common fields from CPUArchState here. */
>> int cpu_index; /* used by alpha TCG */
>
> Here you are adding a field to CPUState, fine with me. (Please add a
> documentation line above the struct then.)
>
>> diff --git a/target-ppc/cpu.h b/target-ppc/cpu.h
>> index bb84767..8052f6b 100644
>> --- a/target-ppc/cpu.h
>> +++ b/target-ppc/cpu.h
>> @@ -1074,6 +1074,9 @@ struct CPUPPCState {
>> */
>> uint8_t fit_period[4];
>> uint8_t wdt_period[4];
>> +
>> + /* The CPU index used in the device tree. KVM uses this index too */
>> + int cpu_dt_id;
>
> But I believe I have requested a number of times not to add random
> fields to CPUPPCState unless they are accessed by TCG. Please place the
> new field in PowerPCCPU instead and put the description into the struct
> documentation.
It is accessed by xics.c which is used in TCG. Or I misinterpret the
rule, do not I?
--
With best regards
Alexey Kardashevskiy -- icq: 52150396