From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [PATCH V2] ARM: dts: bcm283x: Reserve first page for firmware Date: Tue, 9 May 2017 09:52:53 -0700 Message-ID: References: <8737cenfhc.fsf@eliezer.anholt.net> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <8737cenfhc.fsf-omZaPlIz5HhaEpDpdNBo/KxOck334EZe@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Eric Anholt , Phil Elwell , Rob Herring , Mark Rutland , Russell King , bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org On 05/09/2017 09:48 AM, Eric Anholt wrote: > Phil Elwell writes: > >> The Raspberry Pi startup stub files for multi-core BCM27XX processors >> make the secondary CPUs spin until the corresponding mailbox is >> written. These stubs are loaded at physical address 0x00000xxx (as seen >> by the ARMs), but this page will be reused by the kernel unless it is >> explicitly reserved, causing the waiting cores to execute random code. >> >> Use the /memreserve/ Device Tree directive to mark the first page as >> off-limits to the kernel. >> >> See: https://github.com/raspberrypi/linux/issues/1989 >> >> Signed-off-by: Phil Elwell > > This looks great. We're currently in the merge window, so I can't > generate a new bcm2835-dt-next yet, but I'll pick this patch up when I > do. > > Thanks for submitting upstream! Considering that this is a fix, we could actually squeeze it in the devicetree/fixes branch and we could submit those as soon as v4.12-rc1 is tagged, your call. -- Florian -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html