Re: [PATCH v2] mtd: avoid stack overflow in MTD CFI code

From: Arnd Bergmann
Date: Fri Mar 04 2016 - 18:26:20 EST


On Friday 04 March 2016 13:21:59 Brian Norris wrote:
>
> Looking a little closer at this... why do we need the changes to
> include/linux/mtd/map.h again? It should be fine to leave these
> definitions as-is, right? They don't contribute to the large stack
> usage, do they?
>
> Maybe I'm just missing something obvious, so please do enlighten
>

It's been a while since I created the patch, and the originally
failing configuration currently doesn't produce this (probably
because something else changed). I remember that it was something
rather subtle, but don't exactly remember what happened.

I've reverted the patch now, trying to reproduce it on my
randconfig setup, but I might not be able to get back to you
in the next week while I'm traveling.

Arnd