[BUG] perf, arm64, acpi: sleeping function called from invalid context
From: Linus Walleij <hidden>
Date: 2018-01-30 15:34:07
On Tue, Jan 30, 2018 at 2:48 PM, Will Deacon [off-list ref] wrote:
quoted
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