Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751993AbcLFWKG (ORCPT ); Tue, 6 Dec 2016 17:10:06 -0500 Received: from frisell.zx2c4.com ([192.95.5.64]:41509 "EHLO frisell.zx2c4.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751388AbcLFWKE (ORCPT ); Tue, 6 Dec 2016 17:10:04 -0500 MIME-Version: 1.0 In-Reply-To: <1480685957-18809-1-git-send-email-matt.redfearn@imgtec.com> References: <1480685957-18809-1-git-send-email-matt.redfearn@imgtec.com> From: "Jason A. Donenfeld" Date: Tue, 6 Dec 2016 23:09:28 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 0/5] MIPS: Add per-cpu IRQ stack To: Matt Redfearn Cc: Ralf Baechle , linux-mips@linux-mips.org, Thomas Gleixner , Paolo Bonzini , Marcin Nowakowski , Chris Metcalf , Petr Mladek , LKML , Andrea Gelmini , James Hogan , Paul Burton , Jiri Slaby , "Maciej W. Rozycki" , Aaron Tomlin , Andrew Morton Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2406 Lines: 62 Hi Matt, Thanks for submitting this. A happy OpenWRT/WireGuard user has reported to me that this patch set frees ~1300 bytes of stack on a small MIPS router. This kind of savings should allow me to reintroduce my crypto operations to be on the stack, rather than the conditional MIPS kmallocing, which will be a performance win. By the way, it looks like x86 and SPARC have separately allocated hard IRQ and soft IRQ stacks. ARM has only one for both, similar to this MIPS patchset. Jason On Fri, Dec 2, 2016 at 2:39 PM, Matt Redfearn wrote: > > This series adds a separate stack for each CPU wihin the system to use > when handling IRQs. Previously IRQs were handled on the kernel stack of > the current task. If that task was deep down a call stack at the point > of the interrupt, and handling the interrupt required a deep IRQ stack, > then there was a likelihood of stack overflow. Since the kernel stack > is in normal unmapped memory, overflowing it can lead to silent > corruption of other kernel data, with weird and wonderful results. > > Before this patch series, ftracing the maximum stack size of a v4.9-rc6 > kernel running on a Ci40 board gave: > 4996 > > And with this series: > 4084 > > Handling interrupts on a separate stack reduces the maximum kernel stack > usage in this configuration by ~900 bytes. > > Since do_IRQ is now invoked on a separate stack, we select > HAVE_IRQ_EXIT_ON_IRQ_STACK so that softirqs will also be executed on the > irq stack rather than attempting to switch with do_softirq_own_stack(). > > This series has been tested on MIPS Boston, Malta and SEAD3 platforms, > Pistachio on the Creator Ci40 board and Cavium Octeon III. > > > > Matt Redfearn (5): > MIPS: Introduce irq_stack > MIPS: Stack unwinding while on IRQ stack > MIPS: Only change $28 to thread_info if coming from user mode > MIPS: Switch to the irq_stack in interrupts > MIPS: Select HAVE_IRQ_EXIT_ON_IRQ_STACK > > arch/mips/Kconfig | 1 + > arch/mips/include/asm/irq.h | 12 ++++++ > arch/mips/include/asm/stackframe.h | 10 +++++ > arch/mips/kernel/asm-offsets.c | 1 + > arch/mips/kernel/genex.S | 81 +++++++++++++++++++++++++++++++++++--- > arch/mips/kernel/irq.c | 11 ++++++ > arch/mips/kernel/process.c | 15 ++++++- > 7 files changed, 125 insertions(+), 6 deletions(-) > > -- > 2.7.4 >