On 11/04/16 17:26, Laurent Pinchart wrote:
> On Friday 19 Feb 2016 11:59:40 Marc Zyngier wrote:
>> On 19/02/16 09:18, Linus Walleij wrote:
>>> Top-quoting so everyone on the new To:-line gets the context.
>>>
>>> I definately need an indication from an irqchip maintainer like tglx or
>>> Marc Z before I merge this. Also, as in reply to the previous letter,
>>> coordinate efforts with Jon Hunters similar problem space.
>>
>> Seems pretty straightforward to me.
>>
>> Acked-by: Marc Zyngier <[email protected]>
>
> Too straightforward to be correct :-/
>
> [ 6.232681] BUG: sleeping function called from invalid context at
> /home/laurent/src/iob/renesas/linux/drivers/base/power/runtime.c:955
> [ 6.244795] in_atomic(): 1, irqs_disabled(): 128, pid: 658, name: udevd
> [ 6.251429] CPU: 3 PID: 658 Comm: udevd Tainted: P
> 4.6.0-rc3 #756
> [ 6.258844] Hardware name: Generic R8A7790 (Flattened Device Tree)
[...]
Ah! That will teach me a lesson.
> The .irq_request_resources() handler is called with a spinlock held, it thus
> can't call the synchronous version of the PM runtime functions.
OK, so we're back to square one. Is that just a matter of calling the
non-synchronous version? My hunch is that it is not that simple...
Geert?
M.
--
Jazz is not dead. It just smells funny...
