linux-kernel - Re: [PATCH 1/1] kbuild: Add DT schema check also when applying DT overlays

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] [day] [month] [year] [list]

Message-ID: <CAL_JsqJp5aHJiibQ6=dcBPKQ8k=sK2Nbgter7Y7hczJP1AO0yg@mail.gmail.com>
Date: Mon, 26 Feb 2024 22:13:16 -0600
From: Rob Herring <robh+dt@...nel.org>
To: Masahiro Yamada <masahiroy@...nel.org>
Cc: Alexander Stein <alexander.stein@...lbox.org>, DTML <devicetree@...r.kernel.org>, 
	Nathan Chancellor <nathan@...nel.org>, Nicolas Schier <nicolas@...sle.eu>, linux-kbuild@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/1] kbuild: Add DT schema check also when applying DT overlays

On Mon, Feb 26, 2024 at 1:03 AM Masahiro Yamada <masahiroy@...nel.org> wrote:
>
> + Rob Herring, DT List
>
>
> On Mon, Feb 26, 2024 at 12:12 AM Alexander Stein
> <alexander.stein@...lbox.org> wrote:
> >
> > Add a similar mechanism as for regular DT files to call the DT checker
> > after applying DT overlays to base DT files.

I suspect this needs more work than just enabling checking. Otherwise,
I would have done just that. Depending on the split with an overlay,
we could see strange results.

The requirement in the kernel is overlays be applied to a base tree at
build time (though that is not checked ATM. When we check that result,
we're checking the overlay anyways. Obviously, that's a bit harder to
trace the source of errors, but it is otherwise redundant to check
overlays in addition.

Rob