[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-6.2 v3 01/11] machine: Minor refactor/cleanup for the smp
From: |
Andrew Jones |
Subject: |
Re: [PATCH for-6.2 v3 01/11] machine: Minor refactor/cleanup for the smp parsers |
Date: |
Wed, 28 Jul 2021 22:16:49 +0200 |
On Wed, Jul 28, 2021 at 11:48:38AM +0800, Yanan Wang wrote:
> To pave the way for the functional improvement in later patches,
> make some refactor/cleanup for the smp parsers, including using
> local maxcpus instead of ms->smp.max_cpus in the calculation,
> defaulting dies to 0 initially like other members, cleanup the
> sanity check for dies.
>
> No functional change intended.
>
> Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
> ---
> hw/core/machine.c | 19 +++++++++++--------
> hw/i386/pc.c | 23 ++++++++++++++---------
> 2 files changed, 25 insertions(+), 17 deletions(-)
>
> diff --git a/hw/core/machine.c b/hw/core/machine.c
> index e1533dfc47..ffc0629854 100644
> --- a/hw/core/machine.c
> +++ b/hw/core/machine.c
> @@ -747,9 +747,11 @@ static void smp_parse(MachineState *ms, SMPConfiguration
> *config, Error **errp)
> unsigned sockets = config->has_sockets ? config->sockets : 0;
> unsigned cores = config->has_cores ? config->cores : 0;
> unsigned threads = config->has_threads ? config->threads : 0;
> + unsigned maxcpus = config->has_maxcpus ? config->maxcpus : 0;
>
> - if (config->has_dies && config->dies != 0 && config->dies != 1) {
> + if (config->has_dies && config->dies > 1) {
> error_setg(errp, "dies not supported by this machine's CPU
> topology");
> + return;
> }
>
> /* compute missing values, prefer sockets over cores over threads */
> @@ -760,8 +762,8 @@ static void smp_parse(MachineState *ms, SMPConfiguration
> *config, Error **errp)
> sockets = sockets > 0 ? sockets : 1;
> cpus = cores * threads * sockets;
> } else {
> - ms->smp.max_cpus = config->has_maxcpus ? config->maxcpus : cpus;
> - sockets = ms->smp.max_cpus / (cores * threads);
> + maxcpus = maxcpus > 0 ? maxcpus : cpus;
> + sockets = maxcpus / (sockets * cores);
Should be divided by (cores * threads) like before.
Thanks,
drew
- [PATCH for-6.2 v3 00/11] machine: smp parsing fixes and improvement, Yanan Wang, 2021/07/27
- [PATCH for-6.2 v3 05/11] hw: Add compat machines for 6.2, Yanan Wang, 2021/07/27
- [PATCH for-6.2 v3 01/11] machine: Minor refactor/cleanup for the smp parsers, Yanan Wang, 2021/07/27
- Re: [PATCH for-6.2 v3 01/11] machine: Minor refactor/cleanup for the smp parsers,
Andrew Jones <=
- [PATCH for-6.2 v3 08/11] machine: Tweak the order of topology members in struct CpuTopology, Yanan Wang, 2021/07/27
- [PATCH for-6.2 v3 04/11] machine: Improve the error reporting of smp parsing, Yanan Wang, 2021/07/27
- [PATCH for-6.2 v3 02/11] machine: Uniformly use maxcpus to calculate the omitted parameters, Yanan Wang, 2021/07/27
- [PATCH for-6.2 v3 07/11] machine: Use ms instead of global current_machine in sanity-check, Yanan Wang, 2021/07/27
- [PATCH for-6.2 v3 10/11] machine: Remove smp_parse callback from MachineClass, Yanan Wang, 2021/07/27