Re: [PATCH]selftests/powerpc: skip tests for unavailable mitigations.

2021-12-21 Thread Michael Ellerman
On Mon, 13 Dec 2021 22:12:23 +0530, Sachin Sant wrote:
> Mitigation patching test iterates over a set of mitigations irrespective
> of whether a certain mitigation is supported/available in the kernel.
> This causes following messages on a kernel where some mitigations
> are unavailable:
> 
>   Spawned threads enabling/disabling mitigations ...
>   cat: entry_flush: No such file or directory
>   cat: uaccess_flush: No such file or directory
>   Waiting for timeout ...
>   OK
> 
> [...]

Applied to powerpc/next.

[1/1] selftests/powerpc: skip tests for unavailable mitigations.
  https://git.kernel.org/powerpc/c/18678591846d668649fbd4f87b4a4c470818d386

cheers


Re: [PATCH]selftests/powerpc: skip tests for unavailable mitigations.

2021-12-14 Thread Russell Currey
On Mon, 2021-12-13 at 22:12 +0530, Sachin Sant wrote:
> Mitigation patching test iterates over a set of mitigations
> irrespective
> of whether a certain mitigation is supported/available in the kernel.
> This causes following messages on a kernel where some mitigations
> are unavailable:
> 
>   Spawned threads enabling/disabling mitigations ...
>   cat: entry_flush: No such file or directory
>   cat: uaccess_flush: No such file or directory
>   Waiting for timeout ...
>   OK
> 
> This patch adds a check for available mitigations in the kernel.
> 
> Reported-by: Nageswara R Sastry 
> Signed-off-by: Sachin Sant 

Reviewed-by: Russell Currey 


Re: [PATCH]selftests/powerpc: skip tests for unavailable mitigations.

2021-12-13 Thread Nageswara Sastry




On 13/12/21 10:12 pm, Sachin Sant wrote:

Mitigation patching test iterates over a set of mitigations irrespective
of whether a certain mitigation is supported/available in the kernel.
This causes following messages on a kernel where some mitigations
are unavailable:

   Spawned threads enabling/disabling mitigations ...
   cat: entry_flush: No such file or directory
   cat: uaccess_flush: No such file or directory
   Waiting for timeout ...
   OK

This patch adds a check for available mitigations in the kernel.

Reported-by: Nageswara R Sastry 
Signed-off-by: Sachin Sant 


Tested-by: Nageswara R Sastry 


---
diff -Naurp aa/tools/testing/selftests/powerpc/security/mitigation-patching.sh 
bb/tools/testing/selftests/powerpc/security/mitigation-patching.sh
--- aa/tools/testing/selftests/powerpc/security/mitigation-patching.sh  
2021-12-13 10:17:05.714127154 -0500
+++ bb/tools/testing/selftests/powerpc/security/mitigation-patching.sh  
2021-12-13 10:19:32.575315913 -0500
@@ -44,7 +44,10 @@ mitigations="barrier_nospec stf_barrier
  
  for m in $mitigations

  do
-do_one "$m" &
+if [[ -f /sys/kernel/debug/powerpc/$m ]]
+then
+do_one "$m" &
+fi
  done
  
  echo "Spawned threads enabling/disabling mitigations ..."


--
Thanks and Regards
R.Nageswara Sastry


[PATCH]selftests/powerpc: skip tests for unavailable mitigations.

2021-12-13 Thread Sachin Sant
Mitigation patching test iterates over a set of mitigations irrespective
of whether a certain mitigation is supported/available in the kernel.
This causes following messages on a kernel where some mitigations
are unavailable:

  Spawned threads enabling/disabling mitigations ...
  cat: entry_flush: No such file or directory
  cat: uaccess_flush: No such file or directory
  Waiting for timeout ...
  OK

This patch adds a check for available mitigations in the kernel.

Reported-by: Nageswara R Sastry 
Signed-off-by: Sachin Sant 
---
diff -Naurp aa/tools/testing/selftests/powerpc/security/mitigation-patching.sh 
bb/tools/testing/selftests/powerpc/security/mitigation-patching.sh
--- aa/tools/testing/selftests/powerpc/security/mitigation-patching.sh  
2021-12-13 10:17:05.714127154 -0500
+++ bb/tools/testing/selftests/powerpc/security/mitigation-patching.sh  
2021-12-13 10:19:32.575315913 -0500
@@ -44,7 +44,10 @@ mitigations="barrier_nospec stf_barrier
 
 for m in $mitigations
 do
-do_one "$m" &
+if [[ -f /sys/kernel/debug/powerpc/$m ]]
+then
+do_one "$m" &
+fi
 done
 
 echo "Spawned threads enabling/disabling mitigations ..."