2018-03-21 14:55:05

by Jiaxun Yang

[permalink] [raw]
Subject: [PATCH 1/2] MIPS: Introduce has_cpu_mips*_user in cpu-features.h

Some processors support user mode instructions ISA level witch is
different with the ISA level it should be treated in kernel, such
as Loongson 3A1000 3B1000 3A1500 3B1500 support all mips64r2 user
mode instructions however, they should be treated as mips64r1 in
kernel.

So we introduce has_cpu_mips*_user to decide witch level should be
displayed in cpuinfo to prevent misleading userspace programs.

Signed-off-by: Jiaxun Yang <[email protected]>
---
arch/mips/include/asm/cpu-features.h | 39 ++++++++++++++++++++++++++++++++++++
arch/mips/kernel/proc.c | 22 ++++++++++----------
2 files changed, 50 insertions(+), 11 deletions(-)

diff --git a/arch/mips/include/asm/cpu-features.h b/arch/mips/include/asm/cpu-features.h
index 721b698bfe3c..0eff1956e229 100644
--- a/arch/mips/include/asm/cpu-features.h
+++ b/arch/mips/include/asm/cpu-features.h
@@ -251,6 +251,45 @@
# define cpu_has_mips64r6 (cpu_data[0].isa_level & MIPS_CPU_ISA_M64R6)
#endif

+/*
+ * For the CPU that has a user mode instructions ISA level witch is different
+ * from the ISA level it should be treated in kernel, this ISA level will
+ * be displayed in cpuinfo as a reference for userspace programs.
+ */
+#ifndef cpu_has_mips_1_user
+# define cpu_has_mips_1_user (cpu_has_mips_1)
+#endif
+#ifndef cpu_has_mips_2_user
+# define cpu_has_mips_2_user (cpu_has_mips_2)
+#endif
+#ifndef cpu_has_mips_3_user
+# define cpu_has_mips_3_user (cpu_has_mips_3)
+#endif
+#ifndef cpu_has_mips_4_user
+# define cpu_has_mips_4_user (cpu_has_mips_4)
+#endif
+#ifndef cpu_has_mips_5_user
+# define cpu_has_mips_5_user (cpu_has_mips_5)
+#endif
+#ifndef cpu_has_mips32r1_user
+# define cpu_has_mips32r1_user (cpu_has_mips32r1)
+#endif
+#ifndef cpu_has_mips32r2_user
+# define cpu_has_mips32r2_user (cpu_has_mips32r2)
+#endif
+#ifndef cpu_has_mips32r6_user
+# define cpu_has_mips32r6_user (cpu_has_mips32r6)
+#endif
+#ifndef cpu_has_mips64r1_user
+# define cpu_has_mips64r1_user (cpu_has_mips64r1)
+#endif
+#ifndef cpu_has_mips64r2_user
+# define cpu_has_mips64r2_user (cpu_has_mips64r2)
+#endif
+#ifndef cpu_has_mips64r6_user
+# define cpu_has_mips64r6_user (cpu_has_mips64r6)
+#endif
+
/*
* Shortcuts ...
*/
diff --git a/arch/mips/kernel/proc.c b/arch/mips/kernel/proc.c
index b2de408a259e..65a9a695af3c 100644
--- a/arch/mips/kernel/proc.c
+++ b/arch/mips/kernel/proc.c
@@ -84,27 +84,27 @@ static int show_cpuinfo(struct seq_file *m, void *v)
}

seq_printf(m, "isa\t\t\t:");
- if (cpu_has_mips_1)
+ if (cpu_has_mips_1_user)
seq_printf(m, " mips1");
- if (cpu_has_mips_2)
+ if (cpu_has_mips_2_user)
seq_printf(m, "%s", " mips2");
- if (cpu_has_mips_3)
+ if (cpu_has_mips_3_user)
seq_printf(m, "%s", " mips3");
- if (cpu_has_mips_4)
+ if (cpu_has_mips_4_user)
seq_printf(m, "%s", " mips4");
- if (cpu_has_mips_5)
+ if (cpu_has_mips_5_user)
seq_printf(m, "%s", " mips5");
- if (cpu_has_mips32r1)
+ if (cpu_has_mips32r1_user)
seq_printf(m, "%s", " mips32r1");
- if (cpu_has_mips32r2)
+ if (cpu_has_mips32r2_user)
seq_printf(m, "%s", " mips32r2");
- if (cpu_has_mips32r6)
+ if (cpu_has_mips32r6_user)
seq_printf(m, "%s", " mips32r6");
- if (cpu_has_mips64r1)
+ if (cpu_has_mips64r1_user)
seq_printf(m, "%s", " mips64r1");
- if (cpu_has_mips64r2)
+ if (cpu_has_mips64r2_user)
seq_printf(m, "%s", " mips64r2");
- if (cpu_has_mips64r6)
+ if (cpu_has_mips64r6_user)
seq_printf(m, "%s", " mips64r6");
seq_printf(m, "\n");

