Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20201230033950.13839-1-jiaxun.yang@flygoat.com> <20201230033950.13839-2-jiaxun.yang@flygoat.com>
In-Reply-To: <20201230033950.13839-2-jiaxun.yang@flygoat.com>
From:   Huacai Chen <chenhc@lemote.com>
Date:   Thu, 31 Dec 2020 08:27:03 +0800
Message-ID: <CAAhV-H58NOimxLCO7a48u5T2-kkpquJpyLveHrbKt00C756k8Q@mail.gmail.com>
Subject: Re: [PATCH 2/2] MIPS: Loongson64: Set cluster for cores
To:     Jiaxun Yang <jiaxun.yang@flygoat.com>
Cc:     "open list:MIPS" <linux-mips@vger.kernel.org>,
        Tiezhu Yang <yangtiezhu@loongson.cn>,
        Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
        LKML <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

Reviewed-by: Huacai Chen <chenhuacai@kernel.org>

On Wed, Dec 30, 2020 at 11:43 AM Jiaxun Yang <jiaxun.yang@flygoat.com> wrote:
>
> cluster is required for cacheinfo to set shared_cpu_map correctly.
>
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> Reviewed-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> Tested-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> ---
>  arch/mips/loongson64/smp.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/arch/mips/loongson64/smp.c b/arch/mips/loongson64/smp.c
> index e744e1bee49e..fae3a97d853c 100644
> --- a/arch/mips/loongson64/smp.c
> +++ b/arch/mips/loongson64/smp.c
> @@ -403,6 +403,8 @@ static void __init loongson3_smp_setup(void)
>                         __cpu_number_map[i] = num;
>                         __cpu_logical_map[num] = i;
>                         set_cpu_possible(num, true);
> +                       /* Loongson processors are always grouped by 4 */
> +                       cpu_set_cluster(&cpu_data[num], i / 4);
>                         num++;
>                 }
>                 i++;
> --
> 2.30.0
>