[BUG] perf, arm64, acpi: sleeping function called from invalid context

[Linus Walleij]

On Tue, Jan 30, 2018 at 2:48 PM, Will Deacon <will.deacon at arm.com> wrote:

>> Changing the allocations in arm_pmu_alloc() to GFP_ATOMIC didn't help,
>> as the interrupt request is also not happy in this context.
>>
>> Would it be possible to init the PMUs later?
>
> I know that Mark's had a good go at fixing this, but we ran into problems
> having the fix co-exist with the IRQ bouncing workaround we perform for the
> PMU on U8500 platforms. Frustratingly, those platforms don't appear to be
> available any more, so we're being held up by something that we're unable
> to test and might be considered dead.

Not any more dead than the OMAP3 based Nokia n900 or the
Samsung S3C stuff I would say. Just these don't have weird PMU
counter interrupts :D

The Ux500 Galaxy S Advance phone from Samsung was picked up by
hobbyists from PostmarketOS as a hacking target, maybe that needs to
become more accessible using just upstream, mea culpa.

I am willing to test any patches on the reference designs though.

> Linus, Lee: do we still need to support PMU interrupts on U8500? It's
> causing us real headaches with ACPI-based arm64 systems. [the answer might
> be "yes", but I have to ask!]

Can we still use perf without the IRQs? I.e. I guess it will it fall
back to some
sampling method that is "good enough"? We don't do a lot of performance
testing using perf admittedly.

I do think it is unfair to have to support a bunch of weird
workarounds just because ST Micro decided to connect these two IRQs
to the same GIC line using an OR gate. IIRC that was the issue.

Yours,
Linus Walleij