diff mbox series

[v5,02/10] arm64: add sysfs vulnerability show for spectre v1

Message ID 20190227010544.597579-3-jeremy.linton@arm.com
State Superseded
Headers show
Series arm64: add system vulnerability sysfs entries | expand

Commit Message

Jeremy Linton Feb. 27, 2019, 1:05 a.m. UTC
From: Mian Yousaf Kaukab <ykaukab@suse.de>


spectre v1, has been mitigated, and the mitigation is
always active.

Signed-off-by: Mian Yousaf Kaukab <ykaukab@suse.de>

Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>

---
 arch/arm64/kernel/cpu_errata.c | 6 ++++++
 1 file changed, 6 insertions(+)

-- 
2.20.1

Comments

Suzuki K Poulose Feb. 28, 2019, 6:29 p.m. UTC | #1
On 27/02/2019 01:05, Jeremy Linton wrote:
> From: Mian Yousaf Kaukab <ykaukab@suse.de>

> 

> spectre v1, has been mitigated, and the mitigation is

> always active.

> 

> Signed-off-by: Mian Yousaf Kaukab <ykaukab@suse.de>

> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>

> ---

>   arch/arm64/kernel/cpu_errata.c | 6 ++++++

>   1 file changed, 6 insertions(+)

> 

> diff --git a/arch/arm64/kernel/cpu_errata.c b/arch/arm64/kernel/cpu_errata.c

> index d2b2c69d31bb..ad58958becb6 100644

> --- a/arch/arm64/kernel/cpu_errata.c

> +++ b/arch/arm64/kernel/cpu_errata.c

> @@ -755,3 +755,9 @@ const struct arm64_cpu_capabilities arm64_errata[] = {

>   	{

>   	}

>   };

> +

> +ssize_t cpu_show_spectre_v1(struct device *dev, struct device_attribute *attr,

> +		char *buf)

> +{

> +	return sprintf(buf, "Mitigation: __user pointer sanitization\n");

> +}

> 


minor nit: This could possibly have been in the cpufeature.c, where we keep
the spectre_v2 routine.

Either way,

Acked-by: Suzuki K Poulose <suzuki.poulose@arm.com>
Andre Przywara March 1, 2019, 6:54 a.m. UTC | #2
Hi,

On 2/26/19 7:05 PM, Jeremy Linton wrote:
> From: Mian Yousaf Kaukab <ykaukab@suse.de>

> 

> spectre v1, has been mitigated, and the mitigation is

> always active.

> 

> Signed-off-by: Mian Yousaf Kaukab <ykaukab@suse.de>

> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>

> ---

>   arch/arm64/kernel/cpu_errata.c | 6 ++++++

>   1 file changed, 6 insertions(+)

> 

> diff --git a/arch/arm64/kernel/cpu_errata.c b/arch/arm64/kernel/cpu_errata.c

> index d2b2c69d31bb..ad58958becb6 100644

> --- a/arch/arm64/kernel/cpu_errata.c

> +++ b/arch/arm64/kernel/cpu_errata.c

> @@ -755,3 +755,9 @@ const struct arm64_cpu_capabilities arm64_errata[] = {

>   	{

>   	}

>   };

> +

> +ssize_t cpu_show_spectre_v1(struct device *dev, struct device_attribute *attr,

> +		char *buf)


w/s issue, but it's not critical:

Reviewed-by: Andre Przywara <andre.przywara@arm.com>


Cheers,
Andre

> +{

> +	return sprintf(buf, "Mitigation: __user pointer sanitization\n");

> +}

>
diff mbox series

Patch

diff --git a/arch/arm64/kernel/cpu_errata.c b/arch/arm64/kernel/cpu_errata.c
index d2b2c69d31bb..ad58958becb6 100644
--- a/arch/arm64/kernel/cpu_errata.c
+++ b/arch/arm64/kernel/cpu_errata.c
@@ -755,3 +755,9 @@  const struct arm64_cpu_capabilities arm64_errata[] = {
 	{
 	}
 };
+
+ssize_t cpu_show_spectre_v1(struct device *dev, struct device_attribute *attr,
+		char *buf)
+{
+	return sprintf(buf, "Mitigation: __user pointer sanitization\n");
+}