From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752637AbZKLFXD (ORCPT ); Thu, 12 Nov 2009 00:23:03 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752245AbZKLFXD (ORCPT ); Thu, 12 Nov 2009 00:23:03 -0500 Received: from hera.kernel.org ([140.211.167.34]:39716 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751891AbZKLFXB (ORCPT ); Thu, 12 Nov 2009 00:23:01 -0500 Message-ID: <4AFB9B60.7010009@kernel.org> Date: Wed, 11 Nov 2009 21:21:36 -0800 From: Yinghai Lu User-Agent: Thunderbird 2.0.0.23 (X11/20090817) MIME-Version: 1.0 To: ykzhao CC: "Rafael J. Wysocki" , Ingo Molnar , Thomas Gleixner , "H. Peter Anvin" , Len Brown , "linux-kernel@vger.kernel.org" , ACPI Devel Maling List , pm list Subject: Re: [PATCH] x86: make sure wakeup code is below 1M -v2 References: <4AF7D820.5040503@kernel.org> <200911091315.13151.rjw@sisk.pl> <4AFA210B.3020207@kernel.org> <1257930767.31183.113.camel@localhost.localdomain> <4AFB0B02.4050306@kernel.org> <1257988632.31183.134.camel@localhost.localdomain> <4AFB7D21.6050004@kernel.org> <1257997072.3888.4.camel@localhost.localdomain> In-Reply-To: <1257997072.3888.4.camel@localhost.localdomain> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org ykzhao wrote: > On Thu, 2009-11-12 at 11:12 +0800, Yinghai Lu wrote: >> ykzhao wrote: >>> On Thu, 2009-11-12 at 03:05 +0800, Yinghai Lu wrote: >>>> ykzhao wrote: >>>>> On Wed, 2009-11-11 at 10:27 +0800, Yinghai Lu wrote: >>>>>> try to find_e820_area/reserve_early, and call acpi_reserve_memory early >>>>>> >>>>>> to get area is below 1M >>>>>> >>>>>> -v2: change function name to acpi_reserve_wakeup_memory according to Rafael >>>>> It seems that the function of find_e820_area is called in several >>>>> places. >>>>> >Initmem_init: bootmap = find_e820_area(0, end_pfn<>>>> bootmap_size, PAGE_SIZE); >>>>> >>>>> If we also call it in the acpi_reserve_wakeup_memory, do we get the same >>>>> base address as that obtained in initmem_init? >>>> no. find_e820_area will check the reserve res array that could be updated by reserve_early. >>> It will check the reserved region array when calling the function of >>> find_e820_area. >>> But it seems that the array is not updated when the find_e820_area is >>> called in the function of initmem_init. >> right after that will use reserve_bootmem for those range in initmem_init. > Yes. The reserve_bootmem is called for the range in initmem_init. > But the reserved_early array is not updated. it is not needed anymore because bootmem for that node is ready at that point, could use reserve_bootmem_node directly. and before that early_res_to_bootmem will convert that early resource that fall into that node range to bootmem reserved too. please check code setup_node_bootmem/early_node_mem/early_res_to_bootmem ...and reserve_bootmem_node... YH