Re: [PATCH v2 05/19] lib/stackdepot: use fixed-sized slots for stack records

Andrey Konovalov
2023-09-15T16:47:18Z

On Fri, Sep 15, 2023 at 10:56 AM Marco Elver <elver@google.com> wrote:
> > --- a/lib/Kconfig
> > +++ b/lib/Kconfig
> > @@ -708,13 +708,19 @@ config ARCH_STACKWALK
> > bool
> >
> > config STACKDEPOT
> > - bool
> > + bool "Stack depot: stack trace storage that avoids duplication"
> > select STACKTRACE
> >
> > config STACKDEPOT_ALWAYS_INIT
> > - bool
> > + bool "Always initialize stack depot during early boot"
> > select STACKDEPOT
>
> This makes both STACKDEPOT and STACKDEPOT_ALWAYS_INIT configurable by
> users: https://www.kernel.org/doc/html/next/kbuild/kconfig-language.html#menu-attributes
>
> Usually the way to add documentation for non-user-configurable options
> is to add text in the "help" section of the config.
>
> I think the change here is not what was intended.

Ah, didn't know about that. Will fix in v3. Thanks!