--
2.16.2



2018-03-21 14:55:20

by Jiaxun Yang

[permalink] [raw]
Subject: [PATCH 2/2] MIPS: Loongson64: Define has_cpu_mips64r2_user for Loongson-3

All loongson-3 processors support mips64r2 usermode instructions.
However 3A1000 3B1000 3B1500 should be treated as mips64r1 in kernel.

Signed-off-by: Jiaxun Yang <[email protected]>
---
arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h b/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h
index 581915ce231c..71c893249374 100644
--- a/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h
+++ b/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h
@@ -49,6 +49,7 @@
#define cpu_has_wsbh 1
#define cpu_has_ic_fills_f_dc 1
#define cpu_hwrena_impl_bits 0xc0000000
+#define cpu_has_mips64r2_user 1
#endif

#endif /* __ASM_MACH_LOONGSON64_CPU_FEATURE_OVERRIDES_H */
--
2.16.2


2018-03-22 13:45:33

by James Hogan

[permalink] [raw]
Subject: Re: [PATCH 1/2] MIPS: Introduce has_cpu_mips*_user in cpu-features.h

On Wed, Mar 21, 2018 at 10:53:03PM +0800, Jiaxun Yang wrote:
> Some processors support user mode instructions ISA level witch is

nit: s/witch/which/ here, below, and in the comment.

Otherwise it doesn't look unreasonable.

Cheers
James

> different with the ISA level it should be treated in kernel, such
> as Loongson 3A1000 3B1000 3A1500 3B1500 support all mips64r2 user
> mode instructions however, they should be treated as mips64r1 in
> kernel.
>
> So we introduce has_cpu_mips*_user to decide witch level should be
> displayed in cpuinfo to prevent misleading userspace programs.
>
> Signed-off-by: Jiaxun Yang <[email protected]>
> ---
> arch/mips/include/asm/cpu-features.h | 39 ++++++++++++++++++++++++++++++++++++
> arch/mips/kernel/proc.c | 22 ++++++++++----------
> 2 files changed, 50 insertions(+), 11 deletions(-)
>
> diff --git a/arch/mips/include/asm/cpu-features.h b/arch/mips/include/asm/cpu-features.h
> index 721b698bfe3c..0eff1956e229 100644
> --- a/arch/mips/include/asm/cpu-features.h
> +++ b/arch/mips/include/asm/cpu-features.h
> @@ -251,6 +251,45 @@
> # define cpu_has_mips64r6 (cpu_data[0].isa_level & MIPS_CPU_ISA_M64R6)
> #endif
>
> +/*
> + * For the CPU that has a user mode instructions ISA level witch is different
> + * from the ISA level it should be treated in kernel, this ISA level will
> + * be displayed in cpuinfo as a reference for userspace programs.
> + */
> +#ifndef cpu_has_mips_1_user
> +# define cpu_has_mips_1_user (cpu_has_mips_1)
> +#endif
> +#ifndef cpu_has_mips_2_user
> +# define cpu_has_mips_2_user (cpu_has_mips_2)
> +#endif
> +#ifndef cpu_has_mips_3_user
> +# define cpu_has_mips_3_user (cpu_has_mips_3)
> +#endif
> +#ifndef cpu_has_mips_4_user
> +# define cpu_has_mips_4_user (cpu_has_mips_4)
> +#endif
> +#ifndef cpu_has_mips_5_user
> +# define cpu_has_mips_5_user (cpu_has_mips_5)
> +#endif
> +#ifndef cpu_has_mips32r1_user
> +# define cpu_has_mips32r1_user (cpu_has_mips32r1)
> +#endif
> +#ifndef cpu_has_mips32r2_user
> +# define cpu_has_mips32r2_user (cpu_has_mips32r2)
> +#endif
> +#ifndef cpu_has_mips32r6_user
> +# define cpu_has_mips32r6_user (cpu_has_mips32r6)
> +#endif
> +#ifndef cpu_has_mips64r1_user
> +# define cpu_has_mips64r1_user (cpu_has_mips64r1)
> +#endif
> +#ifndef cpu_has_mips64r2_user
> +# define cpu_has_mips64r2_user (cpu_has_mips64r2)
> +#endif
> +#ifndef cpu_has_mips64r6_user
> +# define cpu_has_mips64r6_user (cpu_has_mips64r6)
> +#endif
> +
> /*
> * Shortcuts ...
> */
> diff --git a/arch/mips/kernel/proc.c b/arch/mips/kernel/proc.c
> index b2de408a259e..65a9a695af3c 100644
> --- a/arch/mips/kernel/proc.c
> +++ b/arch/mips/kernel/proc.c
> @@ -84,27 +84,27 @@ static int show_cpuinfo(struct seq_file *m, void *v)
> }
>
> seq_printf(m, "isa\t\t\t:");
> - if (cpu_has_mips_1)
> + if (cpu_has_mips_1_user)
> seq_printf(m, " mips1");
> - if (cpu_has_mips_2)
> + if (cpu_has_mips_2_user)
> seq_printf(m, "%s", " mips2");
> - if (cpu_has_mips_3)
> + if (cpu_has_mips_3_user)
> seq_printf(m, "%s", " mips3");
> - if (cpu_has_mips_4)
> + if (cpu_has_mips_4_user)
> seq_printf(m, "%s", " mips4");
> - if (cpu_has_mips_5)
> + if (cpu_has_mips_5_user)
> seq_printf(m, "%s", " mips5");
> - if (cpu_has_mips32r1)
> + if (cpu_has_mips32r1_user)
> seq_printf(m, "%s", " mips32r1");
> - if (cpu_has_mips32r2)
> + if (cpu_has_mips32r2_user)
> seq_printf(m, "%s", " mips32r2");
> - if (cpu_has_mips32r6)
> + if (cpu_has_mips32r6_user)
> seq_printf(m, "%s", " mips32r6");
> - if (cpu_has_mips64r1)
> + if (cpu_has_mips64r1_user)
> seq_printf(m, "%s", " mips64r1");
> - if (cpu_has_mips64r2)
> + if (cpu_has_mips64r2_user)
> seq_printf(m, "%s", " mips64r2");
> - if (cpu_has_mips64r6)
> + if (cpu_has_mips64r6_user)
> seq_printf(m, "%s", " mips64r6");
> seq_printf(m, "\n");
>
> --
> 2.16.2
>


