From: "Jason A. Donenfeld" Subject: Re: [PATCH net-next v5 02/20] zinc: introduce minimal cryptography library Date: Fri, 21 Sep 2018 05:16:11 +0200 Message-ID: References: <20180918161646.19105-1-Jason@zx2c4.com> <20180918161646.19105-3-Jason@zx2c4.com> <20180921031255.GB11109@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: Arnd Bergmann , Ard Biesheuvel , Eric Biggers , LKML , Netdev , Linux Crypto Mailing List , David Miller , Greg Kroah-Hartman , Samuel Neves , Andrew Lutomirski , Jean-Philippe Aumasson To: Andrew Lunn Return-path: In-Reply-To: <20180921031255.GB11109@lunn.ch> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-crypto.vger.kernel.org Hi Andrew, On Fri, Sep 21, 2018 at 5:13 AM Andrew Lunn wrote: > Do you any stack usage information? > > A VPN can be at the bottom of some deep stack calls. Swap on NFS over > the VPN? If you have one frame of 1K, you might be O.K. But if you > have a few of these, i can see there might be issues of overflowing > the stack. It's pretty small, and most of the work is punted onto workqueues pretty early, so the depth isn't as much of an issue as it would be if I did everything from ndo_start_xmit. I've done some stack tracking using the debug option for that, and it's generally far from the deepest thing running. Jason