Re: [RFC 0/2] Add workaround for core wake-up on IPI for i.MX8MQ

From: Martin Kepplinger <martink@posteo.de>
To: Abel Vesa <abelvesa@gmail.com>, Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	Marc Zyngier <marc.zyngier@arm.com>,
	Lucas Stach <l.stach@pengutronix.de>, Bai Ping <ping.bai@nxp.com>,
	Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
	Leonard Crestez <leonard.crestez@nxp.com>
Cc: devicetree@vger.kernel.org, Carlo Caione <ccaione@baylibre.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [RFC 0/2] Add workaround for core wake-up on IPI for i.MX8MQ
Date: Wed, 30 Oct 2019 07:11:37 +0100	[thread overview]
Message-ID: <7d3a868a-768c-3cb1-c6d8-bf5fcd1ddd1c@posteo.de> (raw)
In-Reply-To: <d217a9d2-fc60-e057-6775-116542e39e8d@posteo.de>

On 23.06.19 13:47, Martin Kepplinger wrote:
> On 10.06.19 14:13, Abel Vesa wrote:
>> This is another alternative for the RFC:
>> https://lkml.org/lkml/2019/3/27/545
>>
>> This new workaround proposal is a little bit more hacky but more contained
>> since everything is done within the irq-imx-gpcv2 driver.
>>
>> Basically, it 'hijacks' the registered gic_raise_softirq __smp_cross_call
>> handler and registers instead a wrapper which calls in the 'hijacked' 
>> handler, after that calling into EL3 which will take care of the actual
>> wake up. This time, instead of expanding the PSCI ABI, we use a new vendor SIP.
>>
>> I also have the patches ready for TF-A but I'll hold on to them until I see if
>> this has a chance of getting in.
> 

Hi Abel,

Running this workaround doesn't seem to work anymore on 5.4-rcX. Linux
doesn't boot, with ATF unchanged (includes your workaround changes). I
can try to add more details to this...

Have you tested this for 5.4? Could you update this workaround? Please
let me know if I missed any earlier update on this (having a cpu-sleep
idle state).

thanks!

                              martin