linux-kernel - Re: [PATCH] ARM: zynq: Reserve correct amount of non-DMA RAM

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: <8db45834-ddda-ea44-567c-71ddac67e1bf@monstr.eu>
Date:   Mon, 14 Nov 2016 16:10:22 +0100
From:   Michal Simek <monstr@...str.eu>
To:     Nathan Rossi <nathan@...hanrossi.com>,
        Kyle Roeschley <kyle.roeschley@...com>
Cc:     linux-arm <linux-arm-kernel@...ts.infradead.org>,
        linux@...linux.org.uk,
        Soren Brinkmann <soren.brinkmann@...inx.com>,
        LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] ARM: zynq: Reserve correct amount of non-DMA RAM

On 10.11.2016 10:57, Nathan Rossi wrote:
> On 10 November 2016 at 19:33, Nathan Rossi <nathan@...hanrossi.com> wrote:
>> On 10 November 2016 at 18:41, Michal Simek <monstr@...str.eu> wrote:
>>> + Nathan
>>>
>>> 2016-10-31 17:26 GMT+01:00 Kyle Roeschley <kyle.roeschley@...com>:
>>>>
>>>> On Zynq, we haven't been reserving the correct amount of DMA-incapable
>>>> RAM to keep DMA away from it (per the Zynq TRM Section 4.1, it should be
>>>> the first 512k). In older kernels, this was masked by the
>>>> memblock_reserve call in arm_memblock_init(). Now, reserve the correct
>>>> amount excplicitly rather than relying on swapper_pg_dir, which is an
>>>> address and not a size anyway.
>>>>
>>>> Fixes: 46f5b96 ("ARM: zynq: Reserve not DMAable space in front of the
>>>> kernel")
>>>>
>>>> Signed-off-by: Kyle Roeschley <kyle.roeschley@...com>
>>
>> Tested-by: Nathan Rossi <nathan@...hanrossi.com>
>>
>> For reference this causes problems with DEBUG_RODATA (which changed to
> 
> Sorry typo -> s/causes/caused/, as in "... this [incorrect reserving
> of the lower 512K] caused ...".
> 
> Regards,
> Nathan
> 
>> default yes for CPU_V7 in v4.6) due to padding memory between
>> .head.text and .text, allowing memory below 0x80000 to be available
>> for allocation as non-reserved memory.
Applied to zynq/soc
https://github.com/Xilinx/linux-xlnx/commits/zynq/soc

Thanks,
Michal



Download attachment "signature.asc" of type "application/pgp-signature" (199 bytes)