[F][SRU][PATCH 1/1] selftests/eeh: Skip ahci adapters

[Guilherme Piccoli]

On Wed, Feb 24, 2021 at 4:15 AM Po-Hsu Lin <po-hsu.lin at canonical.com> wrote:
>
> From: Michael Ellerman <mpe at ellerman.id.au>
>
> BugLink: https://bugs.launchpad.net/bugs/1916468
>
> The ahci driver doesn't support error recovery, and if your root
> filesystem is attached to it the eeh-basic.sh test will likely kill
> your machine.
>
> So skip any device we see using the ahci driver.
>
> Signed-off-by: Michael Ellerman <mpe at ellerman.id.au>
> Link: https://lore.kernel.org/r/20200326061144.2006522-1-mpe@ellerman.id.au
> (cherry picked from commit bbe9064f30f06e7791d04f9f61a842226a6a44fe)
> Signed-off-by: Po-Hsu Lin <po-hsu.lin at canonical.com>
> ---
>  tools/testing/selftests/powerpc/eeh/eeh-basic.sh | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/tools/testing/selftests/powerpc/eeh/eeh-basic.sh b/tools/testing/selftests/powerpc/eeh/eeh-basic.sh
> index cf001a2..0d783e1 100755
> --- a/tools/testing/selftests/powerpc/eeh/eeh-basic.sh
> +++ b/tools/testing/selftests/powerpc/eeh/eeh-basic.sh
> @@ -43,6 +43,11 @@ for dev in `ls -1 /sys/bus/pci/devices/ | grep '\.0$'` ; do
>                 continue;
>         fi
>
> +       if [ "ahci" = "$(basename $(realpath /sys/bus/pci/devices/$dev/driver))" ] ; then
> +               echo "$dev, Skipped: ahci doesn't support recovery"
> +               continue
> +       fi
> +
>         # Don't inject errosr into an already-frozen PE. This happens with
>         # PEs that contain multiple PCI devices (e.g. multi-function cards)
>         # and injecting new errors during the recovery process will probably
> --
> 2.7.4
>
>

Makes total sense, thanks Po-Hsu Lin!
Acked-by: Guilherme G. Piccoli <gpiccoli at canonical.com>