Re: [PATCH net-next v5 02/20] zinc: introduce minimal cryptography library

From: Jason A. Donenfeld
Date: Thu Sep 20 2018 - 23:21:32 EST


Hi Andrew,

On Fri, Sep 21, 2018 at 5:13 AM Andrew Lunn <andrew@xxxxxxx> 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