Attachments:
(No filename) (3.88 kB)
signature.asc (849.00 B)
Digital signature
Download all attachments

2018-03-22 14:35:02

by Jiaxun Yang

[permalink] [raw]
Subject: [PATCH v2 1/2] MIPS: Introduce has_cpu_mips*_user in cpu-features.h

Some processors support user mode instructions ISA level which is
different with the ISA level it should be treated in kernel, such
as Loongson 3A1000 3B1000 3A1500 3B1500 support all mips64r2 user
mode instructions however, they should be treated as mips64r1 in
kernel.

So we introduce has_cpu_mips*_user to decide which level should be
displayed in cpuinfo to prevent misleading userspace programs.

Signed-off-by: Jiaxun Yang <[email protected]>
---
arch/mips/include/asm/cpu-features.h | 39 ++++++++++++++++++++++++++++++++++++
arch/mips/kernel/proc.c | 22 ++++++++++----------
2 files changed, 50 insertions(+), 11 deletions(-)

diff --git a/arch/mips/include/asm/cpu-features.h b/arch/mips/include/asm/cpu-features.h
index 721b698bfe3c..0eff1956e229 100644
--- a/arch/mips/include/asm/cpu-features.h
+++ b/arch/mips/include/asm/cpu-features.h
@@ -251,6 +251,45 @@
# define cpu_has_mips64r6 (cpu_data[0].isa_level & MIPS_CPU_ISA_M64R6)
#endif

+/*
+ * For the CPU that has a user mode instructions ISA level which is different
+ * from the ISA level it should be treated in kernel, this ISA level will
+ * be displayed in cpuinfo as a reference for userspace programs.
+ */
+#ifndef cpu_has_mips_1_user
+# define cpu_has_mips_1_user (cpu_has_mips_1)
+#endif
+#ifndef cpu_has_mips_2_user
+# define cpu_has_mips_2_user (cpu_has_mips_2)
+#endif
+#ifndef cpu_has_mips_3_user
+# define cpu_has_mips_3_user (cpu_has_mips_3)
+#endif
+#ifndef cpu_has_mips_4_user
+# define cpu_has_mips_4_user (cpu_has_mips_4)
+#endif
+#ifndef cpu_has_mips_5_user
+# define cpu_has_mips_5_user (cpu_has_mips_5)
+#endif
+#ifndef cpu_has_mips32r1_user
+# define cpu_has_mips32r1_user (cpu_has_mips32r1)
+#endif
+#ifndef cpu_has_mips32r2_user
+# define cpu_has_mips32r2_user (cpu_has_mips32r2)
+#endif
+#ifndef cpu_has_mips32r6_user
+# define cpu_has_mips32r6_user (cpu_has_mips32r6)
+#endif
+#ifndef cpu_has_mips64r1_user
+# define cpu_has_mips64r1_user (cpu_has_mips64r1)
+#endif
+#ifndef cpu_has_mips64r2_user
+# define cpu_has_mips64r2_user (cpu_has_mips64r2)
+#endif
+#ifndef cpu_has_mips64r6_user
+# define cpu_has_mips64r6_user (cpu_has_mips64r6)
+#endif
+
/*
* Shortcuts ...
*/
diff --git a/arch/mips/kernel/proc.c b/arch/mips/kernel/proc.c
index b2de408a259e..65a9a695af3c 100644
--- a/arch/mips/kernel/proc.c
+++ b/arch/mips/kernel/proc.c
@@ -84,27 +84,27 @@ static int show_cpuinfo(struct seq_file *m, void *v)
}

