On Tue, May 13 2025, Eric Auger <eric.au...@redhat.com> wrote:
> On 4/14/25 6:38 PM, Cornelia Huck wrote:
>> From: Eric Auger <eric.au...@redhat.com>
(...)
>> +static bool
>> +kvm_arm_get_host_cpu_features(ARMCPU *cpu, ARMHostCPUFeatures *ahcf,
>> + bool exhaustive)
> maybe the exhaustive term shall be replaced by handle_writable_id_regs
> which tells more explicitly what it does
Maybe get_all_writable_id_regs?
>> {
>> /* Identify the feature bits corresponding to the host CPU, and
>> * fill out the ARMHostCPUClass fields accordingly. To do this
(...)
>> diff --git a/target/arm/kvm_arm.h b/target/arm/kvm_arm.h
>> index 8d1f20ca8d89..90ba4f7d8987 100644
>> --- a/target/arm/kvm_arm.h
>> +++ b/target/arm/kvm_arm.h
>> @@ -141,8 +141,12 @@ uint32_t kvm_arm_sve_get_vls(ARMCPU *cpu);
>> *
>> * Set up the ARMCPU struct fields up to match the information probed
>> * from the host CPU.
>> + *
>> + * @cpu: cpu object
>> + * @exhaustive: if true, all the feature ID regs are queried instead of
> s/queried/handled
>> + * a subset
And make this "if true, additionally get the contents of all writable ID
registers as well"?
>> */
>> -void kvm_arm_set_cpu_features_from_host(ARMCPU *cpu);
>> +void kvm_arm_set_cpu_features_from_host(ARMCPU *cpu, bool exhaustive);
