From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Cyrus-Session-Id: sloti22d1t05-3899936-1516717086-2-15806586801098069289 X-Sieve: CMU Sieve 3.0 X-Spam-known-sender: no X-Spam-score: 0.0 X-Spam-hits: BAYES_00 -1.9, ME_NOAUTH 0.01, RCVD_IN_DNSWL_HI -5, T_RP_MATCHES_RCVD -0.01, LANGUAGES en, BAYES_USED global, SA_VERSION 3.4.0 X-Spam-source: IP='209.132.180.67', Host='vger.kernel.org', Country='US', FromHeader='org', MailFrom='org' X-Spam-charsets: plain='utf-8' X-Attached: signature.asc X-Resolved-to: greg@kroah.com X-Delivered-to: greg@kroah.com X-Mail-from: stable-owner@vger.kernel.org ARC-Seal: i=1; a=rsa-sha256; cv=none; d=messagingengine.com; s=arctest; t=1516717085; b=DUpR9sLDR2FlmzXUj58Kjfk7Rye7BKoIfVfjw6UdTRMVoFh l/R5vHD5K14ieA6kWp8seIFkgavmA4NMgQuefA+KMqvjMiUYK5P0N1dElfAkR/Tn 1sV0Fu7PBW9mCJJzNi6KlqQ/4fXrdv4vDLZsK2ML8+11PxoT937RZECCH9lOfa8z PXjgKGXdjMpL0pis0V8ppJblIQDP1CRhBuKf1AJjpJABdlV+M0VtUwTZUMPQsfH3 0mZGOPfyFDocFqAncr+8Eo5yty6qS3h6d8MXtHtTUIAxNWFWpVH68Padx3kLmqJL qzhiixA1PSBMQE2ZhJY+EMd0YkjKkf/3norGxlw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=date:from:to:cc:subject:message-id :references:mime-version:content-type:in-reply-to:sender :list-id; s=arctest; t=1516717085; bh=BV8ObNax3FzU30i+tmGBNpRWai FMLTiv3GDJP1vUhik=; b=Ij6AaJ5Dzmv+EjQAhV4FveQdHn3vG6QgHWNrlW9y+s Ke864Wj5o1GDB43f5sap1FfNnKN6BNd3sOp7AVtOitoOaFOEkm0hyYEjAm1IJd2Y YRawDEYl4NgImT6RBLDh3/bSszduD6xU4aC7lH8xVDC/XWSlA2GLHtxiLrq/J7N9 LM8Lx6IBfRuH7VZhkuRrvWL6Qr5H8+zBR25Af37JfTJ2dK3I3tiWdkP79zl8FbYe Lpz/M5SwCHwR9tpN4f1SLtpYj2+QBIU2AbXCbw0Lphv6yiKD6GxBvCrWeyZh5VdP zh0ypu50A1KmwAEo05MsDThQtBKl0w40zNskQFKpX1oQ== ARC-Authentication-Results: i=1; mx4.messagingengine.com; arc=none (no signatures found); dkim=none (no signatures found); dmarc=none (p=none,has-list-id=yes,d=none) header.from=kernel.org; iprev=pass policy.iprev=209.132.180.67 (vger.kernel.org); spf=none smtp.mailfrom=stable-owner@vger.kernel.org smtp.helo=vger.kernel.org; x-aligned-from=orgdomain_pass; x-ptr=pass x-ptr-helo=vger.kernel.org x-ptr-lookup=vger.kernel.org; x-return-mx=pass smtp.domain=vger.kernel.org smtp.result=pass smtp_org.domain=kernel.org smtp_org.result=pass smtp_is_org_domain=no header.domain=kernel.org header.result=pass header_is_org_domain=yes Authentication-Results: mx4.messagingengine.com; arc=none (no signatures found); dkim=none (no signatures found); dmarc=none (p=none,has-list-id=yes,d=none) header.from=kernel.org; iprev=pass policy.iprev=209.132.180.67 (vger.kernel.org); spf=none smtp.mailfrom=stable-owner@vger.kernel.org smtp.helo=vger.kernel.org; x-aligned-from=orgdomain_pass; x-ptr=pass x-ptr-helo=vger.kernel.org x-ptr-lookup=vger.kernel.org; x-return-mx=pass smtp.domain=vger.kernel.org smtp.result=pass smtp_org.domain=kernel.org smtp_org.result=pass smtp_is_org_domain=no header.domain=kernel.org header.result=pass header_is_org_domain=yes Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751663AbeAWOSD (ORCPT ); Tue, 23 Jan 2018 09:18:03 -0500 Received: from mail.kernel.org ([198.145.29.99]:56766 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751629AbeAWOSC (ORCPT ); Tue, 23 Jan 2018 09:18:02 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E6AC32178B Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=jhogan@kernel.org Date: Tue, 23 Jan 2018 14:17:57 +0000 From: James Hogan To: Mathieu Malaterre Cc: Marcin Nowakowski , "# v4 . 11" , Ralf Baechle , linux-mips@linux-mips.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] MIPS: fix incorrect mem=X@Y handling Message-ID: <20180123141756.GE22211@saruman> References: <1504609608-7694-1-git-send-email-marcin.nowakowski@imgtec.com> <20171221210100.12002-1-malat@debian.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="YkJPYEFdoxh/AXLE" Content-Disposition: inline In-Reply-To: <20171221210100.12002-1-malat@debian.org> User-Agent: Mutt/1.7.2 (2016-11-26) Sender: stable-owner@vger.kernel.org X-Mailing-List: stable@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-Mailing-List: linux-kernel@vger.kernel.org List-ID: --YkJPYEFdoxh/AXLE Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Dec 21, 2017 at 10:00:59PM +0100, Mathieu Malaterre wrote: > From: Marcin Nowakowski >=20 > Change 73fbc1eba7ff added a fix to ensure that the memory range between Please refer to commits with e.g. commit 73fbc1eba7ff ("MIPS: fix mem=3DX@Y commandline processing"). > PHYS_OFFSET and low memory address specified by mem=3D cmdline argument is > not later processed by free_all_bootmem. > This change was incorrect for systems where the commandline specifies > more than 1 mem argument, as it will cause all memory between > PHYS_OFFSET and each of the memory offsets to be marked as reserved, > which results in parts of the RAM marked as reserved (Creator CI20's > u-boot has a default commandline argument 'mem=3D256M@0x0 > mem=3D768M@0x30000000'). >=20 > Change the behaviour to ensure that only the range between PHYS_OFFSET > and the lowest start address of the memories is marked as protected. >=20 > This change also ensures that the range is marked protected even if it's > only defined through the devicetree and not only via commandline > arguments. >=20 > Reported-by: Mathieu Malaterre > Signed-off-by: Marcin Nowakowski > Fixes: 73fbc1eba7ff ("MIPS: fix mem=3DX@Y commandline processing") > Cc: # v4.11 I'm guessing that should technically be v4.11+ > --- > v2: Use updated email adress, add tag for stable. > arch/mips/kernel/setup.c | 19 ++++++++++++++++--- > 1 file changed, 16 insertions(+), 3 deletions(-) >=20 > diff --git a/arch/mips/kernel/setup.c b/arch/mips/kernel/setup.c > index 702c678de116..f19d61224c71 100644 > --- a/arch/mips/kernel/setup.c > +++ b/arch/mips/kernel/setup.c > @@ -375,6 +375,7 @@ static void __init bootmem_init(void) > unsigned long reserved_end; > unsigned long mapstart =3D ~0UL; > unsigned long bootmap_size; > + phys_addr_t ramstart =3D ~0UL; Although practically it might not matter, technically phys_addr_t may be 64-bits (CONFIG_PHYS_ADDR_T_64BIT) even on a 32-bit kernels, in which case ~0UL may not be sufficiently large. Maybe that should be ~(phys_addr_t)0, or perhaps (phys_addr_t)ULLONG_MAX to match add_memory_region(). > bool bootmap_valid =3D false; > int i; > =20 > @@ -395,6 +396,21 @@ static void __init bootmem_init(void) > max_low_pfn =3D 0; > =20 > /* > + * Reserve any memory between the start of RAM and PHYS_OFFSET > + */ > + for (i =3D 0; i < boot_mem_map.nr_map; i++) { > + if (boot_mem_map.map[i].type !=3D BOOT_MEM_RAM) > + continue; > + > + ramstart =3D min(ramstart, boot_mem_map.map[i].addr); Is it worth incorporating this into the existing loop below ... > + } > + > + if (ramstart > PHYS_OFFSET) > + add_memory_region(PHYS_OFFSET, ramstart - PHYS_OFFSET, > + BOOT_MEM_RESERVED); =2E.. and this then placed below that loop? Otherwise I can't find fault with this patch, though i'm not intimately familiar with bootmem. Cheers James > + > + > + /* > * Find the highest page frame number we have available. > */ > for (i =3D 0; i < boot_mem_map.nr_map; i++) { > @@ -664,9 +680,6 @@ static int __init early_parse_mem(char *p) > =20 > add_memory_region(start, size, BOOT_MEM_RAM); > =20 > - if (start && start > PHYS_OFFSET) > - add_memory_region(PHYS_OFFSET, start - PHYS_OFFSET, > - BOOT_MEM_RESERVED); > return 0; > } > early_param("mem", early_parse_mem); > --=20 > 2.11.0 >=20 --YkJPYEFdoxh/AXLE Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEd80NauSabkiESfLYbAtpk944dnoFAlpnRA4ACgkQbAtpk944 dnp22Q/8De+B5OeBDSwCcBMxj10ZQK8NplvIZyNvT4eo/V0myFjVitdj33gWUyAI zSeI1wPi+XwpTbc9ODPTOh9troPXCt4RQ3AjFL28A+L/fCDmsMGpKcWQeT2n4p+H 2pfN9zW4/viVyQoAKdpNYdTW5llMbxiI9n4Hu9y0UgWaSR/2tx9pNDUX2jYVuidM Vu9Ruw4TpPPoxSlGzp3KHwC281UqqNAxq/gV+SM1LW5PxRjln/ZnMlmlyX/kVauS /8qre8JgAPb3gXH1B2GJ9xAnfPK+SnpnaEqTKKzLzs71EjhDoBDuPzrUj1zdJ97q h1f9vdN6ZsuhzSGQt8y4g3RuqQtf5e+NhEdNJwuKnmJomb/5/nCPEKRWPiylH9S/ aPOS7zX0ijdEgg95X6YH/ryNLE/W4RfUl0unxq+qU7A0jWJ9CC8D2bGA0xsEqCeI 7Un0fhF5MmIxXLN2CtnUyKqjpJCr1ocvPPxX5velCja0jxWfZf5eBSFG1hOl/uMP ju/IPPQHacA9C7pGY8Xhhoh8vII2ZvpTXGT07VvZ3xgBWaBZdv0igwptxNIZpeym YaF3tTNfBwPKitpCDLvVa6Yekm3ZWSPAERpNtejtrCGW4nrTHZb3ZKMS8Vknk2XV D5anqso1+FyMhYsyeaom/CgTZQSmC59Nt44Wq2ZNW3IXhHEMEkE= =D+jJ -----END PGP SIGNATURE----- --YkJPYEFdoxh/AXLE--