Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754851AbcKJJFt (ORCPT ); Thu, 10 Nov 2016 04:05:49 -0500 Received: from Galois.linutronix.de ([146.0.238.70]:60791 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752760AbcKJJFr (ORCPT ); Thu, 10 Nov 2016 04:05:47 -0500 Date: Thu, 10 Nov 2016 10:03:11 +0100 (CET) From: Thomas Gleixner To: "Jason A. Donenfeld" cc: LKML , linux-mips@linux-mips.org, linux-mm@kvack.org, WireGuard mailing list , k@vodka.home.kg Subject: Re: Proposal: HAVE_SEPARATE_IRQ_STACK? In-Reply-To: Message-ID: References: User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1472 Lines: 42 On Thu, 10 Nov 2016, Jason A. Donenfeld wrote: > Hey Thomas, > > On Wed, Nov 9, 2016 at 10:40 PM, Thomas Gleixner wrote: > > That preempt_disable() prevents merily preemption as the name says, but it > > wont prevent softirq handlers from running on return from interrupt. So > > what's the point? > > Oh, interesting. Okay, then in that case the proposed define wouldn't > be useful for my purposes. If you want to go with that config, then you need local_bh_disable()/enable() to fend softirqs off, which disables also preemption. > What clever tricks do I have at my disposal, then? Make MIPS use interrupt stacks. > >> If not, do you have a better solution for me (which doesn't > >> involve using kmalloc or choosing a different crypto primitive)? > > > > What's wrong with using kmalloc? > > It's cumbersome and potentially slow. This is crypto code, where speed > matters a lot. Avoiding allocations is usually the lowest hanging > fruit among optimizations. To give you some idea, here's a somewhat > horrible solution using kmalloc I hacked together: [1]. I'm not to > happy with what it looks like, code-wise, and there's around a 16% > slowdown, which isn't nice either. Does the slowdown come from the kmalloc overhead or mostly from the less efficient code? If it's mainly kmalloc, then you can preallocate the buffer once for the kthread you're running in and be done with it. If it's the code, then bad luck. Thanks, tglx