Hi Imran, On Tue, Aug 31, 2021 at 8:28 AM Imran Khan <imran.f.khan@xxxxxxxxxx> wrote: > So far CONFIG_STACKDEPOT option was being selected by > features that need STACKDEPOT support for their operations, > for example KASAN. > Since next patch makes use of STACKDEPOT to store user tracking > information for slub debugger and since user tracking info may > or may not store stack trace for allocating and freeing contexts, > make STACKDEPOT explicitly configurable. > > Signed-off-by: Imran Khan <imran.f.khan@xxxxxxxxxx> Thanks for your patch! > --- > lib/Kconfig | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/lib/Kconfig b/lib/Kconfig > index 6a6ae5312fa0..7e4b54f48af7 100644 > --- a/lib/Kconfig > +++ b/lib/Kconfig > @@ -665,8 +665,9 @@ config ARCH_STACKWALK > bool > > config STACKDEPOT > - bool > + def_bool n Why this change? "n" is the default anyway. > select STACKTRACE > + prompt "Enable stackdepot support" > > config STACK_HASH_ORDER > int "stack depot hash size (12 => 4KB, 20 => 1024KB)" Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds