From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752789AbbCZBGt (ORCPT ); Wed, 25 Mar 2015 21:06:49 -0400 Received: from eu-smtp-delivery-143.mimecast.com ([207.82.80.143]:6099 "EHLO eu-smtp-delivery-143.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752428AbbCZBDN convert rfc822-to-8bit (ORCPT ); Wed, 25 Mar 2015 21:03:13 -0400 Date: Wed, 25 Mar 2015 16:43:54 +0000 From: Catalin Marinas To: Hanjun Guo Cc: "Rafael J. Wysocki" , Will Deacon , Olof Johansson , Grant Likely , Mark Rutland , Ashwin Chaugule , Lorenzo Pieralisi , Robert Richter , Arnd Bergmann , Graeme Gregory , linaro-acpi@lists.linaro.org, Marc Zyngier , Jon Masters , Timur Tabi , Mark Salter , linux-kernel@vger.kernel.org, Leif Lindholm , Ard Biesheuvel , linux-acpi@vger.kernel.org, Mark Brown , Suravee Suthikulpanit , Sudeep Holla , linux-arm-kernel@lists.infradead.org Subject: Re: [patch v11 03/23] ARM64: allow late use of early_ioremap Message-ID: <20150325164354.GB14585@localhost> References: <1427205776-5060-1-git-send-email-hanjun.guo@linaro.org> <1427205776-5060-4-git-send-email-hanjun.guo@linaro.org> MIME-Version: 1.0 In-Reply-To: <1427205776-5060-4-git-send-email-hanjun.guo@linaro.org> User-Agent: Mutt/1.5.21 (2010-09-15) X-OriginalArrivalTime: 26 Mar 2015 00:56:49.0018 (UTC) FILETIME=[BD5B5DA0:01D0675F] X-MC-Unique: eP3-QDLbRmy400ljg4KIWg-3 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: 8BIT Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 24, 2015 at 10:02:36PM +0800, Hanjun Guo wrote: > From: Mark Salter > > Commit 0e63ea48b4d8 (arm64/efi: add missing call to early_ioremap_reset()) > added a missing call to early_ioremap_reset(). This triggers a BUG if code > tries using early_ioremap() after the early_ioremap_reset(). This is a > problem for some ACPI code which needs short-lived temporary mappings > after paging_init() but before acpi_early_init() in start_kernel(). This > patch adds definitions for the __late_set_fixmap() and __late_clear_fixmap() > which avoids the BUG by allowing later use of early_ioremap(). > > CC: Leif Lindholm > CC: Ard Biesheuvel > Tested-by: Suravee Suthikulpanit > Tested-by: Mark Langsdorf > Tested-by: Jon Masters > Tested-by: Robert Richter > Tested-by: Timur Tabi > Acked-by: Robert Richter > Reviewed-by: Grant Likely > Signed-off-by: Mark Salter > Signed-off-by: Hanjun Guo Acked-by: Catalin Marinas