Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933791Ab3CHHG5 (ORCPT ); Fri, 8 Mar 2013 02:06:57 -0500 Received: from mail-pb0-f44.google.com ([209.85.160.44]:55291 "EHLO mail-pb0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753117Ab3CHHGz (ORCPT ); Fri, 8 Mar 2013 02:06:55 -0500 Date: Thu, 7 Mar 2013 23:06:51 -0800 From: Tejun Heo To: Yinghai Lu Cc: "Yu, Fenghua" , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Andrew Morton , Thomas Renninger , Tang Chen , linux-kernel@vger.kernel.org, Pekka Enberg , Jacob Shin , "Rafael J. Wysocki" , linux-acpi@vger.kernel.org Subject: Re: [PATCH 04/14] x86, ACPI: make acpi override finding work with 32bit flat mode Message-ID: <20130308070651.GN14556@mtj.dyndns.org> References: <1362718720-27048-1-git-send-email-yinghai@kernel.org> <1362718720-27048-5-git-send-email-yinghai@kernel.org> <20130308055023.GF14556@mtj.dyndns.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2851 Lines: 69 Hello, Yinghai. On Thu, Mar 07, 2013 at 10:57:21PM -0800, Yinghai Lu wrote: > On Thu, Mar 7, 2013 at 9:50 PM, Tejun Heo wrote: > > On Thu, Mar 07, 2013 at 08:58:30PM -0800, Yinghai Lu wrote: > >> We will find acpi tables in initrd during head_32.S in 32bit flat mode. > >> > >> So need acpi_initrd_override_find could take phys directly. > > > > The patch description doesn't explain even half of what's going on. > > hope HPA could understand. > > Access initrd before relocate_initrd and init_memory mapping. I really hope the changelogs were better. Eh well... > >> -/* All but ACPI_SIG_RSDP and ACPI_SIG_FACS: */ > >> -static const char * const table_sigs[] = { > >> - ACPI_SIG_BERT, ACPI_SIG_CPEP, ACPI_SIG_ECDT, ACPI_SIG_EINJ, > >> - ACPI_SIG_ERST, ACPI_SIG_HEST, ACPI_SIG_MADT, ACPI_SIG_MSCT, > >> - ACPI_SIG_SBST, ACPI_SIG_SLIT, ACPI_SIG_SRAT, ACPI_SIG_ASF, > >> - ACPI_SIG_BOOT, ACPI_SIG_DBGP, ACPI_SIG_DMAR, ACPI_SIG_HPET, > >> - ACPI_SIG_IBFT, ACPI_SIG_IVRS, ACPI_SIG_MCFG, ACPI_SIG_MCHI, > >> - ACPI_SIG_SLIC, ACPI_SIG_SPCR, ACPI_SIG_SPMI, ACPI_SIG_TCPA, > >> - ACPI_SIG_UEFI, ACPI_SIG_WAET, ACPI_SIG_WDAT, ACPI_SIG_WDDT, > >> - ACPI_SIG_WDRT, ACPI_SIG_DSDT, ACPI_SIG_FADT, ACPI_SIG_PSDT, > >> - ACPI_SIG_RSDT, ACPI_SIG_XSDT, ACPI_SIG_SSDT, NULL }; > > > > Why is this table made a stack variable? What's the benefit of doing > > that? > > so I do need to switch global variables to phys and access it. I can't really understand what your response means. Can you please elaborate? > > Is it really necessary to make the function take both virtual and > > physical addresses? Can't we just make the function take phys_addr_t > > and update everyone to call with physaddr? Also @is_phys isn't simple > > address switch. It also changes error reporting. If you're gonna > > keep @is_phys, let's at least write up a function comment explaining > > what's going on and why we need it. But, really, if at all possible, > > let's change the function to take single type of argument and > > predicate error message printing on something else (e.g. early printk > > initialized or whatever). > > yes, one for 32bit from head_32.S, phys. > one for 64bit from head64.c. with _va. head64.c can't call with phys? Why not? > Not sure if I could use early_printk from head_32.S, as Fenghua does > not print out > from microcode updating early in the same parts. ISTR it works but it doens't have to (although it would be much nicer if it did). You can test whether printk is online and skip if it isn't online yet. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/