linux-kernel - Re: [PATCH] kconfig: Document the 'symbol' struct

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAFkk2KTp4njaTw5xAPC7V1cArui=7jqu3ORMTCzaACz6dngnyA@mail.gmail.com>
Date:   Thu, 5 Oct 2017 05:14:41 +0200
From:   Ulf Magnusson <ulfalizer@...il.com>
To:     yann.morin.1998@...e.fr, linux-kbuild@...r.kernel.org
Cc:     linux-kernel@...r.kernel.org, Ulf Magnusson <ulfalizer@...il.com>,
        Masahiro Yamada <yamada.masahiro@...ionext.com>,
        Arnaud Lacombe <lacombar@...il.com>
Subject: Re: [PATCH] kconfig: Document the 'symbol' struct

On Wed, Oct 4, 2017 at 7:48 AM, Ulf Magnusson <ulfalizer@...il.com> wrote:
> Visibility and choices in particular might be a bit tricky to figure
> out.
>
> Also fix existing comment to point out that P_MENU is also used for
> menus.
>
> Signed-off-by: Ulf Magnusson <ulfalizer@...il.com>
> ---
>  scripts/kconfig/expr.h | 45 ++++++++++++++++++++++++++++++++++++++++++++-
>  1 file changed, 44 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/kconfig/expr.h b/scripts/kconfig/expr.h
> index a73f762..6344067 100644
> --- a/scripts/kconfig/expr.h
> +++ b/scripts/kconfig/expr.h
> @@ -74,17 +74,60 @@ enum {
>         S_DEF_COUNT
>  };
>
> +/*
> + * Represents a configuration symbol.
> + *
> + * Choices are represented as a special kind of symbol and have the
> + * SYMBOL_CHOICE bit set in 'flags'.
> + */
>  struct symbol {
> +       /* The next symbol in the same bucket in the symbol hash table */
>         struct symbol *next;
> +
> +       /* The name of the symbol, e.g. "FOO" for 'config FOO' */
>         char *name;
> +
> +       /* S_BOOLEAN, S_TRISTATE, ... */
>         enum symbol_type type;
> +
> +       /*
> +        * The calculated value of the symbol. The SYMBOL_VALID bit is set in
> +        * 'flags' when this is up to date. Note that this value might differ
> +        * from the user value set in e.g. a .config file, due to visibility.
> +        */
>         struct symbol_value curr;
> +
> +       /*
> +        * Values for the symbol provided from outside. def[S_DEF_USER] holds
> +        * the .config value.
> +        */
>         struct symbol_value def[S_DEF_COUNT];
> +
> +       /*
> +        * An upper bound on the tristate value the user can set for the symbol
> +        * if it is a boolean or tristate. Calculated from prompt dependencies,
> +        * which also inherit dependencies from enclosing menus, choices, and
> +        * ifs. If 'n', the user value will be ignored.
> +        *
> +        * Symbols lacking prompts always have visibility 'n'.
> +        */
>         tristate visible;
> +
> +       /* SYMBOL_* flags */
>         int flags;
> +
> +       /* List of properties. See prop_type. */
>         struct property *prop;
> +
> +       /* Dependencies from enclosing menus, choices, and ifs */
>         struct expr_value dir_dep;
> +
> +       /* Reverse dependencies through being selected by other symbols */
>         struct expr_value rev_dep;
> +
> +       /*
> +        * "Weak" reverse dependencies through being implied by other symbols
> +        */
>         struct expr_value implied;
>  };
>
> @@ -133,7 +176,7 @@ enum prop_type {
>         P_UNKNOWN,
>         P_PROMPT,   /* prompt "foo prompt" or "BAZ Value" */
>         P_COMMENT,  /* text associated with a comment */
> -       P_MENU,     /* prompt associated with a menuconfig option */
> +       P_MENU,     /* prompt associated with a menu or menuconfig symbol */
>         P_DEFAULT,  /* default y */
>         P_CHOICE,   /* choice value */
>         P_SELECT,   /* select BAR */
> --
> 2.7.4
>

Adding some CCs just because I forgot. No panic.

Cheers,
Ulf