Just a ping...
Hi Sunil, could you please take some time to look at this patch? Thank
you!
Regards,
Zhao
On Mon, May 29, 2023 at 08:43:31PM +0800, Zhao Liu wrote:
> Date: Mon, 29 May 2023 20:43:31 +0800
> From: Zhao Liu <zhao1....@linux.intel.com>
> Subject: [PATCH] i386/WHPX: Fix error message when fail to set
> ProcessorCount
> X-Mailer: git-send-email 2.34.1
>
> From: Zhao Liu <zhao1....@intel.com>
>
> 003f230e37d7 ("machine: Tweak the order of topology members in struct
> CpuTopology") changes the meaning of MachineState.smp.cores from "the
> number of cores in one package" to "the number of cores in one die"
> and doesn't fix other uses of MachineState.smp.cores. And because of
> the introduction of cluster, now smp.cores just means "the number of
> cores in one cluster". This clearly does not fit the semantics here.
>
> And before this error message, WHvSetPartitionProperty() is called to
> set prop.ProcessorCount.
>
> So the error message should show the prop.ProcessorCount other than
> "cores per cluster" or "cores per package".
>
> Cc: Sunil Muthuswamy <sunil...@microsoft.com>
> Signed-off-by: Zhao Liu <zhao1....@intel.com>
> ---
> target/i386/whpx/whpx-all.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/target/i386/whpx/whpx-all.c b/target/i386/whpx/whpx-all.c
> index 52af81683c1e..5882bf22d0a1 100644
> --- a/target/i386/whpx/whpx-all.c
> +++ b/target/i386/whpx/whpx-all.c
> @@ -2613,8 +2613,8 @@ static int whpx_accel_init(MachineState *ms)
> sizeof(WHV_PARTITION_PROPERTY));
>
> if (FAILED(hr)) {
> - error_report("WHPX: Failed to set partition core count to %d,"
> - " hr=%08lx", ms->smp.cores, hr);
> + error_report("WHPX: Failed to set partition processor count to %d,"
> + " hr=%08lx", prop.ProcessorCount, hr);
> ret = -EINVAL;
> goto error;
> }
> --
> 2.34.1
>
