* [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
@ 2020-03-11 6:33 Tiezhu Yang
2020-03-11 6:53 ` 回复:[PATCH] " Jiaxun Yang
` (2 more replies)
0 siblings, 3 replies; 11+ messages in thread
From: Tiezhu Yang @ 2020-03-11 6:33 UTC (permalink / raw)
To: Thomas Bogendoerfer, Huacai Chen, Jiaxun Yang
Cc: linux-mips, linux-kernel, Xuefeng Li
In the current code, when execute command "cat /proc/cpuinfo" or "lscpu",
it can not get cpu type and frequency directly because the model name is
not exist, so add it.
E.g. without this patch:
[loongson@localhost ~]$ lscpu
Architecture: mips64
Byte Order: Little Endian
CPU(s): 4
On-line CPU(s) list: 0-3
Thread(s) per core: 1
Core(s) per socket: 4
Socket(s): 1
NUMA node(s): 1
L1d cache: 64K
L1i cache: 64K
L2 cache: 2048K
NUMA node0 CPU(s): 0-3
With this patch:
[loongson@localhost ~]$ lscpu
Architecture: mips64
Byte Order: Little Endian
CPU(s): 4
On-line CPU(s) list: 0-3
Thread(s) per core: 1
Core(s) per socket: 4
Socket(s): 1
NUMA node(s): 1
Model name: Loongson-3A R3 (Loongson-3A3000) @ 1449MHz
L1d cache: 64K
L1i cache: 64K
L2 cache: 2048K
NUMA node0 CPU(s): 0-3
Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
arch/mips/include/asm/cpu-info.h | 1 +
arch/mips/kernel/cpu-probe.c | 27 +++++++++++++++++++++++----
arch/mips/kernel/proc.c | 4 ++++
3 files changed, 28 insertions(+), 4 deletions(-)
diff --git a/arch/mips/include/asm/cpu-info.h b/arch/mips/include/asm/cpu-info.h
index ed7ffe4..50e924e 100644
--- a/arch/mips/include/asm/cpu-info.h
+++ b/arch/mips/include/asm/cpu-info.h
@@ -115,6 +115,7 @@ extern struct cpuinfo_mips cpu_data[];
extern void cpu_probe(void);
extern void cpu_report(void);
+extern const char *__model_name[];
extern const char *__cpu_name[];
#define cpu_name_string() __cpu_name[raw_smp_processor_id()]
diff --git a/arch/mips/kernel/cpu-probe.c b/arch/mips/kernel/cpu-probe.c
index 6ab6b03..3ae40cc 100644
--- a/arch/mips/kernel/cpu-probe.c
+++ b/arch/mips/kernel/cpu-probe.c
@@ -1548,6 +1548,7 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
set_elf_platform(cpu, "loongson2e");
set_isa(c, MIPS_CPU_ISA_III);
c->fpu_msk31 |= FPU_CSR_CONDX;
+ __model_name[cpu] = "Loongson-2E";
break;
case PRID_REV_LOONGSON2F:
c->cputype = CPU_LOONGSON2EF;
@@ -1555,23 +1556,37 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
set_elf_platform(cpu, "loongson2f");
set_isa(c, MIPS_CPU_ISA_III);
c->fpu_msk31 |= FPU_CSR_CONDX;
+ __model_name[cpu] = "Loongson-2F";
break;
case PRID_REV_LOONGSON3A_R1:
c->cputype = CPU_LOONGSON64;
__cpu_name[cpu] = "ICT Loongson-3";
set_elf_platform(cpu, "loongson3a");
set_isa(c, MIPS_CPU_ISA_M64R1);
- c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
- MIPS_ASE_LOONGSON_EXT);
+ c->ases |= (MIPS_ASE_LOONGSON_MMI |
+ MIPS_ASE_LOONGSON_CAM |
+ MIPS_ASE_LOONGSON_EXT);
+ __model_name[cpu] = "Loongson-3A R1 (Loongson-3A1000)";
break;
case PRID_REV_LOONGSON3B_R1:
+ c->cputype = CPU_LOONGSON64;
+ __cpu_name[cpu] = "ICT Loongson-3";
+ set_elf_platform(cpu, "loongson3b");
+ set_isa(c, MIPS_CPU_ISA_M64R1);
+ c->ases |= (MIPS_ASE_LOONGSON_MMI |
+ MIPS_ASE_LOONGSON_CAM |
+ MIPS_ASE_LOONGSON_EXT);
+ __model_name[cpu] = "Loongson-3B R1 (Loongson-3B1000)";
+ break;
case PRID_REV_LOONGSON3B_R2:
c->cputype = CPU_LOONGSON64;
__cpu_name[cpu] = "ICT Loongson-3";
set_elf_platform(cpu, "loongson3b");
set_isa(c, MIPS_CPU_ISA_M64R1);
- c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
- MIPS_ASE_LOONGSON_EXT);
+ c->ases |= (MIPS_ASE_LOONGSON_MMI |
+ MIPS_ASE_LOONGSON_CAM |
+ MIPS_ASE_LOONGSON_EXT);
+ __model_name[cpu] = "Loongson-3B R2 (Loongson-3B1500)";
break;
}
@@ -1926,6 +1941,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
__cpu_name[cpu] = "ICT Loongson-3";
set_elf_platform(cpu, "loongson3a");
set_isa(c, MIPS_CPU_ISA_M64R2);
+ __model_name[cpu] = "Loongson-3A R2 (Loongson-3A2000)";
break;
case PRID_REV_LOONGSON3A_R3_0:
case PRID_REV_LOONGSON3A_R3_1:
@@ -1933,6 +1949,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
__cpu_name[cpu] = "ICT Loongson-3";
set_elf_platform(cpu, "loongson3a");
set_isa(c, MIPS_CPU_ISA_M64R2);
+ __model_name[cpu] = "Loongson-3A R3 (Loongson-3A3000)";
break;
}
@@ -1952,6 +1969,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
c->writecombine = _CACHE_UNCACHED_ACCELERATED;
c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
MIPS_ASE_LOONGSON_EXT | MIPS_ASE_LOONGSON_EXT2);
+ __model_name[cpu] = "Loongson-3A R4 (Loongson-3A4000)";
break;
default:
panic("Unknown Loongson Processor ID!");
@@ -2111,6 +2129,7 @@ u64 __ua_limit;
EXPORT_SYMBOL(__ua_limit);
#endif
+const char *__model_name[NR_CPUS];
const char *__cpu_name[NR_CPUS];
const char *__elf_platform;
diff --git a/arch/mips/kernel/proc.c b/arch/mips/kernel/proc.c
index f8d3671..5fc74e6 100644
--- a/arch/mips/kernel/proc.c
+++ b/arch/mips/kernel/proc.c
@@ -15,6 +15,7 @@
#include <asm/mipsregs.h>
#include <asm/processor.h>
#include <asm/prom.h>
+#include <asm/time.h>
unsigned int vced_count, vcei_count;
@@ -63,6 +64,9 @@ static int show_cpuinfo(struct seq_file *m, void *v)
seq_printf(m, fmt, __cpu_name[n],
(version >> 4) & 0x0f, version & 0x0f,
(fp_vers >> 4) & 0x0f, fp_vers & 0x0f);
+ if (__model_name[n])
+ seq_printf(m, "model name\t\t: %s @ %uMHz\n",
+ __model_name[n], mips_hpt_frequency / 500000);
seq_printf(m, "BogoMIPS\t\t: %u.%02u\n",
cpu_data[n].udelay_val / (500000/HZ),
(cpu_data[n].udelay_val / (5000/HZ)) % 100);
--
2.1.0
^ permalink raw reply related [flat|nested] 11+ messages in thread* 回复:[PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
2020-03-11 6:33 [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo Tiezhu Yang
@ 2020-03-11 6:53 ` Jiaxun Yang
2020-03-11 9:28 ` Tiezhu Yang
2020-03-11 7:18 ` [PATCH] " YunQiang Su
2020-03-11 9:58 ` Manuel Lauss
2 siblings, 1 reply; 11+ messages in thread
From: Jiaxun Yang @ 2020-03-11 6:53 UTC (permalink / raw)
To: Tiezhu Yang
Cc: Thomas Bogendoerfer, Huacai Chen, linux-mips, linux-kernel, Xuefeng Li
---- 在 星期三, 2020-03-11 14:33:34 Tiezhu Yang <yangtiezhu@loongson.cn> 撰写 ----
> In the current code, when execute command "cat /proc/cpuinfo" or "lscpu",
> it can not get cpu type and frequency directly because the model name is
> not exist, so add it.
>
> E.g. without this patch:
>
> [loongson@localhost ~]$ lscpu
> Architecture: mips64
> Byte Order: Little Endian
> CPU(s): 4
> On-line CPU(s) list: 0-3
> Thread(s) per core: 1
> Core(s) per socket: 4
> Socket(s): 1
> NUMA node(s): 1
> L1d cache: 64K
> L1i cache: 64K
> L2 cache: 2048K
> NUMA node0 CPU(s): 0-3
>
> With this patch:
>
> [loongson@localhost ~]$ lscpu
> Architecture: mips64
> Byte Order: Little Endian
> CPU(s): 4
> On-line CPU(s) list: 0-3
> Thread(s) per core: 1
> Core(s) per socket: 4
> Socket(s): 1
> NUMA node(s): 1
> Model name: Loongson-3A R3 (Loongson-3A3000) @ 1449MHz
> L1d cache: 64K
> L1i cache: 64K
> L2 cache: 2048K
> NUMA node0 CPU(s): 0-3
Hi Tiezhu,
Thanks for your patch, you're the successor of Huacai:
https://www.linux-mips.org/archives/linux-mips/2018-09/msg00113.html
I think it's worthy to have this string in cpuinfo as many userspace
program is parsing it beacuse x86 have it.
See my review comments below:
>
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> ---
> arch/mips/include/asm/cpu-info.h | 1 +
> arch/mips/kernel/cpu-probe.c | 27 +++++++++++++++++++++++----
> arch/mips/kernel/proc.c | 4 ++++
> 3 files changed, 28 insertions(+), 4 deletions(-)
>
> diff --git a/arch/mips/include/asm/cpu-info.h b/arch/mips/include/asm/cpu-info.h
> index ed7ffe4..50e924e 100644
> --- a/arch/mips/include/asm/cpu-info.h
> +++ b/arch/mips/include/asm/cpu-info.h
> @@ -115,6 +115,7 @@ extern struct cpuinfo_mips cpu_data[];
> extern void cpu_probe(void);
> extern void cpu_report(void);
>
> +extern const char *__model_name[];
> extern const char *__cpu_name[];
> #define cpu_name_string() __cpu_name[raw_smp_processor_id()]
>
> diff --git a/arch/mips/kernel/cpu-probe.c b/arch/mips/kernel/cpu-probe.c
> index 6ab6b03..3ae40cc 100644
> --- a/arch/mips/kernel/cpu-probe.c
> +++ b/arch/mips/kernel/cpu-probe.c
> @@ -1548,6 +1548,7 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
> set_elf_platform(cpu, "loongson2e");
> set_isa(c, MIPS_CPU_ISA_III);
> c->fpu_msk31 |= FPU_CSR_CONDX;
> + __model_name[cpu] = "Loongson-2E";
> break;
> case PRID_REV_LOONGSON2F:
> c->cputype = CPU_LOONGSON2EF;
> @@ -1555,23 +1556,37 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
> set_elf_platform(cpu, "loongson2f");
> set_isa(c, MIPS_CPU_ISA_III);
> c->fpu_msk31 |= FPU_CSR_CONDX;
> + __model_name[cpu] = "Loongson-2F";
> break;
> case PRID_REV_LOONGSON3A_R1:
> c->cputype = CPU_LOONGSON64;
> __cpu_name[cpu] = "ICT Loongson-3";
> set_elf_platform(cpu, "loongson3a");
> set_isa(c, MIPS_CPU_ISA_M64R1);
> - c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> - MIPS_ASE_LOONGSON_EXT);
> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> + MIPS_ASE_LOONGSON_CAM |
> + MIPS_ASE_LOONGSON_EXT);
Is the newline intentional?
Also applied to code below.
> + __model_name[cpu] = "Loongson-3A R1 (Loongson-3A1000)";
> break;
> case PRID_REV_LOONGSON3B_R1:
> + c->cputype = CPU_LOONGSON64;
> + __cpu_name[cpu] = "ICT Loongson-3";
> + set_elf_platform(cpu, "loongson3b");
> + set_isa(c, MIPS_CPU_ISA_M64R1);
> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> + MIPS_ASE_LOONGSON_CAM |
> + MIPS_ASE_LOONGSON_EXT);
> + __model_name[cpu] = "Loongson-3B R1 (Loongson-3B1000)";
> + break;
> case PRID_REV_LOONGSON3B_R2:
> c->cputype = CPU_LOONGSON64;
> __cpu_name[cpu] = "ICT Loongson-3";
> set_elf_platform(cpu, "loongson3b");
> set_isa(c, MIPS_CPU_ISA_M64R1);
> - c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> - MIPS_ASE_LOONGSON_EXT);
> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> + MIPS_ASE_LOONGSON_CAM |
> + MIPS_ASE_LOONGSON_EXT);
> + __model_name[cpu] = "Loongson-3B R2 (Loongson-3B1500)";
> break;
> }
>
> @@ -1926,6 +1941,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> __cpu_name[cpu] = "ICT Loongson-3";
> set_elf_platform(cpu, "loongson3a");
> set_isa(c, MIPS_CPU_ISA_M64R2);
> + __model_name[cpu] = "Loongson-3A R2 (Loongson-3A2000)";
> break;
> case PRID_REV_LOONGSON3A_R3_0:
> case PRID_REV_LOONGSON3A_R3_1:
> @@ -1933,6 +1949,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> __cpu_name[cpu] = "ICT Loongson-3";
> set_elf_platform(cpu, "loongson3a");
> set_isa(c, MIPS_CPU_ISA_M64R2);
> + __model_name[cpu] = "Loongson-3A R3 (Loongson-3A3000)";
> break;
> }
>
> @@ -1952,6 +1969,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> c->writecombine = _CACHE_UNCACHED_ACCELERATED;
> c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> MIPS_ASE_LOONGSON_EXT | MIPS_ASE_LOONGSON_EXT2);
> + __model_name[cpu] = "Loongson-3A R4 (Loongson-3A4000)";
> break;
> default:
> panic("Unknown Loongson Processor ID!");
> @@ -2111,6 +2129,7 @@ u64 __ua_limit;
> EXPORT_SYMBOL(__ua_limit);
> #endif
>
> +const char *__model_name[NR_CPUS];
> const char *__cpu_name[NR_CPUS];
> const char *__elf_platform;
>
> diff --git a/arch/mips/kernel/proc.c b/arch/mips/kernel/proc.c
> index f8d3671..5fc74e6 100644
> --- a/arch/mips/kernel/proc.c
> +++ b/arch/mips/kernel/proc.c
> @@ -15,6 +15,7 @@
> #include <asm/mipsregs.h>
> #include <asm/processor.h>
> #include <asm/prom.h>
> +#include <asm/time.h>
>
> unsigned int vced_count, vcei_count;
>
> @@ -63,6 +64,9 @@ static int show_cpuinfo(struct seq_file *m, void *v)
> seq_printf(m, fmt, __cpu_name[n],
> (version >> 4) & 0x0f, version & 0x0f,
> (fp_vers >> 4) & 0x0f, fp_vers & 0x0f);
> + if (__model_name[n])
> + seq_printf(m, "model name\t\t: %s @ %uMHz\n",
> + __model_name[n], mips_hpt_frequency / 500000);
> seq_printf(m, "BogoMIPS\t\t: %u.%02u\n",
Actually I don't think take mips_hpt_frequency here is a good option. mips_hpt_frequency
is used by cevt-r4k. Processors like Ingenic don't use cevt-r4k don't have it.
Probably you can do as what Huacai did, append the frequency part in platform code.
> cpu_data[n].udelay_val / (500000/HZ),
> (cpu_data[n].udelay_val / (5000/HZ)) % 100);
> --
> 2.1.0
>
>
Thanks.
--
Jiaxun Yang
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: 回复:[PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
2020-03-11 6:53 ` 回复:[PATCH] " Jiaxun Yang
@ 2020-03-11 9:28 ` Tiezhu Yang
0 siblings, 0 replies; 11+ messages in thread
From: Tiezhu Yang @ 2020-03-11 9:28 UTC (permalink / raw)
To: Jiaxun Yang
Cc: Thomas Bogendoerfer, Huacai Chen, linux-mips, linux-kernel, Xuefeng Li
On 03/11/2020 02:53 PM, Jiaxun Yang wrote:
>
> ---- 在 星期三, 2020-03-11 14:33:34 Tiezhu Yang <yangtiezhu@loongson.cn> 撰写 ----
> > In the current code, when execute command "cat /proc/cpuinfo" or "lscpu",
> > it can not get cpu type and frequency directly because the model name is
> > not exist, so add it.
> >
> > E.g. without this patch:
> >
> > [loongson@localhost ~]$ lscpu
> > Architecture: mips64
> > Byte Order: Little Endian
> > CPU(s): 4
> > On-line CPU(s) list: 0-3
> > Thread(s) per core: 1
> > Core(s) per socket: 4
> > Socket(s): 1
> > NUMA node(s): 1
> > L1d cache: 64K
> > L1i cache: 64K
> > L2 cache: 2048K
> > NUMA node0 CPU(s): 0-3
> >
> > With this patch:
> >
> > [loongson@localhost ~]$ lscpu
> > Architecture: mips64
> > Byte Order: Little Endian
> > CPU(s): 4
> > On-line CPU(s) list: 0-3
> > Thread(s) per core: 1
> > Core(s) per socket: 4
> > Socket(s): 1
> > NUMA node(s): 1
> > Model name: Loongson-3A R3 (Loongson-3A3000) @ 1449MHz
> > L1d cache: 64K
> > L1i cache: 64K
> > L2 cache: 2048K
> > NUMA node0 CPU(s): 0-3
>
> Hi Tiezhu,
>
> Thanks for your patch, you're the successor of Huacai:
>
> https://www.linux-mips.org/archives/linux-mips/2018-09/msg00113.html
Hi Jiaxun,
Thanks for your notice.
>
> I think it's worthy to have this string in cpuinfo as many userspace
> program is parsing it beacuse x86 have it.
>
> See my review comments below:
>
> >
> > Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> > ---
> > arch/mips/include/asm/cpu-info.h | 1 +
> > arch/mips/kernel/cpu-probe.c | 27 +++++++++++++++++++++++----
> > arch/mips/kernel/proc.c | 4 ++++
> > 3 files changed, 28 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/mips/include/asm/cpu-info.h b/arch/mips/include/asm/cpu-info.h
> > index ed7ffe4..50e924e 100644
> > --- a/arch/mips/include/asm/cpu-info.h
> > +++ b/arch/mips/include/asm/cpu-info.h
> > @@ -115,6 +115,7 @@ extern struct cpuinfo_mips cpu_data[];
> > extern void cpu_probe(void);
> > extern void cpu_report(void);
> >
> > +extern const char *__model_name[];
> > extern const char *__cpu_name[];
> > #define cpu_name_string() __cpu_name[raw_smp_processor_id()]
> >
> > diff --git a/arch/mips/kernel/cpu-probe.c b/arch/mips/kernel/cpu-probe.c
> > index 6ab6b03..3ae40cc 100644
> > --- a/arch/mips/kernel/cpu-probe.c
> > +++ b/arch/mips/kernel/cpu-probe.c
> > @@ -1548,6 +1548,7 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
> > set_elf_platform(cpu, "loongson2e");
> > set_isa(c, MIPS_CPU_ISA_III);
> > c->fpu_msk31 |= FPU_CSR_CONDX;
> > + __model_name[cpu] = "Loongson-2E";
> > break;
> > case PRID_REV_LOONGSON2F:
> > c->cputype = CPU_LOONGSON2EF;
> > @@ -1555,23 +1556,37 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
> > set_elf_platform(cpu, "loongson2f");
> > set_isa(c, MIPS_CPU_ISA_III);
> > c->fpu_msk31 |= FPU_CSR_CONDX;
> > + __model_name[cpu] = "Loongson-2F";
> > break;
> > case PRID_REV_LOONGSON3A_R1:
> > c->cputype = CPU_LOONGSON64;
> > __cpu_name[cpu] = "ICT Loongson-3";
> > set_elf_platform(cpu, "loongson3a");
> > set_isa(c, MIPS_CPU_ISA_M64R1);
> > - c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> > - MIPS_ASE_LOONGSON_EXT);
> > + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> > + MIPS_ASE_LOONGSON_CAM |
> > + MIPS_ASE_LOONGSON_EXT);
>
> Is the newline intentional?
> Also applied to code below.
Yes, just fix the checkpatch.pl warning:
WARNING: line over 80 characters
#78: FILE: arch/mips/kernel/cpu-probe.c:1575:
+ c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
>
> > + __model_name[cpu] = "Loongson-3A R1 (Loongson-3A1000)";
> > break;
> > case PRID_REV_LOONGSON3B_R1:
> > + c->cputype = CPU_LOONGSON64;
> > + __cpu_name[cpu] = "ICT Loongson-3";
> > + set_elf_platform(cpu, "loongson3b");
> > + set_isa(c, MIPS_CPU_ISA_M64R1);
> > + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> > + MIPS_ASE_LOONGSON_CAM |
> > + MIPS_ASE_LOONGSON_EXT);
> > + __model_name[cpu] = "Loongson-3B R1 (Loongson-3B1000)";
> > + break;
> > case PRID_REV_LOONGSON3B_R2:
> > c->cputype = CPU_LOONGSON64;
> > __cpu_name[cpu] = "ICT Loongson-3";
> > set_elf_platform(cpu, "loongson3b");
> > set_isa(c, MIPS_CPU_ISA_M64R1);
> > - c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> > - MIPS_ASE_LOONGSON_EXT);
> > + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> > + MIPS_ASE_LOONGSON_CAM |
> > + MIPS_ASE_LOONGSON_EXT);
> > + __model_name[cpu] = "Loongson-3B R2 (Loongson-3B1500)";
> > break;
> > }
> >
> > @@ -1926,6 +1941,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> > __cpu_name[cpu] = "ICT Loongson-3";
> > set_elf_platform(cpu, "loongson3a");
> > set_isa(c, MIPS_CPU_ISA_M64R2);
> > + __model_name[cpu] = "Loongson-3A R2 (Loongson-3A2000)";
> > break;
> > case PRID_REV_LOONGSON3A_R3_0:
> > case PRID_REV_LOONGSON3A_R3_1:
> > @@ -1933,6 +1949,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> > __cpu_name[cpu] = "ICT Loongson-3";
> > set_elf_platform(cpu, "loongson3a");
> > set_isa(c, MIPS_CPU_ISA_M64R2);
> > + __model_name[cpu] = "Loongson-3A R3 (Loongson-3A3000)";
> > break;
> > }
> >
> > @@ -1952,6 +1969,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> > c->writecombine = _CACHE_UNCACHED_ACCELERATED;
> > c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> > MIPS_ASE_LOONGSON_EXT | MIPS_ASE_LOONGSON_EXT2);
> > + __model_name[cpu] = "Loongson-3A R4 (Loongson-3A4000)";
> > break;
> > default:
> > panic("Unknown Loongson Processor ID!");
> > @@ -2111,6 +2129,7 @@ u64 __ua_limit;
> > EXPORT_SYMBOL(__ua_limit);
> > #endif
> >
> > +const char *__model_name[NR_CPUS];
> > const char *__cpu_name[NR_CPUS];
> > const char *__elf_platform;
> >
> > diff --git a/arch/mips/kernel/proc.c b/arch/mips/kernel/proc.c
> > index f8d3671..5fc74e6 100644
> > --- a/arch/mips/kernel/proc.c
> > +++ b/arch/mips/kernel/proc.c
> > @@ -15,6 +15,7 @@
> > #include <asm/mipsregs.h>
> > #include <asm/processor.h>
> > #include <asm/prom.h>
> > +#include <asm/time.h>
> >
> > unsigned int vced_count, vcei_count;
> >
> > @@ -63,6 +64,9 @@ static int show_cpuinfo(struct seq_file *m, void *v)
> > seq_printf(m, fmt, __cpu_name[n],
> > (version >> 4) & 0x0f, version & 0x0f,
> > (fp_vers >> 4) & 0x0f, fp_vers & 0x0f);
> > + if (__model_name[n])
> > + seq_printf(m, "model name\t\t: %s @ %uMHz\n",
> > + __model_name[n], mips_hpt_frequency / 500000);
> > seq_printf(m, "BogoMIPS\t\t: %u.%02u\n",
>
> Actually I don't think take mips_hpt_frequency here is a good option. mips_hpt_frequency
> is used by cevt-r4k. Processors like Ingenic don't use cevt-r4k don't have it.
>
> Probably you can do as what Huacai did, append the frequency part in platform code.
OK, I will do it.
Thanks,
Tiezhu Yang
>
>> cpu_data[n].udelay_val / (500000/HZ),
> > (cpu_data[n].udelay_val / (5000/HZ)) % 100);
> > --
> > 2.1.0
> >
> >
>
> Thanks.
>
> --
> Jiaxun Yang
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
2020-03-11 6:33 [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo Tiezhu Yang
2020-03-11 6:53 ` 回复:[PATCH] " Jiaxun Yang
@ 2020-03-11 7:18 ` YunQiang Su
2020-03-11 7:41 ` Jiaxun Yang
2020-03-11 9:40 ` Tiezhu Yang
2020-03-11 9:58 ` Manuel Lauss
2 siblings, 2 replies; 11+ messages in thread
From: YunQiang Su @ 2020-03-11 7:18 UTC (permalink / raw)
To: Tiezhu Yang
Cc: Thomas Bogendoerfer, Huacai Chen, Jiaxun Yang, linux-mips,
linux-kernel, Xuefeng Li
Tiezhu Yang <yangtiezhu@loongson.cn> 于2020年3月11日周三 下午2:33写道：
>
> In the current code, when execute command "cat /proc/cpuinfo" or "lscpu",
> it can not get cpu type and frequency directly because the model name is
> not exist, so add it.
You patch is almost the same with the previous Huacai's.
How do you think about the copyright issue?
FYI: GPL is copyleft instead of no-copyright-exists-at-all.
I have no idea about whether Loongson has any consideration about copyright,
while it is really quite important.
>
> E.g. without this patch:
>
> [loongson@localhost ~]$ lscpu
> Architecture: mips64
> Byte Order: Little Endian
> CPU(s): 4
> On-line CPU(s) list: 0-3
> Thread(s) per core: 1
> Core(s) per socket: 4
> Socket(s): 1
> NUMA node(s): 1
> L1d cache: 64K
> L1i cache: 64K
> L2 cache: 2048K
> NUMA node0 CPU(s): 0-3
>
> With this patch:
>
> [loongson@localhost ~]$ lscpu
> Architecture: mips64
> Byte Order: Little Endian
> CPU(s): 4
> On-line CPU(s) list: 0-3
> Thread(s) per core: 1
> Core(s) per socket: 4
> Socket(s): 1
> NUMA node(s): 1
> Model name: Loongson-3A R3 (Loongson-3A3000) @ 1449MHz
> L1d cache: 64K
> L1i cache: 64K
> L2 cache: 2048K
> NUMA node0 CPU(s): 0-3
>
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> ---
> arch/mips/include/asm/cpu-info.h | 1 +
> arch/mips/kernel/cpu-probe.c | 27 +++++++++++++++++++++++----
> arch/mips/kernel/proc.c | 4 ++++
> 3 files changed, 28 insertions(+), 4 deletions(-)
>
> diff --git a/arch/mips/include/asm/cpu-info.h b/arch/mips/include/asm/cpu-info.h
> index ed7ffe4..50e924e 100644
> --- a/arch/mips/include/asm/cpu-info.h
> +++ b/arch/mips/include/asm/cpu-info.h
> @@ -115,6 +115,7 @@ extern struct cpuinfo_mips cpu_data[];
> extern void cpu_probe(void);
> extern void cpu_report(void);
>
> +extern const char *__model_name[];
> extern const char *__cpu_name[];
> #define cpu_name_string() __cpu_name[raw_smp_processor_id()]
>
> diff --git a/arch/mips/kernel/cpu-probe.c b/arch/mips/kernel/cpu-probe.c
> index 6ab6b03..3ae40cc 100644
> --- a/arch/mips/kernel/cpu-probe.c
> +++ b/arch/mips/kernel/cpu-probe.c
> @@ -1548,6 +1548,7 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
> set_elf_platform(cpu, "loongson2e");
> set_isa(c, MIPS_CPU_ISA_III);
> c->fpu_msk31 |= FPU_CSR_CONDX;
> + __model_name[cpu] = "Loongson-2E";
> break;
> case PRID_REV_LOONGSON2F:
> c->cputype = CPU_LOONGSON2EF;
> @@ -1555,23 +1556,37 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
> set_elf_platform(cpu, "loongson2f");
> set_isa(c, MIPS_CPU_ISA_III);
> c->fpu_msk31 |= FPU_CSR_CONDX;
> + __model_name[cpu] = "Loongson-2F";
> break;
> case PRID_REV_LOONGSON3A_R1:
> c->cputype = CPU_LOONGSON64;
> __cpu_name[cpu] = "ICT Loongson-3";
> set_elf_platform(cpu, "loongson3a");
> set_isa(c, MIPS_CPU_ISA_M64R1);
> - c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> - MIPS_ASE_LOONGSON_EXT);
> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> + MIPS_ASE_LOONGSON_CAM |
> + MIPS_ASE_LOONGSON_EXT);
> + __model_name[cpu] = "Loongson-3A R1 (Loongson-3A1000)";
> break;
> case PRID_REV_LOONGSON3B_R1:
> + c->cputype = CPU_LOONGSON64;
> + __cpu_name[cpu] = "ICT Loongson-3";
> + set_elf_platform(cpu, "loongson3b");
> + set_isa(c, MIPS_CPU_ISA_M64R1);
> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> + MIPS_ASE_LOONGSON_CAM |
> + MIPS_ASE_LOONGSON_EXT);
> + __model_name[cpu] = "Loongson-3B R1 (Loongson-3B1000)";
> + break;
> case PRID_REV_LOONGSON3B_R2:
> c->cputype = CPU_LOONGSON64;
> __cpu_name[cpu] = "ICT Loongson-3";
> set_elf_platform(cpu, "loongson3b");
> set_isa(c, MIPS_CPU_ISA_M64R1);
> - c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> - MIPS_ASE_LOONGSON_EXT);
> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> + MIPS_ASE_LOONGSON_CAM |
> + MIPS_ASE_LOONGSON_EXT);
> + __model_name[cpu] = "Loongson-3B R2 (Loongson-3B1500)";
> break;
> }
>
> @@ -1926,6 +1941,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> __cpu_name[cpu] = "ICT Loongson-3";
> set_elf_platform(cpu, "loongson3a");
> set_isa(c, MIPS_CPU_ISA_M64R2);
> + __model_name[cpu] = "Loongson-3A R2 (Loongson-3A2000)";
> break;
> case PRID_REV_LOONGSON3A_R3_0:
> case PRID_REV_LOONGSON3A_R3_1:
> @@ -1933,6 +1949,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> __cpu_name[cpu] = "ICT Loongson-3";
> set_elf_platform(cpu, "loongson3a");
> set_isa(c, MIPS_CPU_ISA_M64R2);
> + __model_name[cpu] = "Loongson-3A R3 (Loongson-3A3000)";
> break;
> }
>
> @@ -1952,6 +1969,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> c->writecombine = _CACHE_UNCACHED_ACCELERATED;
> c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> MIPS_ASE_LOONGSON_EXT | MIPS_ASE_LOONGSON_EXT2);
> + __model_name[cpu] = "Loongson-3A R4 (Loongson-3A4000)";
> break;
> default:
> panic("Unknown Loongson Processor ID!");
> @@ -2111,6 +2129,7 @@ u64 __ua_limit;
> EXPORT_SYMBOL(__ua_limit);
> #endif
>
> +const char *__model_name[NR_CPUS];
> const char *__cpu_name[NR_CPUS];
> const char *__elf_platform;
>
> diff --git a/arch/mips/kernel/proc.c b/arch/mips/kernel/proc.c
> index f8d3671..5fc74e6 100644
> --- a/arch/mips/kernel/proc.c
> +++ b/arch/mips/kernel/proc.c
> @@ -15,6 +15,7 @@
> #include <asm/mipsregs.h>
> #include <asm/processor.h>
> #include <asm/prom.h>
> +#include <asm/time.h>
>
> unsigned int vced_count, vcei_count;
>
> @@ -63,6 +64,9 @@ static int show_cpuinfo(struct seq_file *m, void *v)
> seq_printf(m, fmt, __cpu_name[n],
> (version >> 4) & 0x0f, version & 0x0f,
> (fp_vers >> 4) & 0x0f, fp_vers & 0x0f);
> + if (__model_name[n])
> + seq_printf(m, "model name\t\t: %s @ %uMHz\n",
> + __model_name[n], mips_hpt_frequency / 500000);
> seq_printf(m, "BogoMIPS\t\t: %u.%02u\n",
> cpu_data[n].udelay_val / (500000/HZ),
> (cpu_data[n].udelay_val / (5000/HZ)) % 100);
> --
> 2.1.0
>
--
YunQiang Su
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
2020-03-11 7:18 ` [PATCH] " YunQiang Su
@ 2020-03-11 7:41 ` Jiaxun Yang
2020-03-11 9:40 ` Tiezhu Yang
1 sibling, 0 replies; 11+ messages in thread
From: Jiaxun Yang @ 2020-03-11 7:41 UTC (permalink / raw)
To: YunQiang Su
Cc: Tiezhu Yang, Thomas Bogendoerfer, Huacai Chen, linux-mips,
linux-kernel, Xuefeng Li
---- 在 星期三, 2020-03-11 15:18:59 YunQiang Su <wzssyqa@gmail.com> 撰写 ----
> Tiezhu Yang <yangtiezhu@loongson.cn> 于2020年3月11日周三 下午2:33写道：
> >
> > In the current code, when execute command "cat /proc/cpuinfo" or "lscpu",
> > it can not get cpu type and frequency directly because the model name is
> > not exist, so add it.
>
> You patch is almost the same with the previous Huacai's.
> How do you think about the copyright issue?
>
> FYI: GPL is copyleft instead of no-copyright-exists-at-all.
> I have no idea about whether Loongson has any consideration about copyright,
> while it is really quite important.
That's my concern as well. Thing happend at rust [1] made us doubting Loongson's
attitude on copyright & licenses related stuff. And the fact of Loongson's longterm
licenses violation on GCC(GPL), Firefox(MPL) and other projects is not only
harmful but also dangerous to the FOSS community.
We're welling to see the Loongson which loved by RMS come back.
[1]: https://github.com/rust-lang/rust/pull/69508
>
> >
> > E.g. without this patch:
> >
> > [loongson@localhost ~]$ lscpu
> > Architecture: mips64
> > Byte Order: Little Endian
> > CPU(s): 4
> > On-line CPU(s) list: 0-3
> > Thread(s) per core: 1
> > Core(s) per socket: 4
> > Socket(s): 1
> > NUMA node(s): 1
> > L1d cache: 64K
> > L1i cache: 64K
> > L2 cache: 2048K
> > NUMA node0 CPU(s): 0-3
> >
> > With this patch:
> >
> > [loongson@localhost ~]$ lscpu
> > Architecture: mips64
> > Byte Order: Little Endian
> > CPU(s): 4
> > On-line CPU(s) list: 0-3
> > Thread(s) per core: 1
> > Core(s) per socket: 4
> > Socket(s): 1
> > NUMA node(s): 1
> > Model name: Loongson-3A R3 (Loongson-3A3000) @ 1449MHz
> > L1d cache: 64K
> > L1i cache: 64K
> > L2 cache: 2048K
> > NUMA node0 CPU(s): 0-3
> >
> > Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> > ---
> > arch/mips/include/asm/cpu-info.h | 1 +
> > arch/mips/kernel/cpu-probe.c | 27 +++++++++++++++++++++++----
> > arch/mips/kernel/proc.c | 4 ++++
> > 3 files changed, 28 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/mips/include/asm/cpu-info.h b/arch/mips/include/asm/cpu-info.h
> > index ed7ffe4..50e924e 100644
> > --- a/arch/mips/include/asm/cpu-info.h
> > +++ b/arch/mips/include/asm/cpu-info.h
> > @@ -115,6 +115,7 @@ extern struct cpuinfo_mips cpu_data[];
> > extern void cpu_probe(void);
> > extern void cpu_report(void);
> >
> > +extern const char *__model_name[];
> > extern const char *__cpu_name[];
> > #define cpu_name_string() __cpu_name[raw_smp_processor_id()]
> >
> > diff --git a/arch/mips/kernel/cpu-probe.c b/arch/mips/kernel/cpu-probe.c
> > index 6ab6b03..3ae40cc 100644
> > --- a/arch/mips/kernel/cpu-probe.c
> > +++ b/arch/mips/kernel/cpu-probe.c
> > @@ -1548,6 +1548,7 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
> > set_elf_platform(cpu, "loongson2e");
> > set_isa(c, MIPS_CPU_ISA_III);
> > c->fpu_msk31 |= FPU_CSR_CONDX;
> > + __model_name[cpu] = "Loongson-2E";
> > break;
> > case PRID_REV_LOONGSON2F:
> > c->cputype = CPU_LOONGSON2EF;
> > @@ -1555,23 +1556,37 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
> > set_elf_platform(cpu, "loongson2f");
> > set_isa(c, MIPS_CPU_ISA_III);
> > c->fpu_msk31 |= FPU_CSR_CONDX;
> > + __model_name[cpu] = "Loongson-2F";
> > break;
> > case PRID_REV_LOONGSON3A_R1:
> > c->cputype = CPU_LOONGSON64;
> > __cpu_name[cpu] = "ICT Loongson-3";
> > set_elf_platform(cpu, "loongson3a");
> > set_isa(c, MIPS_CPU_ISA_M64R1);
> > - c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> > - MIPS_ASE_LOONGSON_EXT);
> > + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> > + MIPS_ASE_LOONGSON_CAM |
> > + MIPS_ASE_LOONGSON_EXT);
> > + __model_name[cpu] = "Loongson-3A R1 (Loongson-3A1000)";
> > break;
> > case PRID_REV_LOONGSON3B_R1:
> > + c->cputype = CPU_LOONGSON64;
> > + __cpu_name[cpu] = "ICT Loongson-3";
> > + set_elf_platform(cpu, "loongson3b");
> > + set_isa(c, MIPS_CPU_ISA_M64R1);
> > + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> > + MIPS_ASE_LOONGSON_CAM |
> > + MIPS_ASE_LOONGSON_EXT);
> > + __model_name[cpu] = "Loongson-3B R1 (Loongson-3B1000)";
> > + break;
> > case PRID_REV_LOONGSON3B_R2:
> > c->cputype = CPU_LOONGSON64;
> > __cpu_name[cpu] = "ICT Loongson-3";
> > set_elf_platform(cpu, "loongson3b");
> > set_isa(c, MIPS_CPU_ISA_M64R1);
> > - c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> > - MIPS_ASE_LOONGSON_EXT);
> > + c->ases |= (MIPS_ASE_LOONGSON_MMI |
> > + MIPS_ASE_LOONGSON_CAM |
> > + MIPS_ASE_LOONGSON_EXT);
> > + __model_name[cpu] = "Loongson-3B R2 (Loongson-3B1500)";
> > break;
> > }
> >
> > @@ -1926,6 +1941,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> > __cpu_name[cpu] = "ICT Loongson-3";
> > set_elf_platform(cpu, "loongson3a");
> > set_isa(c, MIPS_CPU_ISA_M64R2);
> > + __model_name[cpu] = "Loongson-3A R2 (Loongson-3A2000)";
> > break;
> > case PRID_REV_LOONGSON3A_R3_0:
> > case PRID_REV_LOONGSON3A_R3_1:
> > @@ -1933,6 +1949,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> > __cpu_name[cpu] = "ICT Loongson-3";
> > set_elf_platform(cpu, "loongson3a");
> > set_isa(c, MIPS_CPU_ISA_M64R2);
> > + __model_name[cpu] = "Loongson-3A R3 (Loongson-3A3000)";
> > break;
> > }
> >
> > @@ -1952,6 +1969,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
> > c->writecombine = _CACHE_UNCACHED_ACCELERATED;
> > c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
> > MIPS_ASE_LOONGSON_EXT | MIPS_ASE_LOONGSON_EXT2);
> > + __model_name[cpu] = "Loongson-3A R4 (Loongson-3A4000)";
> > break;
> > default:
> > panic("Unknown Loongson Processor ID!");
> > @@ -2111,6 +2129,7 @@ u64 __ua_limit;
> > EXPORT_SYMBOL(__ua_limit);
> > #endif
> >
> > +const char *__model_name[NR_CPUS];
> > const char *__cpu_name[NR_CPUS];
> > const char *__elf_platform;
> >
> > diff --git a/arch/mips/kernel/proc.c b/arch/mips/kernel/proc.c
> > index f8d3671..5fc74e6 100644
> > --- a/arch/mips/kernel/proc.c
> > +++ b/arch/mips/kernel/proc.c
> > @@ -15,6 +15,7 @@
> > #include <asm/mipsregs.h>
> > #include <asm/processor.h>
> > #include <asm/prom.h>
> > +#include <asm/time.h>
> >
> > unsigned int vced_count, vcei_count;
> >
> > @@ -63,6 +64,9 @@ static int show_cpuinfo(struct seq_file *m, void *v)
> > seq_printf(m, fmt, __cpu_name[n],
> > (version >> 4) & 0x0f, version & 0x0f,
> > (fp_vers >> 4) & 0x0f, fp_vers & 0x0f);
> > + if (__model_name[n])
> > + seq_printf(m, "model name\t\t: %s @ %uMHz\n",
> > + __model_name[n], mips_hpt_frequency / 500000);
> > seq_printf(m, "BogoMIPS\t\t: %u.%02u\n",
> > cpu_data[n].udelay_val / (500000/HZ),
> > (cpu_data[n].udelay_val / (5000/HZ)) % 100);
> > --
> > 2.1.0
> >
>
>
> --
> YunQiang Su
>
--
Jiaxun Yang
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
2020-03-11 7:18 ` [PATCH] " YunQiang Su
2020-03-11 7:41 ` Jiaxun Yang
@ 2020-03-11 9:40 ` Tiezhu Yang
2020-03-12 7:25 ` Tiezhu Yang
1 sibling, 1 reply; 11+ messages in thread
From: Tiezhu Yang @ 2020-03-11 9:40 UTC (permalink / raw)
To: YunQiang Su
Cc: Thomas Bogendoerfer, Huacai Chen, Jiaxun Yang, linux-mips,
linux-kernel, Xuefeng Li
On 03/11/2020 03:18 PM, YunQiang Su wrote:
> Tiezhu Yang <yangtiezhu@loongson.cn> 于2020年3月11日周三 下午2:33写道：
>> In the current code, when execute command "cat /proc/cpuinfo" or "lscpu",
>> it can not get cpu type and frequency directly because the model name is
>> not exist, so add it.
> You patch is almost the same with the previous Huacai's.
> How do you think about the copyright issue?
>
> FYI: GPL is copyleft instead of no-copyright-exists-at-all.
> I have no idea about whether Loongson has any consideration about copyright,
> while it is really quite important.
Hi YunQiang & Jiaxun,
Frankly, I don't know there exists a similar patch in 2018
which has not been merged into the mainline tree, I just
want to fix the issue of model name when execute cmd
"cat /proc/cpuinfo" or "lscpu".
How to avoid and solve this case?
Add a proper label in the patch?
Thanks,
Tiezhu Yang
>
>> E.g. without this patch:
>>
>> [loongson@localhost ~]$ lscpu
>> Architecture: mips64
>> Byte Order: Little Endian
>> CPU(s): 4
>> On-line CPU(s) list: 0-3
>> Thread(s) per core: 1
>> Core(s) per socket: 4
>> Socket(s): 1
>> NUMA node(s): 1
>> L1d cache: 64K
>> L1i cache: 64K
>> L2 cache: 2048K
>> NUMA node0 CPU(s): 0-3
>>
>> With this patch:
>>
>> [loongson@localhost ~]$ lscpu
>> Architecture: mips64
>> Byte Order: Little Endian
>> CPU(s): 4
>> On-line CPU(s) list: 0-3
>> Thread(s) per core: 1
>> Core(s) per socket: 4
>> Socket(s): 1
>> NUMA node(s): 1
>> Model name: Loongson-3A R3 (Loongson-3A3000) @ 1449MHz
>> L1d cache: 64K
>> L1i cache: 64K
>> L2 cache: 2048K
>> NUMA node0 CPU(s): 0-3
>>
>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>> ---
>> arch/mips/include/asm/cpu-info.h | 1 +
>> arch/mips/kernel/cpu-probe.c | 27 +++++++++++++++++++++++----
>> arch/mips/kernel/proc.c | 4 ++++
>> 3 files changed, 28 insertions(+), 4 deletions(-)
>>
>> diff --git a/arch/mips/include/asm/cpu-info.h b/arch/mips/include/asm/cpu-info.h
>> index ed7ffe4..50e924e 100644
>> --- a/arch/mips/include/asm/cpu-info.h
>> +++ b/arch/mips/include/asm/cpu-info.h
>> @@ -115,6 +115,7 @@ extern struct cpuinfo_mips cpu_data[];
>> extern void cpu_probe(void);
>> extern void cpu_report(void);
>>
>> +extern const char *__model_name[];
>> extern const char *__cpu_name[];
>> #define cpu_name_string() __cpu_name[raw_smp_processor_id()]
>>
>> diff --git a/arch/mips/kernel/cpu-probe.c b/arch/mips/kernel/cpu-probe.c
>> index 6ab6b03..3ae40cc 100644
>> --- a/arch/mips/kernel/cpu-probe.c
>> +++ b/arch/mips/kernel/cpu-probe.c
>> @@ -1548,6 +1548,7 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
>> set_elf_platform(cpu, "loongson2e");
>> set_isa(c, MIPS_CPU_ISA_III);
>> c->fpu_msk31 |= FPU_CSR_CONDX;
>> + __model_name[cpu] = "Loongson-2E";
>> break;
>> case PRID_REV_LOONGSON2F:
>> c->cputype = CPU_LOONGSON2EF;
>> @@ -1555,23 +1556,37 @@ static inline void cpu_probe_legacy(struct cpuinfo_mips *c, unsigned int cpu)
>> set_elf_platform(cpu, "loongson2f");
>> set_isa(c, MIPS_CPU_ISA_III);
>> c->fpu_msk31 |= FPU_CSR_CONDX;
>> + __model_name[cpu] = "Loongson-2F";
>> break;
>> case PRID_REV_LOONGSON3A_R1:
>> c->cputype = CPU_LOONGSON64;
>> __cpu_name[cpu] = "ICT Loongson-3";
>> set_elf_platform(cpu, "loongson3a");
>> set_isa(c, MIPS_CPU_ISA_M64R1);
>> - c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
>> - MIPS_ASE_LOONGSON_EXT);
>> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
>> + MIPS_ASE_LOONGSON_CAM |
>> + MIPS_ASE_LOONGSON_EXT);
>> + __model_name[cpu] = "Loongson-3A R1 (Loongson-3A1000)";
>> break;
>> case PRID_REV_LOONGSON3B_R1:
>> + c->cputype = CPU_LOONGSON64;
>> + __cpu_name[cpu] = "ICT Loongson-3";
>> + set_elf_platform(cpu, "loongson3b");
>> + set_isa(c, MIPS_CPU_ISA_M64R1);
>> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
>> + MIPS_ASE_LOONGSON_CAM |
>> + MIPS_ASE_LOONGSON_EXT);
>> + __model_name[cpu] = "Loongson-3B R1 (Loongson-3B1000)";
>> + break;
>> case PRID_REV_LOONGSON3B_R2:
>> c->cputype = CPU_LOONGSON64;
>> __cpu_name[cpu] = "ICT Loongson-3";
>> set_elf_platform(cpu, "loongson3b");
>> set_isa(c, MIPS_CPU_ISA_M64R1);
>> - c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
>> - MIPS_ASE_LOONGSON_EXT);
>> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
>> + MIPS_ASE_LOONGSON_CAM |
>> + MIPS_ASE_LOONGSON_EXT);
>> + __model_name[cpu] = "Loongson-3B R2 (Loongson-3B1500)";
>> break;
>> }
>>
>> @@ -1926,6 +1941,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
>> __cpu_name[cpu] = "ICT Loongson-3";
>> set_elf_platform(cpu, "loongson3a");
>> set_isa(c, MIPS_CPU_ISA_M64R2);
>> + __model_name[cpu] = "Loongson-3A R2 (Loongson-3A2000)";
>> break;
>> case PRID_REV_LOONGSON3A_R3_0:
>> case PRID_REV_LOONGSON3A_R3_1:
>> @@ -1933,6 +1949,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
>> __cpu_name[cpu] = "ICT Loongson-3";
>> set_elf_platform(cpu, "loongson3a");
>> set_isa(c, MIPS_CPU_ISA_M64R2);
>> + __model_name[cpu] = "Loongson-3A R3 (Loongson-3A3000)";
>> break;
>> }
>>
>> @@ -1952,6 +1969,7 @@ static inline void cpu_probe_loongson(struct cpuinfo_mips *c, unsigned int cpu)
>> c->writecombine = _CACHE_UNCACHED_ACCELERATED;
>> c->ases |= (MIPS_ASE_LOONGSON_MMI | MIPS_ASE_LOONGSON_CAM |
>> MIPS_ASE_LOONGSON_EXT | MIPS_ASE_LOONGSON_EXT2);
>> + __model_name[cpu] = "Loongson-3A R4 (Loongson-3A4000)";
>> break;
>> default:
>> panic("Unknown Loongson Processor ID!");
>> @@ -2111,6 +2129,7 @@ u64 __ua_limit;
>> EXPORT_SYMBOL(__ua_limit);
>> #endif
>>
>> +const char *__model_name[NR_CPUS];
>> const char *__cpu_name[NR_CPUS];
>> const char *__elf_platform;
>>
>> diff --git a/arch/mips/kernel/proc.c b/arch/mips/kernel/proc.c
>> index f8d3671..5fc74e6 100644
>> --- a/arch/mips/kernel/proc.c
>> +++ b/arch/mips/kernel/proc.c
>> @@ -15,6 +15,7 @@
>> #include <asm/mipsregs.h>
>> #include <asm/processor.h>
>> #include <asm/prom.h>
>> +#include <asm/time.h>
>>
>> unsigned int vced_count, vcei_count;
>>
>> @@ -63,6 +64,9 @@ static int show_cpuinfo(struct seq_file *m, void *v)
>> seq_printf(m, fmt, __cpu_name[n],
>> (version >> 4) & 0x0f, version & 0x0f,
>> (fp_vers >> 4) & 0x0f, fp_vers & 0x0f);
>> + if (__model_name[n])
>> + seq_printf(m, "model name\t\t: %s @ %uMHz\n",
>> + __model_name[n], mips_hpt_frequency / 500000);
>> seq_printf(m, "BogoMIPS\t\t: %u.%02u\n",
>> cpu_data[n].udelay_val / (500000/HZ),
>> (cpu_data[n].udelay_val / (5000/HZ)) % 100);
>> --
>> 2.1.0
>>
>
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
2020-03-11 9:40 ` Tiezhu Yang
@ 2020-03-12 7:25 ` Tiezhu Yang
0 siblings, 0 replies; 11+ messages in thread
From: Tiezhu Yang @ 2020-03-12 7:25 UTC (permalink / raw)
To: YunQiang Su, Huacai Chen, Jiaxun Yang
Cc: Thomas Bogendoerfer, linux-mips, linux-kernel, Xuefeng Li
On 03/11/2020 05:40 PM, Tiezhu Yang wrote:
> On 03/11/2020 03:18 PM, YunQiang Su wrote:
>> Tiezhu Yang <yangtiezhu@loongson.cn> 于2020年3月11日周三 下午2:33写道：
>>> In the current code, when execute command "cat /proc/cpuinfo" or
>>> "lscpu",
>>> it can not get cpu type and frequency directly because the model
>>> name is
>>> not exist, so add it.
>> You patch is almost the same with the previous Huacai's.
>> How do you think about the copyright issue?
>>
>> FYI: GPL is copyleft instead of no-copyright-exists-at-all.
>> I have no idea about whether Loongson has any consideration about
>> copyright,
>> while it is really quite important.
>
> Hi YunQiang & Jiaxun,
>
> Frankly, I don't know there exists a similar patch in 2018
> which has not been merged into the mainline tree, I just
> want to fix the issue of model name when execute cmd
> "cat /proc/cpuinfo" or "lscpu".
>
> How to avoid and solve this case?
> Add a proper label in the patch?
Hi YunQiang, Jiaxun & Huacai,
Through discussion, I think it is better to resend the old patch by Huacai.
I will not send a v2 patch.
Thanks,
Tiezhu Yang
>
> Thanks,
>
> Tiezhu Yang
>
>>
>>> E.g. without this patch:
>>>
>>> [loongson@localhost ~]$ lscpu
>>> Architecture: mips64
>>> Byte Order: Little Endian
>>> CPU(s): 4
>>> On-line CPU(s) list: 0-3
>>> Thread(s) per core: 1
>>> Core(s) per socket: 4
>>> Socket(s): 1
>>> NUMA node(s): 1
>>> L1d cache: 64K
>>> L1i cache: 64K
>>> L2 cache: 2048K
>>> NUMA node0 CPU(s): 0-3
>>>
>>> With this patch:
>>>
>>> [loongson@localhost ~]$ lscpu
>>> Architecture: mips64
>>> Byte Order: Little Endian
>>> CPU(s): 4
>>> On-line CPU(s) list: 0-3
>>> Thread(s) per core: 1
>>> Core(s) per socket: 4
>>> Socket(s): 1
>>> NUMA node(s): 1
>>> Model name: Loongson-3A R3 (Loongson-3A3000) @ 1449MHz
>>> L1d cache: 64K
>>> L1i cache: 64K
>>> L2 cache: 2048K
>>> NUMA node0 CPU(s): 0-3
>>>
>>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>>> ---
>>> arch/mips/include/asm/cpu-info.h | 1 +
>>> arch/mips/kernel/cpu-probe.c | 27 +++++++++++++++++++++++----
>>> arch/mips/kernel/proc.c | 4 ++++
>>> 3 files changed, 28 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/arch/mips/include/asm/cpu-info.h
>>> b/arch/mips/include/asm/cpu-info.h
>>> index ed7ffe4..50e924e 100644
>>> --- a/arch/mips/include/asm/cpu-info.h
>>> +++ b/arch/mips/include/asm/cpu-info.h
>>> @@ -115,6 +115,7 @@ extern struct cpuinfo_mips cpu_data[];
>>> extern void cpu_probe(void);
>>> extern void cpu_report(void);
>>>
>>> +extern const char *__model_name[];
>>> extern const char *__cpu_name[];
>>> #define cpu_name_string() __cpu_name[raw_smp_processor_id()]
>>>
>>> diff --git a/arch/mips/kernel/cpu-probe.c
>>> b/arch/mips/kernel/cpu-probe.c
>>> index 6ab6b03..3ae40cc 100644
>>> --- a/arch/mips/kernel/cpu-probe.c
>>> +++ b/arch/mips/kernel/cpu-probe.c
>>> @@ -1548,6 +1548,7 @@ static inline void cpu_probe_legacy(struct
>>> cpuinfo_mips *c, unsigned int cpu)
>>> set_elf_platform(cpu, "loongson2e");
>>> set_isa(c, MIPS_CPU_ISA_III);
>>> c->fpu_msk31 |= FPU_CSR_CONDX;
>>> + __model_name[cpu] = "Loongson-2E";
>>> break;
>>> case PRID_REV_LOONGSON2F:
>>> c->cputype = CPU_LOONGSON2EF;
>>> @@ -1555,23 +1556,37 @@ static inline void cpu_probe_legacy(struct
>>> cpuinfo_mips *c, unsigned int cpu)
>>> set_elf_platform(cpu, "loongson2f");
>>> set_isa(c, MIPS_CPU_ISA_III);
>>> c->fpu_msk31 |= FPU_CSR_CONDX;
>>> + __model_name[cpu] = "Loongson-2F";
>>> break;
>>> case PRID_REV_LOONGSON3A_R1:
>>> c->cputype = CPU_LOONGSON64;
>>> __cpu_name[cpu] = "ICT Loongson-3";
>>> set_elf_platform(cpu, "loongson3a");
>>> set_isa(c, MIPS_CPU_ISA_M64R1);
>>> - c->ases |= (MIPS_ASE_LOONGSON_MMI |
>>> MIPS_ASE_LOONGSON_CAM |
>>> - MIPS_ASE_LOONGSON_EXT);
>>> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
>>> + MIPS_ASE_LOONGSON_CAM |
>>> + MIPS_ASE_LOONGSON_EXT);
>>> + __model_name[cpu] = "Loongson-3A R1
>>> (Loongson-3A1000)";
>>> break;
>>> case PRID_REV_LOONGSON3B_R1:
>>> + c->cputype = CPU_LOONGSON64;
>>> + __cpu_name[cpu] = "ICT Loongson-3";
>>> + set_elf_platform(cpu, "loongson3b");
>>> + set_isa(c, MIPS_CPU_ISA_M64R1);
>>> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
>>> + MIPS_ASE_LOONGSON_CAM |
>>> + MIPS_ASE_LOONGSON_EXT);
>>> + __model_name[cpu] = "Loongson-3B R1
>>> (Loongson-3B1000)";
>>> + break;
>>> case PRID_REV_LOONGSON3B_R2:
>>> c->cputype = CPU_LOONGSON64;
>>> __cpu_name[cpu] = "ICT Loongson-3";
>>> set_elf_platform(cpu, "loongson3b");
>>> set_isa(c, MIPS_CPU_ISA_M64R1);
>>> - c->ases |= (MIPS_ASE_LOONGSON_MMI |
>>> MIPS_ASE_LOONGSON_CAM |
>>> - MIPS_ASE_LOONGSON_EXT);
>>> + c->ases |= (MIPS_ASE_LOONGSON_MMI |
>>> + MIPS_ASE_LOONGSON_CAM |
>>> + MIPS_ASE_LOONGSON_EXT);
>>> + __model_name[cpu] = "Loongson-3B R2
>>> (Loongson-3B1500)";
>>> break;
>>> }
>>>
>>> @@ -1926,6 +1941,7 @@ static inline void cpu_probe_loongson(struct
>>> cpuinfo_mips *c, unsigned int cpu)
>>> __cpu_name[cpu] = "ICT Loongson-3";
>>> set_elf_platform(cpu, "loongson3a");
>>> set_isa(c, MIPS_CPU_ISA_M64R2);
>>> + __model_name[cpu] = "Loongson-3A R2
>>> (Loongson-3A2000)";
>>> break;
>>> case PRID_REV_LOONGSON3A_R3_0:
>>> case PRID_REV_LOONGSON3A_R3_1:
>>> @@ -1933,6 +1949,7 @@ static inline void cpu_probe_loongson(struct
>>> cpuinfo_mips *c, unsigned int cpu)
>>> __cpu_name[cpu] = "ICT Loongson-3";
>>> set_elf_platform(cpu, "loongson3a");
>>> set_isa(c, MIPS_CPU_ISA_M64R2);
>>> + __model_name[cpu] = "Loongson-3A R3
>>> (Loongson-3A3000)";
>>> break;
>>> }
>>>
>>> @@ -1952,6 +1969,7 @@ static inline void cpu_probe_loongson(struct
>>> cpuinfo_mips *c, unsigned int cpu)
>>> c->writecombine = _CACHE_UNCACHED_ACCELERATED;
>>> c->ases |= (MIPS_ASE_LOONGSON_MMI |
>>> MIPS_ASE_LOONGSON_CAM |
>>> MIPS_ASE_LOONGSON_EXT |
>>> MIPS_ASE_LOONGSON_EXT2);
>>> + __model_name[cpu] = "Loongson-3A R4 (Loongson-3A4000)";
>>> break;
>>> default:
>>> panic("Unknown Loongson Processor ID!");
>>> @@ -2111,6 +2129,7 @@ u64 __ua_limit;
>>> EXPORT_SYMBOL(__ua_limit);
>>> #endif
>>>
>>> +const char *__model_name[NR_CPUS];
>>> const char *__cpu_name[NR_CPUS];
>>> const char *__elf_platform;
>>>
>>> diff --git a/arch/mips/kernel/proc.c b/arch/mips/kernel/proc.c
>>> index f8d3671..5fc74e6 100644
>>> --- a/arch/mips/kernel/proc.c
>>> +++ b/arch/mips/kernel/proc.c
>>> @@ -15,6 +15,7 @@
>>> #include <asm/mipsregs.h>
>>> #include <asm/processor.h>
>>> #include <asm/prom.h>
>>> +#include <asm/time.h>
>>>
>>> unsigned int vced_count, vcei_count;
>>>
>>> @@ -63,6 +64,9 @@ static int show_cpuinfo(struct seq_file *m, void *v)
>>> seq_printf(m, fmt, __cpu_name[n],
>>> (version >> 4) & 0x0f, version & 0x0f,
>>> (fp_vers >> 4) & 0x0f, fp_vers & 0x0f);
>>> + if (__model_name[n])
>>> + seq_printf(m, "model name\t\t: %s @ %uMHz\n",
>>> + __model_name[n], mips_hpt_frequency / 500000);
>>> seq_printf(m, "BogoMIPS\t\t: %u.%02u\n",
>>> cpu_data[n].udelay_val / (500000/HZ),
>>> (cpu_data[n].udelay_val / (5000/HZ)) % 100);
>>> --
>>> 2.1.0
>>>
>>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
2020-03-11 6:33 [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo Tiezhu Yang
2020-03-11 6:53 ` 回复:[PATCH] " Jiaxun Yang
2020-03-11 7:18 ` [PATCH] " YunQiang Su
@ 2020-03-11 9:58 ` Manuel Lauss
2020-03-11 10:27 ` Jiaxun Yang
2 siblings, 1 reply; 11+ messages in thread
From: Manuel Lauss @ 2020-03-11 9:58 UTC (permalink / raw)
To: Tiezhu Yang
Cc: Thomas Bogendoerfer, Huacai Chen, Jiaxun Yang, Linux-MIPS, LKML,
Xuefeng Li
Hello,
> Model name: Loongson-3A R3 (Loongson-3A3000) @ 1449MHz
How is this different from the __cpu_name[] array? Doesn't it serve
the same purpose?
E.g. on Alchemy, it lists the correct Model name. (I.e. "Au1000", "Au1250", ...)
Manuel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
2020-03-11 9:58 ` Manuel Lauss
@ 2020-03-11 10:27 ` Jiaxun Yang
2020-03-20 21:24 ` Maciej W. Rozycki
0 siblings, 1 reply; 11+ messages in thread
From: Jiaxun Yang @ 2020-03-11 10:27 UTC (permalink / raw)
To: Manuel Lauss, Tiezhu Yang
Cc: Thomas Bogendoerfer, Huacai Chen, Linux-MIPS, LKML, Xuefeng Li
于 2020年3月11日 GMT+08:00 下午5:58:48, Manuel Lauss <manuel.lauss@gmail.com> 写到:
>Hello,
>
>> Model name: Loongson-3A R3 (Loongson-3A3000) @ 1449MHz
>
>How is this different from the __cpu_name[] array? Doesn't it serve
>the same purpose?
>E.g. on Alchemy, it lists the correct Model name. (I.e. "Au1000",
>"Au1250", ...)
Hi,
__cpu_name[] will be displayed as "cpu model" in cpuinfo, however
in x86 world, the name line is started with "model name".
Most user applications like lscpu, hwinfo is following x86's rule, we don't have superpower to move all of them.
Also rename "cpu model" will break current ABI, so just create a new array
for it would be a better option.
For why Loongson is doing this, I guess they need a chance to show
their business processor name instead of revision.
Thanks
>
>Manuel
--
Jiaxun Yang
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
2020-03-11 10:27 ` Jiaxun Yang
@ 2020-03-20 21:24 ` Maciej W. Rozycki
2020-03-21 1:20 ` YunQiang Su
0 siblings, 1 reply; 11+ messages in thread
From: Maciej W. Rozycki @ 2020-03-20 21:24 UTC (permalink / raw)
To: Jiaxun Yang
Cc: Manuel Lauss, Tiezhu Yang, Thomas Bogendoerfer, Huacai Chen,
Linux-MIPS, LKML, Xuefeng Li
On Wed, 11 Mar 2020, Jiaxun Yang wrote:
> __cpu_name[] will be displayed as "cpu model" in cpuinfo, however
> in x86 world, the name line is started with "model name".
> Most user applications like lscpu, hwinfo is following x86's rule, we don't have superpower to move all of them.
>
> Also rename "cpu model" will break current ABI, so just create a new array
> for it would be a better option.
Well, /proc/cpuinfo is not an ABI, it's meant for human interaction (use
`sysfs' for machine processing).
If tools want to use it anyway, then fine, but they need to adapt to the
relative volatility of the free-form text interface, and also learn all
the world is not x86 (just like all the MIPS world is not Loongson, so if
you argue about adding effectively a duplicate field, then that would have
to be applied across all the platforms). IOW fix the tool and do not work
its deficiencies around in the kernel just because it seems easier for you
to do.
Here's how example /proc/cpuinfo looks like on RISC-V/Linux BTW:
processor : 0
hart : 1
isa : rv64imafdc
mmu : sv39
uarch : sifive,rocket0
[...]
-- every Linux platform is different in this respect and tools have to
live with that.
Maciej
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo
2020-03-20 21:24 ` Maciej W. Rozycki
@ 2020-03-21 1:20 ` YunQiang Su
0 siblings, 0 replies; 11+ messages in thread
From: YunQiang Su @ 2020-03-21 1:20 UTC (permalink / raw)
To: Maciej W. Rozycki
Cc: Jiaxun Yang, Manuel Lauss, Tiezhu Yang, Thomas Bogendoerfer,
Huacai Chen, Linux-MIPS, LKML, Xuefeng Li
Maciej W. Rozycki <macro@linux-mips.org> 于2020年3月21日周六 上午5:24写道：
>
> On Wed, 11 Mar 2020, Jiaxun Yang wrote:
>
> > __cpu_name[] will be displayed as "cpu model" in cpuinfo, however
> > in x86 world, the name line is started with "model name".
> > Most user applications like lscpu, hwinfo is following x86's rule, we don't have superpower to move all of them.
> >
> > Also rename "cpu model" will break current ABI, so just create a new array
> > for it would be a better option.
>
> Well, /proc/cpuinfo is not an ABI, it's meant for human interaction (use
> `sysfs' for machine processing).
In fact there do be some code to use cpuinfo to detect cpu. Gcc is
even in this list.
https://codesearch.debian.net/search?q=cpuinfo&literal=1
>
> If tools want to use it anyway, then fine, but they need to adapt to the
> relative volatility of the free-form text interface, and also learn all
> the world is not x86 (just like all the MIPS world is not Loongson, so if
> you argue about adding effectively a duplicate field, then that would have
> to be applied across all the platforms). IOW fix the tool and do not work
> its deficiencies around in the kernel just because it seems easier for you
> to do.
>
> Here's how example /proc/cpuinfo looks like on RISC-V/Linux BTW:
>
> processor : 0
> hart : 1
> isa : rv64imafdc
> mmu : sv39
> uarch : sifive,rocket0
If we create cpuinfo for a new architecture, we are freely do anything.
it is OK, as the newly write code can know about it, and there is no
old codes about it.
While if we have some modification to /proc/cpuinfo, of exists architectures,
we must be very careful to keep compatible.
>
> [...]
>
> -- every Linux platform is different in this respect and tools have to
> live with that.
Yes, you are some right, the scripts/programs should be robust,
while they cannot forecast how the people change cpuinfo, if they are
freely....
>
> Maciej
--
YunQiang Su
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2020-03-21 1:20 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-11 6:33 [PATCH] MIPS: Loongson: Add model name to /proc/cpuinfo Tiezhu Yang
2020-03-11 6:53 ` 回复:[PATCH] " Jiaxun Yang
2020-03-11 9:28 ` Tiezhu Yang
2020-03-11 7:18 ` [PATCH] " YunQiang Su
2020-03-11 7:41 ` Jiaxun Yang
2020-03-11 9:40 ` Tiezhu Yang
2020-03-12 7:25 ` Tiezhu Yang
2020-03-11 9:58 ` Manuel Lauss
2020-03-11 10:27 ` Jiaxun Yang
2020-03-20 21:24 ` Maciej W. Rozycki
2020-03-21 1:20 ` YunQiang Su
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.