Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933124AbcLSOVf (ORCPT ); Mon, 19 Dec 2016 09:21:35 -0500 Received: from mailapp01.imgtec.com ([195.59.15.196]:18320 "EHLO mailapp01.imgtec.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756971AbcLSOVH (ORCPT ); Mon, 19 Dec 2016 09:21:07 -0500 From: Matt Redfearn To: Ralf Baechle CC: , "Jason A . Donenfeld" , Thomas Gleixner , Matt Redfearn , Paolo Bonzini , Marcin Nowakowski , Chris Metcalf , Petr Mladek , , Adam Buchbinder , James Hogan , Paul Burton , Jiri Slaby , "Maciej W. Rozycki" , Aaron Tomlin , Andrew Morton Subject: [PATCH v3 0/5] MIPS: Add per-cpu IRQ stack Date: Mon, 19 Dec 2016 14:20:55 +0000 Message-ID: <1482157260-18730-1-git-send-email-matt.redfearn@imgtec.com> X-Mailer: git-send-email 2.7.4 MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [10.150.130.83] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1960 Lines: 52 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 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. Changes in v3: Drop superfluous nop that would have been in delay slot with .set noreorder but is no longer required now that the code is .set reorder. Changes in v2: Drop .set reorder/noreorder when updating $28 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 | 7 ++++ 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, 122 insertions(+), 6 deletions(-) -- 2.7.4