Re: [PATCH v10] xenpm: Add get-core-temp subcommand

["Anthony PERARD" <anthony.perard@xxxxxxxxxx>]

On Tue, Apr 07, 2026 at 02:10:37PM +0000, Teddy Astie wrote:
> +    for ( cpu = 0; cpu < max_cpu_nr; cpu += physinfo.threads_per_core )
> +    {
> +        if ( fetch_dts_temp(xc_handle, cpu, false, &temp) )
> +        {
> +            fprintf(stderr, "[CPU%d] Unable to fetch temperature (%d - 
> %s)\n",
> +                    cpu, errno, strerror(errno));
> +            continue;
> +        }
> +

A trailing whitespace was introduced here since v9.

> +        has_data = true;
> +        printf("CPU%d: %d°C\n", cpu, temp);
> +    }

Beside that, patch looks good to me now:
Reviewed-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

Thanks,


--
 | Vates

XCP-ng & Xen Orchestra - Vates solutions

web: https://vates.tech