linux-kernel - Aw: Re: Re: [RFC] arm64: dts: use size of reserved partition for bl2

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: <trinity-cdcc1156-f62c-40ac-b064-c2592e29e9eb-1680533693486@3c-app-gmx-bap35>
Date:   Mon, 3 Apr 2023 16:54:53 +0200
From:   Frank Wunderlich <frank-w@...lic-files.de>
To:     AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@...labora.com>
Cc:     Frank Wunderlich <linux@...web.de>,
        linux-mediatek@...ts.infradead.org,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Matthias Brugger <matthias.bgg@...il.com>,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org
Subject: Aw: Re:  Re: [RFC] arm64: dts: use size of reserved partition for
 bl2

> Gesendet: Montag, 03. April 2023 um 16:04 Uhr
> Von: "AngeloGioacchino Del Regno" <angelogioacchino.delregno@...labora.com>
> An: "Frank Wunderlich" <frank-w@...lic-files.de>
> Cc: "Frank Wunderlich" <linux@...web.de>, linux-mediatek@...ts.infradead.org, "Rob Herring" <robh+dt@...nel.org>, "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@...aro.org>, "Matthias Brugger" <matthias.bgg@...il.com>, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org
> Betreff: Re: Aw: Re: [RFC] arm64: dts: use size of reserved partition for bl2
>
> Il 03/04/23 15:56, Frank Wunderlich ha scritto:
> >> Gesendet: Montag, 03. April 2023 um 14:43 Uhr
> >> Von: "AngeloGioacchino Del Regno" <angelogioacchino.delregno@...labora.com>
> >> Il 03/04/23 12:58, Frank Wunderlich ha scritto:
> >>> From: Frank Wunderlich <frank-w@...lic-files.de>
> >>>
> >>> To store uncompressed bl2 more space is required than partition is
> >>> actually defined.
> >>>
> >>> Signed-off-by: Frank Wunderlich <frank-w@...lic-files.de>
> >>
> >> If this doesn't get changed anymore, I'm fine with it... but a question arises:
> >> did you send patches to add your BPI-r3 board(s) to upstream u-boot?
> >
> > currently i use the rfb dts for r3 in uboot: arch/arm/dts/mt7986a-sd-rfb.dts
> >
> > this file in upstream uboot has no partitions defined
> >
> > https://source.denx.de/u-boot/u-boot/-/blob/master/arch/arm/dts/mt7986a-sd-rfb.dts#L144
> >
> > but i added them there too and i wrote content by offsets to main device (not to partitions).
> >
> > https://github.com/frank-w/u-boot/blob/2023-04-bpi/arch/arm/dts/mt7986a-sd-rfb.dts#L154
> >
> > so yes basicly it needs to be send there too, maybe as additional dts.
> >
>
> I strongly encourage you to send this change to u-boot as well as changing the
> kernel devicetree... and when you do, you can add a reference to the list and/or
> commit @ u-boot on the commit that you'll send here for the kernel... but there's
> another question...

imho uboot needs an own dts for the board (copy from rfb) for this because of another change
(tick_dly) and i have no rfb for testing.

> ...what happens if you run the kernel on a device with an older partition layout?
> Does anything break?
> I don't think that anything does, since this is read-only anyway, but just to be
> sure... :-)

i have no information about the second partition from the old layout...it was only named reserved.

as openwrt also uses 0x40000 it should be save to use this value here.

if the bl2 does not fit into the bl2-partition (cut off), board simply does not boot,
thats why i want to increase it now. my current bl2 is 197K for nor and i had this problem.

> Besides, you have to fix the commit description, but you knew that already.

sorry missed the mt7986: part in title, or did you mean another part?

i put the additional information below --- to not be visible when patch will be applied.

> P.S.: When you post a RFC, giving it the right commit title helps sometimes! :-P

yes, missed the mt7986-part here, just wanted to hear comments first if thisis the right way