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:   Sat, 27 Aug 2022 16:13:00 +0100
Message-ID: <87wnatra83.wl-maz@kernel.org>
From:   Marc Zyngier <maz@kernel.org>
To:     Puyou Lu <puyou.lu@gmail.com>
Cc:     Thomas Gleixner <tglx@linutronix.de>,
        Robert Richter <rrichter@cavium.com>,
        Catalin Marinas <catalin.marinas@arm.com>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] irqchip/gic-v3: do runtime cpu cap check only when necessary
In-Reply-To: <20220827051328.GA18042@lu-N56VJ>
References: <20220827051328.GA18042@lu-N56VJ>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue)
 FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/27.1
 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO)
MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue")
Content-Type: text/plain; charset=US-ASCII
Precedence: bulk

On Sat, 27 Aug 2022 06:19:27 +0100,
Puyou Lu <puyou.lu@gmail.com> wrote:
> 
> Now cpu cap check is done every exception happens on every arm64 platform,
> but this check is necessary on just few of then, so we can drop this
> check at compile time on others. This can decrease exception handle time
> on most cases.
> 
> Fixes: 6d4e11c5e2e8 ("irqchip/gicv3: Workaround for Cavium ThunderX erratum 23154")
> Signed-off-by: Puyou Lu <puyou.lu@gmail.com>
> ---
>  drivers/irqchip/irq-gic-v3.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c
> index 262658fd5f9e..3f08c2ef1251 100644
> --- a/drivers/irqchip/irq-gic-v3.c
> +++ b/drivers/irqchip/irq-gic-v3.c
> @@ -237,9 +237,11 @@ static void gic_redist_wait_for_rwp(void)
>  
>  static u64 __maybe_unused gic_read_iar(void)
>  {
> +#ifdef CONFIG_CAVIUM_ERRATUM_23154
>  	if (cpus_have_const_cap(ARM64_WORKAROUND_CAVIUM_23154))
>  		return gic_read_iar_cavium_thunderx();
>  	else
> +#endif
>  		return gic_read_iar_common();
>  }
>  #endif

You realise that cpus_have_const_cap() results purely in a couple of
branches once the caps have been finalised, right?

Please provide data showing that it actually "can decrease exception
handle time on most cases", because I'm pretty sure you cannot measure
the difference in any meaningful way.

	M.

-- 
Without deviation from the norm, progress is not possible.