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: <CAHk-=wgiU+frgquJYSjkbQt+sQAshRrbg=9qnf7u58VKcjGgFA@mail.gmail.com>
Date:   Mon, 28 Aug 2023 14:32:50 -0700
From:   Linus Torvalds <torvalds@...ux-foundation.org>
To:     Thomas Gleixner <tglx@...utronix.de>
Cc:     linux-kernel@...r.kernel.org, x86@...nel.org
Subject: Re: [GIT pull] core/entry for v6.6-rc1

On Mon, 28 Aug 2023 at 14:07, Linus Torvalds
<torvalds@...ux-foundation.org> wrote:
>
> Heh. Lovely. I wonder if we have other cases of #ifdef's that just
> aren't #define'd anywhere any more.
>
> But I'm too lazy and/or incompetent to write up some trivial script to check.

I shamed myself into writing two one-liner scripts to create a list of
identifiers we do 'ifdef' on, and another list of identifiers we
#define somewhere.

And then just run 'comm -13' on the two.

And it turns out to be useless, because even after I filter out our
config options (which get #define'd separately), we end up having a
ton of identifiers that we just expect to be set from outside,. or are
just random noise in drivers (some of it more random than others: the
spca501.c tests for ALTER_GAMA in addition to ALTER_GAMMA, which I
assume is just a typo).

Several of them seem to be quite reasonable (ie deflate has
UNALIGNED_OK, expecting people to use -DUNALIGNED_OK from the build),
but it does mean that at least my trivial "let's see what symbols we
test without ever defining" idea was just simplistic garbage.

As so many of my ideas are...

             Linus

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