Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754179AbcKNPLF (ORCPT ); Mon, 14 Nov 2016 10:11:05 -0500 Received: from mail-wm0-f65.google.com ([74.125.82.65]:34188 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753363AbcKNPLA (ORCPT ); Mon, 14 Nov 2016 10:11:00 -0500 Reply-To: monstr@monstr.eu Subject: Re: [PATCH] ARM: zynq: Reserve correct amount of non-DMA RAM References: <20161031162617.10565-1-kyle.roeschley@ni.com> To: Nathan Rossi , Kyle Roeschley Cc: linux-arm , linux@armlinux.org.uk, Soren Brinkmann , LKML From: Michal Simek Message-ID: <8db45834-ddda-ea44-567c-71ddac67e1bf@monstr.eu> Date: Mon, 14 Nov 2016 16:10:22 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="UHiCf2U2WApE3a9XRh9a53e4u7V6BhANT" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2990 Lines: 85 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --UHiCf2U2WApE3a9XRh9a53e4u7V6BhANT Content-Type: multipart/mixed; boundary="7txnGt0SBVBPMne6WlqHV0G2guPOlcl2J"; protected-headers="v1" From: Michal Simek Reply-To: monstr@monstr.eu To: Nathan Rossi , Kyle Roeschley Cc: linux-arm , linux@armlinux.org.uk, Soren Brinkmann , LKML Message-ID: <8db45834-ddda-ea44-567c-71ddac67e1bf@monstr.eu> Subject: Re: [PATCH] ARM: zynq: Reserve correct amount of non-DMA RAM References: <20161031162617.10565-1-kyle.roeschley@ni.com> In-Reply-To: --7txnGt0SBVBPMne6WlqHV0G2guPOlcl2J Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 10.11.2016 10:57, Nathan Rossi wrote: > On 10 November 2016 at 19:33, Nathan Rossi wro= te: >> On 10 November 2016 at 18:41, Michal Simek wrote: >>> + Nathan >>> >>> 2016-10-31 17:26 GMT+01:00 Kyle Roeschley : >>>> >>>> On Zynq, we haven't been reserving the correct amount of DMA-incapab= le >>>> RAM to keep DMA away from it (per the Zynq TRM Section 4.1, it shoul= d be >>>> the first 512k). In older kernels, this was masked by the >>>> memblock_reserve call in arm_memblock_init(). Now, reserve the corre= ct >>>> amount excplicitly rather than relying on swapper_pg_dir, which is a= n >>>> address and not a size anyway. >>>> >>>> Fixes: 46f5b96 ("ARM: zynq: Reserve not DMAable space in front of th= e >>>> kernel") >>>> >>>> Signed-off-by: Kyle Roeschley >> >> Tested-by: Nathan Rossi >> >> For reference this causes problems with DEBUG_RODATA (which changed to= >=20 > Sorry typo -> s/causes/caused/, as in "... this [incorrect reserving > of the lower 512K] caused ...". >=20 > Regards, > Nathan >=20 >> 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 --7txnGt0SBVBPMne6WlqHV0G2guPOlcl2J-- --UHiCf2U2WApE3a9XRh9a53e4u7V6BhANT Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iEYEARECAAYFAlgp094ACgkQykllyylKDCEQQwCeO7oTdnq+eZloisRf0Vf4CkIq AzAAmweK8DWI0xfv3/NZwsHQUGF2gj5t =9oLW -----END PGP SIGNATURE----- --UHiCf2U2WApE3a9XRh9a53e4u7V6BhANT--