Re: [PATCH v3] perf: tests: Disable breakpoint tests on ARM (32-bit)
From: Arnaldo Carvalho de Melo
Date: Tue Dec 04 2018 - 09:13:02 EST
Em Tue, Dec 04, 2018 at 02:06:16PM +0000, Will Deacon escreveu:
> On Mon, Dec 03, 2018 at 11:11:36AM -0800, Florian Fainelli wrote:
> > breakpoint tests on the ARM 32-bit kernel are broken in several ways.
> >
> > The breakpoint length requested does not necessarily match whether the
> > function address has the Thumb bit (bit 0) set or not, and this does
> > matter to the ARM kernel hw_breakpoint infrastructure. See [1] for
> > background.
> >
> > [1]: https://lkml.org/lkml/2018/11/15/205
> >
> > As Will indicated, the overflow handling would require single-stepping
> > which is not supported at the moment. Just disable those tests for the
> > ARM 32-bit platforms and update the comment above to explain these
> > limitations.
> >
> > Signed-off-by: Florian Fainelli <f.fainelli@xxxxxxxxx>
> > ---
> > Changes in v3:
> >
> > - directly incorporate Will's comment, Will can you add your
> > Signed-off-by?
>
> Of course:
>
> Co-developed-by: Will Deacon <will.deacon@xxxxxxx>
> Signed-off-by: Will Deacon <will.deacon@xxxxxxx>
Thanks, added those and applied the patch.
- Arnaldo