On 12/3/2025 11:06 AM, Stanislav Kinsburskii wrote:
> On Wed, Dec 03, 2025 at 09:53:24AM -0800, Nuno Das Neves wrote:
>> Add the definitions for hypervisor, logical processor, and partition
>> stats pages.
>>
>> Move the definition for the VP stats page to its rightful place in
>> hvhdk.h, and add the missing members.
>>
>> These enum members retain their CamelCase style, since they are imported
>> directly from the hypervisor code They will be stringified when printing
>> the stats out, and retain more readability in this form.
>>
>> Signed-off-by: Nuno Das Neves <[email protected]>
>> ---
>>  drivers/hv/mshv_root_main.c |  17 --
>>  include/hyperv/hvhdk.h      | 437 ++++++++++++++++++++++++++++++++++++
>>  2 files changed, 437 insertions(+), 17 deletions(-)
>>
> 
> <snip>
> 
>> +
>> +enum hv_stats_partition_counters {          /* HV_PROCESS_COUNTER */
>> +    PartitionVirtualProcessors              = 1,
>> +    PartitionTlbSize                        = 3,
>> +    PartitionAddressSpaces                  = 4,
>> +    PartitionDepositedPages                 = 5,
>> +    PartitionGpaPages                       = 6,
>> +    PartitionGpaSpaceModifications          = 7,
>> +    PartitionVirtualTlbFlushEntires         = 8,
>> +    PartitionRecommendedTlbSize             = 9,
>> +    PartitionGpaPages4K                     = 10,
>> +    PartitionGpaPages2M                     = 11,
>> +    PartitionGpaPages1G                     = 12,
>> +    PartitionGpaPages512G                   = 13,
>> +    PartitionDevicePages4K                  = 14,
>> +    PartitionDevicePages2M                  = 15,
>> +    PartitionDevicePages1G                  = 16,
>> +    PartitionDevicePages512G                = 17,
>> +    PartitionAttachedDevices                = 18,
>> +    PartitionDeviceInterruptMappings        = 19,
>> +    PartitionIoTlbFlushes                   = 20,
>> +    PartitionIoTlbFlushCost                 = 21,
>> +    PartitionDeviceInterruptErrors          = 22,
>> +    PartitionDeviceDmaErrors                = 23,
>> +    PartitionDeviceInterruptThrottleEvents  = 24,
>> +    PartitionSkippedTimerTicks              = 25,
>> +    PartitionPartitionId                    = 26,
>> +#if IS_ENABLED(CONFIG_X86)
> 
> Why isn't this CONFIG_X86_64 instead (here and below)?

An oversight, thanks for catching it.
I'll change it to CONFIG_X86_64

> 
> Thanks,
> Stanislav


Reply via email to