linux-kernel - Re: [GIT PULL] MFD for v4.13

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]

Date:   Fri, 7 Jul 2017 13:36:45 -0700
From:   Linus Torvalds <torvalds@...ux-foundation.org>
To:     Lee Jones <lee.jones@...aro.org>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [GIT PULL] MFD for v4.13

On Fri, Jul 7, 2017 at 2:50 AM, Lee Jones <lee.jones@...aro.org> wrote:
>
>  include/linux/mfd/madera/registers.h               | 8832 ++++++++++++++++++++

So I've pulled everything but this, because honestly, that file looks
like utter garbage.

Why are there all those _hundreds_ of odd defines for

  MADERA_WSEQ_SEQUENCE_xx

when it looks like you could just do one single one:

  // The sequence is one-based because somebody doesn't
  // know that indices start at 0. Thus the "-2".
  #define MADERA_WSEQ_SEQUENCE(x) (0x3000 + (x)*2 - 2)

and similar things go for for pretty much EVERY SINGLE LINE in that
8-thousand line piece of nasty horrible crud.

Being auto-generated doesn't really make this kind of thing any
better. In fact, it makes it worse, because those stupid hardcoded
names are often *harder* to use, because you cannot use a variable to
index into things (which you may often want).

So we have eight thousand lines of garbage that is

 (a) probably closer to 200x too many lines
 (b) less flexible than doing it right

Honestly, tell me why would I want to merge something monstrous like that?

                Linus