linux-kernel - Re: [PATCH v4 0/3] of: overlay: load overlay symbols into live device tree

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]

Date:   Thu, 20 Jul 2017 09:47:47 -0500
From:   Rob Herring <robh+dt@...nel.org>
To:     Frank Rowand <frowand.list@...il.com>
Cc:     Pantelis Antoniou <pantelis.antoniou@...sulko.com>,
        Pantelis Antoniou <panto@...oniou-consulting.com>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v4 0/3] of: overlay: load overlay symbols into live device tree

On Wed, Jul 19, 2017 at 11:25 AM,  <frowand.list@...il.com> wrote:
> From: Frank Rowand <frank.rowand@...y.com>
>
> Symbols in a loaded overlay are not currently available to subsequently
> loaded overlays because the properties in the overlay's __symbols__
> node are not loaded into the live device tree.
>
> Patch 1 is unittests to test patches 2 and 3.
>
> Patch 2 fixes a problem discovered while developing patch 3.  If
> a node name in an overlay has a unit-address then the overlay
> code does not correctly match the node name against an existing
> node in the live tree.
>
> Patch 3 adds the properties in an overlay's __symbol__ node to
> the overlay changeset.
>
> changes since v3:
>   - fix make W=2 warnings in overlay_base.dts and overlay.dts
>   - update the test results in the patch comments to match the
>     .dts changes
>
> changes since v2:
>   - test on 4.13-rc1
>   - remove redundant comments from patch headers of patch 0 and patch 1
>   - guard make of overlay .dtb files with CONFIG_OF_OVERLAY
>   - guard unittest.c references to overlay .dtb files with
>     CONFIG_OF_OVERLAY
>
> changes since v1:
>   - patch 2: Use for_each_child_of_node() instead of open coding,
>     which allowed removing the new function child_by_full_name().
>
>
>
> Frank Rowand (3):
>   of: overlay: add overlay unittest data for node names and symbols
>   of: overlay: correctly apply overlay node with unit-address
>   of: overlay: add overlay symbols to live device tree

Series applied.

Rob