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;
MIME-Version: 1.0
References: <CAK8P3a2jgQcLaDXX6eOTNrU0RJ2O625e75LBMy6v2ABP0cdoww@mail.gmail.com>
 <CAHk-=wgZSD3W2y6yczad2Am=EfHYyiPzTn3CfXxrriJf9i5W5w@mail.gmail.com> <YvKD4hkZ3erf54DG@linutronix.de>
In-Reply-To: <YvKD4hkZ3erf54DG@linutronix.de>
From:   Arnd Bergmann <arnd@kernel.org>
Date:   Tue, 9 Aug 2022 23:42:29 +0200
Message-ID: <CAK8P3a1pvWK0GUikVG7H=AByOfS4x=JjXezorjF66Hv8FH8uBA@mail.gmail.com>
Subject: Re: [PATCH] asm-generic: Conditionally enable do_softirq_own_stack()
 via Kconfig.
To:     Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc:     Linus Torvalds <torvalds@linux-foundation.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        linux-arch <linux-arch@vger.kernel.org>,
        Linux ARM <linux-arm-kernel@lists.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Tue, Aug 9, 2022 at 5:57 PM Sebastian Andrzej Siewior
<bigeasy@linutronix.de> wrote:
>
> Remove the CONFIG_PREEMPT_RT symbol from the ifdef around
> do_softirq_own_stack() and move it to Kconfig instead.
>
> Enable softirq stacks based on SOFTIRQ_ON_OWN_STACK which depends on
> HAVE_SOFTIRQ_ON_OWN_STACK and its default value is set to !PREEMPT_RT.
> This ensures that softirq stacks are not used on PREEMPT_RT and avoids
> a 'select' statement on an option which has a 'depends' statement.
>
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> ---
>  arch/Kconfig                          | 4 ++++
>  arch/arm/kernel/irq.c                 | 2 +-
>  arch/parisc/kernel/irq.c              | 2 +-
>  arch/powerpc/kernel/irq.c             | 4 ++--
>  arch/s390/include/asm/softirq_stack.h | 2 +-
>  arch/sh/kernel/irq.c                  | 2 +-
>  arch/sparc/kernel/irq_64.c            | 2 +-
>  arch/x86/include/asm/irq_stack.h      | 2 +-
>  arch/x86/kernel/irq_32.c              | 2 +-
>  include/asm-generic/softirq_stack.h   | 2 +-

Thanks for the patch, I assume Linus will want to pick this up himself.
Let me know if I should give it a spin in the asm-generic tree first for
additional build testing.

Acked-by: Arnd Bergmann <arnd@arndb.de>