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  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:   Wed, 6 Mar 2019 10:28:30 -0800
From:   Linus Torvalds <torvalds@...ux-foundation.org>
To:     Russell King - ARM Linux admin <linux@...linux.org.uk>
Cc:     Arnd Bergmann <arnd@...db.de>, soc@...nel.org,
        Linux ARM <linux-arm-kernel@...ts.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        arm-soc <arm@...nel.org>
Subject: Re: [GIT PULL 3/6] ARM: SoC driver updates for 5.1

On Wed, Mar 6, 2019 at 10:16 AM Russell King - ARM Linux admin
<linux@...linux.org.uk> wrote:
>
> Would it be worth splitting up mod_devicetable.h and having drivers
> include just the bus-specific device table header(s) that the driver
> requires?

That would certainly help rebuilds when it changes.

But maybe those changes are just not common enough to worry about, and
maybe it would result in more maintenance pain. Who knows?

I just wanted to mention it since it surprised me and I spent the few
minutes to figure out what the offending header file was, and see if
somebody feels motivated.

It does seem a bit pointless and wrong to have the core acpi.h header
file include this, and then cause files to be recompiled just because
some entirely unrelated device ID model changed.

So maybe it would indeed be better having each device type have its
own header file ("include/linux/acpi_devicetable.h"), and then for the
(few) cases that might want to handle _any_ type could include that
"mod_devicetable.h" that then just aggregates them?

But as mentioned, I'm not sure it's worth it, just throwing this issue
out to see what people think.

            Linus

Powered by blists - more mailing lists