ACK/cmnt: [B][SRU][PATCH 0/1] selftests/powerpc: Remove redundant cp_abort test

Po-Hsu Lin po-hsu.lin at canonical.com
Fri May 3 17:09:48 UTC 2019


Oops,
my bad, must be a wrong copy-paste.
Thanks for spotting this.



On Fri, May 3, 2019 at 10:40 PM Kleber Souza <kleber.souza at canonical.com> wrote:
>
> On 4/25/19 5:34 AM, Po-Hsu Lin wrote:
> > BugLink: https://bugs.launchpad.net/bugs/1812805
>
> The cover letter is mentioning another bug, but the patch itself
> has the correct reference.
>
> >
> > == Justification ==
> > The cp_abort test in context_switch was designed for Power9, however
> > this test is broken and it fails everytime as Paste on Power9 only
> > works on accelerators and no longer on real memory.
> >
> > == Fix ==
> > 00c946a0 (selftests/powerpc: Remove redundant cp_abort test)
> >
> > This will remove the cp_abort test.
> > Patch has already been applied in C and onward.
> >
> > == Test ==
> > Patch verified on a Bionic P9 system, the test will no longer exist.
> >
> > == Regression Potential ==
> > None.
> > This patchset is just for the kernel testing tool.
> >
> >
> > Michael Neuling (1):
> >   selftests/powerpc: Remove redundant cp_abort test
> >
> >  tools/testing/selftests/powerpc/Makefile           |   1 -
> >  .../selftests/powerpc/context_switch/.gitignore    |   1 -
> >  .../selftests/powerpc/context_switch/Makefile      |   5 -
> >  .../selftests/powerpc/context_switch/cp_abort.c    | 110 ---------------------
> >  4 files changed, 117 deletions(-)
> >  delete mode 100644 tools/testing/selftests/powerpc/context_switch/.gitignore
> >  delete mode 100644 tools/testing/selftests/powerpc/context_switch/Makefile
> >  delete mode 100644 tools/testing/selftests/powerpc/context_switch/cp_abort.c
> >
>
>
> Acked-by: Kleber Sacilotto de Souza <kleber.souza at canonical.com>



More information about the kernel-team mailing list