Re: [PATCH v16 07/11] target/s390x/cpu topology: activating CPU topology

Pierre Morel
2023-02-27T14:13:50Z

On 2/27/23 14:26, Thomas Huth wrote:
> On 22/02/2023 15.21, Pierre Morel wrote:
>> The KVM capability KVM_CAP_S390_CPU_TOPOLOGY is used to
>> activate the S390_FEAT_CONFIGURATION_TOPOLOGY feature and
>> the topology facility in the host CPU model for the guest
>> in the case the topology is available in QEMU and in KVM.
>>
>> The feature is disabled by default and fenced for SE
>> (secure execution).
>>
>> Signed-off-by: Pierre Morel <pmorel@linux.ibm.com>
>> ---
>>   hw/s390x/cpu-topology.c   | 2 +-
>>   target/s390x/cpu_models.c | 1 +
>>   target/s390x/kvm/kvm.c    | 9 +++++++++
>>   3 files changed, 11 insertions(+), 1 deletion(-)
>
> Maybe s/activating/activate/ in the subject?

Yes, Thanks.

>
> Anyway:
> Reviewed-by: Thomas Huth <thuth@redhat.com>
>
Thanks,

Regards,

Pierre