diff mbox series

MIPS: Loongson: Use CONFIG_NR_CPUS_DEFAULT_64 to support more CPUs

Message ID 1585638006-19439-1-git-send-email-yangtiezhu@loongson.cn (mailing list archive)
State Mainlined
Commit 5125bfeeb6e4ffc47a5cb2715c342c938854cf20
Headers show
Series MIPS: Loongson: Use CONFIG_NR_CPUS_DEFAULT_64 to support more CPUs | expand

Commit Message

Tiezhu Yang March 31, 2020, 7 a.m. UTC
When I update the mainline kernel on the Loongson 2-way platform which
has 8 CPUs, it only shows 4 CPUs due to NR_CPUS is 4, this is obviously
wrong.

In order to support more CPUs on the Loongson platform, it is better
to use CONFIG_NR_CPUS_DEFAULT_64 instead of CONFIG_NR_CPUS_DEFAULT_4
to specify the maximum number of CPUs which the kernel will support.

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
 arch/mips/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jiaxun Yang March 31, 2020, 7:05 a.m. UTC | #1
于 2020年3月31日 GMT+08:00 下午3:00:06, Tiezhu Yang <yangtiezhu@loongson.cn> 写到:
>When I update the mainline kernel on the Loongson 2-way platform which
>has 8 CPUs, it only shows 4 CPUs due to NR_CPUS is 4, this is obviously
>wrong.
>
>In order to support more CPUs on the Loongson platform, it is better
>to use CONFIG_NR_CPUS_DEFAULT_64 instead of CONFIG_NR_CPUS_DEFAULT_4
>to specify the maximum number of CPUs which the kernel will support.
>
>Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>

Would 16 be a better option?
We have only 4 way platform.

>---
> arch/mips/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig
>index a1f973c..a0b11f5 100644
>--- a/arch/mips/Kconfig
>+++ b/arch/mips/Kconfig
>@@ -475,7 +475,7 @@ config MACH_LOONGSON64
> 	select ISA
> 	select I8259
> 	select IRQ_MIPS_CPU
>-	select NR_CPUS_DEFAULT_4
>+	select NR_CPUS_DEFAULT_64
> 	select USE_GENERIC_EARLY_PRINTK_8250
> 	select SYS_HAS_CPU_LOONGSON64
> 	select SYS_HAS_EARLY_PRINTK
Tiezhu Yang March 31, 2020, 7:21 a.m. UTC | #2
On 03/31/2020 03:05 PM, Jiaxun Yang wrote:

>
> 于 2020年3月31日 GMT+08:00 下午3:00:06, Tiezhu Yang <yangtiezhu@loongson.cn> 写到:
>> When I update the mainline kernel on the Loongson 2-way platform which
>> has 8 CPUs, it only shows 4 CPUs due to NR_CPUS is 4, this is obviously
>> wrong.
>>
>> In order to support more CPUs on the Loongson platform, it is better
>> to use CONFIG_NR_CPUS_DEFAULT_64 instead of CONFIG_NR_CPUS_DEFAULT_4
>> to specify the maximum number of CPUs which the kernel will support.
>>
>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> Would 16 be a better option?
> We have only 4 way platform.

There maybe have more than 4 cores per CPU or more than 4 ways
in the future, so I think 64 is better, otherwise we need to
modify this configuration once more.

>
>> ---
>> arch/mips/Kconfig | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig
>> index a1f973c..a0b11f5 100644
>> --- a/arch/mips/Kconfig
>> +++ b/arch/mips/Kconfig
>> @@ -475,7 +475,7 @@ config MACH_LOONGSON64
>> 	select ISA
>> 	select I8259
>> 	select IRQ_MIPS_CPU
>> -	select NR_CPUS_DEFAULT_4
>> +	select NR_CPUS_DEFAULT_64
>> 	select USE_GENERIC_EARLY_PRINTK_8250
>> 	select SYS_HAS_CPU_LOONGSON64
>> 	select SYS_HAS_EARLY_PRINTK
Jiaxun Yang March 31, 2020, 7:22 a.m. UTC | #3
于 2020年3月31日 GMT+08:00 下午3:21:00, Tiezhu Yang <yangtiezhu@loongson.cn> 写到:
>On 03/31/2020 03:05 PM, Jiaxun Yang wrote:
>
>>
>> 于 2020年3月31日 GMT+08:00 下午3:00:06, Tiezhu Yang
><yangtiezhu@loongson.cn> 写到:
>>> When I update the mainline kernel on the Loongson 2-way platform
>which
>>> has 8 CPUs, it only shows 4 CPUs due to NR_CPUS is 4, this is
>obviously
>>> wrong.
>>>
>>> In order to support more CPUs on the Loongson platform, it is better
>>> to use CONFIG_NR_CPUS_DEFAULT_64 instead of CONFIG_NR_CPUS_DEFAULT_4
>>> to specify the maximum number of CPUs which the kernel will support.
>>>
>>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>> Would 16 be a better option?
>> We have only 4 way platform.
>
>There maybe have more than 4 cores per CPU or more than 4 ways
>in the future, so I think 64 is better, otherwise we need to
>modify this configuration once more.

Ok, then

Acked-by: Jiaxun Yang <jiaxun.yang@flygoat.com>

Thanks.

>
>>
>>> ---
>>> arch/mips/Kconfig | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig
>>> index a1f973c..a0b11f5 100644
>>> --- a/arch/mips/Kconfig
>>> +++ b/arch/mips/Kconfig
>>> @@ -475,7 +475,7 @@ config MACH_LOONGSON64
>>> 	select ISA
>>> 	select I8259
>>> 	select IRQ_MIPS_CPU
>>> -	select NR_CPUS_DEFAULT_4
>>> +	select NR_CPUS_DEFAULT_64
>>> 	select USE_GENERIC_EARLY_PRINTK_8250
>>> 	select SYS_HAS_CPU_LOONGSON64
>>> 	select SYS_HAS_EARLY_PRINTK
Thomas Bogendoerfer April 16, 2020, 4:40 p.m. UTC | #4
On Tue, Mar 31, 2020 at 03:00:06PM +0800, Tiezhu Yang wrote:
> When I update the mainline kernel on the Loongson 2-way platform which
> has 8 CPUs, it only shows 4 CPUs due to NR_CPUS is 4, this is obviously
> wrong.
> 
> In order to support more CPUs on the Loongson platform, it is better
> to use CONFIG_NR_CPUS_DEFAULT_64 instead of CONFIG_NR_CPUS_DEFAULT_4
> to specify the maximum number of CPUs which the kernel will support.
> 
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> ---
>  arch/mips/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

applied to mips-next.

Thomas.
diff mbox series

Patch

diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig
index a1f973c..a0b11f5 100644
--- a/arch/mips/Kconfig
+++ b/arch/mips/Kconfig
@@ -475,7 +475,7 @@  config MACH_LOONGSON64
 	select ISA
 	select I8259
 	select IRQ_MIPS_CPU
-	select NR_CPUS_DEFAULT_4
+	select NR_CPUS_DEFAULT_64
 	select USE_GENERIC_EARLY_PRINTK_8250
 	select SYS_HAS_CPU_LOONGSON64
 	select SYS_HAS_EARLY_PRINTK