Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760944AbcLBNji (ORCPT ); Fri, 2 Dec 2016 08:39:38 -0500 Received: from mailapp01.imgtec.com ([195.59.15.196]:40654 "EHLO mailapp01.imgtec.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753274AbcLBNjg (ORCPT ); Fri, 2 Dec 2016 08:39:36 -0500 From: Matt Redfearn To: Ralf Baechle CC: , "Jason A . Donenfeld" , Thomas Gleixner , Matt Redfearn , Paolo Bonzini , Marcin Nowakowski , Chris Metcalf , Petr Mladek , , Andrea Gelmini , James Hogan , Paul Burton , Jiri Slaby , "Maciej W. Rozycki" , Aaron Tomlin , Andrew Morton Subject: [PATCH 0/5] MIPS: Add per-cpu IRQ stack Date: Fri, 2 Dec 2016 13:39:12 +0000 Message-ID: <1480685957-18809-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: 1751 Lines: 46 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