linux-kernel - Re: [PATCH v5] ARM: boot: Obtain start of physical memory from DTB

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]

Message-ID: <5419f6b6-2d75-763f-abe1-4c6a2c67dc3d@gmail.com>
Date:   Tue, 21 Apr 2020 18:31:28 +0300
From:   Dmitry Osipenko <digetx@...il.com>
To:     Ard Biesheuvel <ardb@...nel.org>,
        Geert Uytterhoeven <geert+renesas@...der.be>
Cc:     Russell King <linux@...linux.org.uk>,
        Nicolas Pitre <nico@...xnic.net>,
        Arnd Bergmann <arnd@...db.de>,
        Eric Miao <eric.miao@...dia.com>,
        Uwe Kleine-König <u.kleine-koenig@...gutronix.de>,
        Marek Szyprowski <m.szyprowski@...sung.com>,
        Chris Brandt <chris.brandt@...esas.com>,
        Linux ARM <linux-arm-kernel@...ts.infradead.org>,
        Linux-Renesas <linux-renesas-soc@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v5] ARM: boot: Obtain start of physical memory from DTB

21.04.2020 18:19, Ard Biesheuvel пишет:
> On Wed, 15 Apr 2020 at 17:34, Geert Uytterhoeven
> <geert+renesas@...der.be> wrote:
>>
>> Currently, the start address of physical memory is obtained by masking
>> the program counter with a fixed mask of 0xf8000000.  This mask value
>> was chosen as a balance between the requirements of different platforms.
>> However, this does require that the start address of physical memory is
>> a multiple of 128 MiB, precluding booting Linux on platforms where this
>> requirement is not fulfilled.
>>
>> Fix this limitation by obtaining the start address from the DTB instead,
>> if available (either explicitly passed, or appended to the kernel).
>> Fall back to the traditional method when needed.
>>
>> This allows to boot Linux on r7s9210/rza2mevb using the 64 MiB of SDRAM
>> on the RZA2MEVB sub board, which is located at 0x0C000000 (CS3 space),
>> i.e. not at a multiple of 128 MiB.
>>
>> Suggested-by: Nicolas Pitre <nico@...xnic.net>
>> Signed-off-by: Geert Uytterhoeven <geert+renesas@...der.be>
>> Reviewed-by: Nicolas Pitre <nico@...xnic.net>
>> Reviewed-by: Ard Biesheuvel <ardb@...nel.org>
>> Tested-by: Marek Szyprowski <m.szyprowski@...sung.com>
>> Tested-by: Dmitry Osipenko <digetx@...il.com>
> 
> This is ready to go into the patch system, no?
> 
> The sooner Russell picks it up, the sooner I can respin my patches
> that go on top.
> 
> Thanks,

The v5 works for me on NVIDIA Tegra, just in case.