seq_printf(m, "isa\t\t\t:");
- if (cpu_has_mips_1)
+ if (cpu_has_mips_1_user)
seq_printf(m, " mips1");
- if (cpu_has_mips_2)
+ if (cpu_has_mips_2_user)
seq_printf(m, "%s", " mips2");
- if (cpu_has_mips_3)
+ if (cpu_has_mips_3_user)
seq_printf(m, "%s", " mips3");
- if (cpu_has_mips_4)
+ if (cpu_has_mips_4_user)
seq_printf(m, "%s", " mips4");
- if (cpu_has_mips_5)
+ if (cpu_has_mips_5_user)
seq_printf(m, "%s", " mips5");
- if (cpu_has_mips32r1)
+ if (cpu_has_mips32r1_user)
seq_printf(m, "%s", " mips32r1");
- if (cpu_has_mips32r2)
+ if (cpu_has_mips32r2_user)
seq_printf(m, "%s", " mips32r2");
- if (cpu_has_mips32r6)
+ if (cpu_has_mips32r6_user)
seq_printf(m, "%s", " mips32r6");
- if (cpu_has_mips64r1)
+ if (cpu_has_mips64r1_user)
seq_printf(m, "%s", " mips64r1");
- if (cpu_has_mips64r2)
+ if (cpu_has_mips64r2_user)
seq_printf(m, "%s", " mips64r2");
- if (cpu_has_mips64r6)
+ if (cpu_has_mips64r6_user)
seq_printf(m, "%s", " mips64r6");
seq_printf(m, "\n");

--
2.16.2


2018-03-22 14:36:28

by Jiaxun Yang

[permalink] [raw]
Subject: [PATCH v2 2/2] MIPS: Loongson64: Define has_cpu_mips*r2_user for Loongson-3

All loongson-3 processors support mips32r2 mips64r2 usermode instructions.
However 3A1000 3B1000 3B1500 should be treated as r1 in kernel.

Signed-off-by: Jiaxun Yang <[email protected]>
---
arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h b/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h
index 581915ce231c..71c893249374 100644
--- a/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h
+++ b/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h
@@ -49,6 +49,8 @@
#define cpu_has_wsbh 1
#define cpu_has_ic_fills_f_dc 1
#define cpu_hwrena_impl_bits 0xc0000000
+#define cpu_has_mips32r2_user 1
+#define cpu_has_mips64r2_user 1
#endif

#endif /* __ASM_MACH_LOONGSON64_CPU_FEATURE_OVERRIDES_H */
--
2.16.2


2018-03-22 14:49:24

by James Hogan

[permalink] [raw]
Subject: Re: [PATCH 2/2] MIPS: Loongson64: Define has_cpu_mips64r2_user for Loongson-3

On Wed, Mar 21, 2018 at 10:53:04PM +0800, Jiaxun Yang wrote:
> All loongson-3 processors support mips64r2 usermode instructions.
> However 3A1000 3B1000 3B1500 should be treated as mips64r1 in kernel.
>
> Signed-off-by: Jiaxun Yang <[email protected]>
> ---
> arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h b/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h
> index 581915ce231c..71c893249374 100644
> --- a/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h
> +++ b/arch/mips/include/asm/mach-loongson64/cpu-feature-overrides.h
> @@ -49,6 +49,7 @@
> #define cpu_has_wsbh 1
> #define cpu_has_ic_fills_f_dc 1
> #define cpu_hwrena_impl_bits 0xc0000000
> +#define cpu_has_mips64r2_user 1

What about cpu_has_mips32r2_user?

Cheers
James


Attachments:
(No filename) (929.00 B)
signature.asc (849.00 B)
Digital signature
Download all attachments