linux-kernel - Re: [PATCH] pcmcia: avoid defines prefixed with CONFIG

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: <CAKXUXMwbkDgwMvB-CDyDbL7CS1eSn4P=87BqfWCyvCvP+SkQ8A@mail.gmail.com>
Date:   Mon, 16 Jan 2023 08:48:41 +0100
From:   Lukas Bulwahn <lukas.bulwahn@...il.com>
To:     peter@...jl.ca
Cc:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux@...inikbrodowski.net" <linux@...inikbrodowski.net>,
        "kernel-janitors@...r.kernel.org" <kernel-janitors@...r.kernel.org>
Subject: Re: [PATCH] pcmcia: avoid defines prefixed with CONFIG

On Fri, Jan 13, 2023 at 4:30 PM Peter Lafreniere <peter@...jl.ca> wrote:
>
> Macros prefixed with "CONFIG_" should be relegated to Kconfig switches,
> so we should change the config state flags to avoid conflicts.
>
> This change affects only code readability, not function.
>

Peter,

If you are interested in doing more clean-up work in this area, please
let me know. I have a longer patch series of various changes that have
been partially submitted and have not been included yet and some
changes I have not sent yet, as they are really minor spelling fixes
in comments. Further, I have a list of known false positives from the
./scripts/checkkconfigsymbols.py warnings, which you can use to filter
out some of the warnings, and some experience on this script to find
what is more relevant to address first and what could go to the
low-priority TODO list (that just might disappear, as old code is
completely deleted).

I can share all that with you if you just give me a ping.

Best regards,

Lukas