Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Sat, 30 May 2020 14:09:45 +0100
Message-ID: <87pnalzi06.wl-maz@kernel.org>
From:   Marc Zyngier <maz@kernel.org>
To:     Jiaxun Yang <jiaxun.yang@flygoat.com>
Cc:     kbuild test robot <lkp@intel.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Jason Cooper <jason@lakedaemon.net>,
        Huacai Chen <chenhc@lemote.com>, linux-kernel@vger.kernel.org,
        linux-mips@vger.kernel.org
Subject: Re: [PATCH 2/2] irqchip: loongson-*: Fix COMPILE_TEST
In-Reply-To: <20200530121113.1797678-3-jiaxun.yang@flygoat.com>
References: <20200530121113.1797678-1-jiaxun.yang@flygoat.com>
        <20200530121113.1797678-3-jiaxun.yang@flygoat.com>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue)
 FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 EasyPG/1.0.0 Emacs/26
 (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
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Sat, 30 May 2020 13:11:13 +0100,
Jiaxun Yang <jiaxun.yang@flygoat.com> wrote:
> 
> spurious_interrupt helper only exists on MIPS and x86,
> so define a dummy function on other architectures to fix
> COMPILE_TEST.
> 
> Reported-by: kbuild test robot <lkp@intel.com>
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> ---
>  drivers/irqchip/irq-loongson-htpic.c   | 4 ++++
>  drivers/irqchip/irq-loongson-htvec.c   | 4 ++++
>  drivers/irqchip/irq-loongson-liointc.c | 4 ++++
>  3 files changed, 12 insertions(+)
> 
> diff --git a/drivers/irqchip/irq-loongson-htpic.c b/drivers/irqchip/irq-loongson-htpic.c
> index dd018c22ea83..7f38fdb2cb43 100644
> --- a/drivers/irqchip/irq-loongson-htpic.c
> +++ b/drivers/irqchip/irq-loongson-htpic.c
> @@ -26,6 +26,10 @@ struct loongson_htpic {
>  
>  static struct loongson_htpic *htpic;
>  
> +#if !defined(CONFIG_MIPS) && !defined(CONFIG_X86)
> +static inline void spurious_interrupt(void) { }
> +#endif

No, that's really horrible, and it doesn't scale. Dropping the
COMPILE_TEST dependency is cleaner, and we should look into having a
common framework across architectures to account for spurious
interrupts.

Not to mention that the HTPIC is not compiled with COMPILE_TEST anyway.

> +
>  static void htpic_irq_dispatch(struct irq_desc *desc)
>  {
>  	struct loongson_htpic *priv = irq_desc_get_handler_data(desc);
> diff --git a/drivers/irqchip/irq-loongson-htvec.c b/drivers/irqchip/irq-loongson-htvec.c
> index 1ece9337c78d..03c3973d7525 100644
> --- a/drivers/irqchip/irq-loongson-htvec.c
> +++ b/drivers/irqchip/irq-loongson-htvec.c
> @@ -33,6 +33,10 @@ struct htvec {
>  	raw_spinlock_t		htvec_lock;
>  };
>  
> +#if !defined(CONFIG_MIPS) && !defined(CONFIG_X86)
> +static inline void spurious_interrupt(void) { }
> +#endif

I've already addressed this by dropping COMPILE_TEST.

> +
>  static void htvec_irq_dispatch(struct irq_desc *desc)
>  {
>  	int i;
> diff --git a/drivers/irqchip/irq-loongson-liointc.c b/drivers/irqchip/irq-loongson-liointc.c
> index 63b61474a0cc..e31744e2d800 100644
> --- a/drivers/irqchip/irq-loongson-liointc.c
> +++ b/drivers/irqchip/irq-loongson-liointc.c
> @@ -46,6 +46,10 @@ struct liointc_priv {
>  	bool				has_lpc_irq_errata;
>  };
>  
> +#if !defined(CONFIG_MIPS) && !defined(CONFIG_X86)
> +static inline void spurious_interrupt(void) { }
> +#endif

Neither is this one.

	M.

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