Messages in this thread |  | | From | Thomas Gleixner <> | Subject | Re: [patch 11/12] softirq: Allow inlining do_softirq_own_stack() | Date | Fri, 05 Feb 2021 12:38:33 +0100 |
| |
On Fri, Feb 05 2021 at 18:14, Lai Jiangshan wrote: > On Fri, Feb 5, 2021 at 10:04 AM Thomas Gleixner <tglx@linutronix.de> wrote: >> static inline void do_softirq_own_stack(void) >> { > > Hello > > This patch and the next patch have three "#if[n]def" with > __ARCH_HAS_DO_SOFTIRQ_INLINE and this one is nested in > __ARCH_HAS_DO_SOFTIRQ. > > I wonder if we can use __ARCH_HAS_DO_SOFTIRQ only. > > For example, we can move "void do_softirq_own_stack(void);" to around > the code where __ARCH_HAS_DO_SOFTIRQ are defined in very ARCHs. > (And for x86, do_softirq_own_stack() is a macro instead of function > declaration as next patch shows)
We can do that as well. No strong preference.
Thanks,
tglx
|  |