Le 10/12/2025 à 09:32, Jan Beulich a écrit :
> On 09.12.2025 18:19, Teddy Astie wrote:
>> Intel provide CPU sensors through "DTS" MSRs. As there MSR are core-specific
>
> Nit: s/there MSR/these MSRs/ ?
>

fixed

>> @@ -86,6 +87,11 @@ static bool msr_read_allowed(unsigned int msr)
>>
>>       case MSR_MCU_OPT_CTRL:
>>           return cpu_has_srbds_ctrl;
>> +
>> +    case MSR_IA32_THERM_STATUS:
>> +    case MSR_TEMPERATURE_TARGET:
>> +    case MSR_PACKAGE_THERM_STATUS:
>> +        return raw_cpu_policy.basic.pm.dts;
>
> I'm pretty sure it was indicated before that the raw policy is likely wrong to
> use for anything like this. The host policy wants using instead, or else 
> specific
> justification should be provided.
>

Indeed, host_cpu_policy is intended here.

> Jan



--
Teddy Astie | Vates XCP-ng Developer

XCP-ng & Xen Orchestra - Vates solutions

web: https://vates.tech



Reply via email to