From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750712AbVJABZt (ORCPT ); Fri, 30 Sep 2005 21:25:49 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1750713AbVJABZs (ORCPT ); Fri, 30 Sep 2005 21:25:48 -0400 Received: from fmr21.intel.com ([143.183.121.13]:33468 "EHLO scsfmr001.sc.intel.com") by vger.kernel.org with ESMTP id S1750712AbVJABZs (ORCPT ); Fri, 30 Sep 2005 21:25:48 -0400 Date: Fri, 30 Sep 2005 18:25:30 -0700 From: "Siddha, Suresh B" To: "Rafael J. Wysocki" , ak@suse.de Cc: discuss@x86-64.org, "Siddha, Suresh B" , Andi Kleen , Pavel Machek , Andrew Morton , LKML Subject: Re: [discuss] Re: [PATCH][Fix][Resend] Fix Bug #4959: Page tables corrupted during resume on x86-64 (take 3) Message-ID: <20050930182530.E28092@unix-os.sc.intel.com> References: <200509281624.29256.rjw@sisk.pl> <20050929165905.B15943@unix-os.sc.intel.com> <200509300726.20528.rjw@sisk.pl> <200509300851.09327.rjw@sisk.pl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <200509300851.09327.rjw@sisk.pl>; from rjw@sisk.pl on Fri, Sep 30, 2005 at 08:51:08AM +0200 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Sep 30, 2005 at 08:51:08AM +0200, Rafael J. Wysocki wrote: > On Friday, 30 of September 2005 07:26, Rafael J. Wysocki wrote: > > Of course. The .config is attached. Generally, it's a non-SMP box, and commenting > > out the zap_low_mappings((0) in setup_arch() makes the box boot again. > > One more datapoint: The box boots if I move the zap_low_mappings((0) > in the following way: > > --- linux-2.6.14-rc2-git7.orig/arch/x86_64/kernel/setup.c 2005-09-30 07:39:35.000000000 +0200 > +++ linux-2.6.14-rc2-git7/arch/x86_64/kernel/setup.c 2005-09-30 08:31:20.000000000 +0200 > @@ -571,8 +571,6 @@ > > init_memory_mapping(0, (end_pfn_map << PAGE_SHIFT)); > > - zap_low_mappings(0); > - > #ifdef CONFIG_ACPI > /* > * Initialize the ACPI boot-time table parser (gets the RSDP and SDT). > @@ -680,6 +678,8 @@ > get_smp_config(); > init_apic_mappings(); > #endif > + zap_low_mappings(0); > + Rafael, I still can't reproduce the issue. All of my EM64T systems are SMP and looks like this problem happens only on an UP system. Looks like there is some code flow which is still assuming the presence of low mappings. Andi, can you please see if you can reproduce this on one of your systems similar to Rafael's. > /* > * Request address space for all standard RAM and ROM resources > > iHowever, if I place the zap_low_mappings((0) before the > #define CONFIG_X86_LOCAL_APIC in line 673, it doesn't boot. > > Certainly init_apic_mappings() is at fault. Could that be a reult of a call to > alloc_bootmem_pages()?. I looked at init_apic_mappings() and didn't give me any clue. alloc_bootmem_pages() don't use low direct mappings. > And one more: I have to boot with "noapic". Are you saying "noapic" is the second data point or you need to use "noapic" along with moving zap_low_mappings(0) thanks, suresh