Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 27 Apr 2023 11:06:57 +0100
From:   Will Deacon <will@kernel.org>
To:     "Bouska, Zdenek" <zdenek.bouska@siemens.com>
Cc:     Thomas Gleixner <tglx@linutronix.de>,
        Catalin Marinas <catalin.marinas@arm.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Kiszka, Jan" <jan.kiszka@siemens.com>,
        "linux-rt-users@vger.kernel.org" <linux-rt-users@vger.kernel.org>,
        Nishanth Menon <nm@ti.com>, Puranjay Mohan <p-mohan@ti.com>
Subject: Re: Unfair qspinlocks on ARM64 without LSE atomics => 3ms delay in
 interrupt handling
Message-ID: <20230427100656.GA10855@willie-the-truck>
References: <AS1PR10MB567534190B05A4493674173BEB659@AS1PR10MB5675.EURPRD10.PROD.OUTLOOK.COM>
 <87pm7qxrg6.ffs@tglx>
 <AS1PR10MB5675F26ADC112F008B52F195EB6A9@AS1PR10MB5675.EURPRD10.PROD.OUTLOOK.COM>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <AS1PR10MB5675F26ADC112F008B52F195EB6A9@AS1PR10MB5675.EURPRD10.PROD.OUTLOOK.COM>
User-Agent: Mutt/1.10.1 (2018-07-13)
Precedence: bulk

On Thu, Apr 27, 2023 at 09:38:59AM +0000, Bouska, Zdenek wrote:
> > Why is this interrupt handling specific? Just because it's the place
> > where you observed it?
> Yes.
> 
> > So if that helps, then this needs to be addressed globaly and not with
> > some crude hack in the interrupt handling code.
> I just wrote, what helps for me. I didn't mean it as a proposal for merge.
> Sorry for confusion.
> 
> I tried using Will's cpu_relax() implementation [1] everywhere but I was not 
> successful with that yet. ARM64's VDSO makes it complicating and even
> if I left original cpu_relax() just in VDSO, Linux did not boot for me.

It definitely seemed to work when I posted it all those years ago, but I'm
not surprised if it needs some TLC to revive it on more recent kernels.

Will