Re: [PATCH 2/2] irqchip: loongson-*: Fix COMPILE_TEST

[Marc Zyngier]

On Sat, 30 May 2020 13:11:13 +0100,
Jiaxun Yang <jiaxun.yang@xxxxxxxxxxx> 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@xxxxxxxxx>
> Signed-off-by: Jiaxun Yang <jiaxun.yang@xxxxxxxxxxx>
> ---
>  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.