From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754899Ab0H3Jbd (ORCPT ); Mon, 30 Aug 2010 05:31:33 -0400 Received: from plane.gmane.org ([80.91.229.3]:48627 "EHLO plane.gmane.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754836Ab0H3Jbb (ORCPT ); Mon, 30 Aug 2010 05:31:31 -0400 Date: Mon, 30 Aug 2010 11:31:12 +0200 From: Florian Mickler To: public-ebiederm-aS9lmoZGLiVWk0Htik3J/w@plane.gmane.org (Eric W. Biederman) Newsgroups: gmane.linux.kernel.kernel-testers,gmane.linux.kernel Cc: "Rafael J. Wysocki" , Linux Kernel Mailing List , Kernel Testers List , Maciej Rutecki , "David Hill" , "H. Peter Anvin" , "H. Peter Anvin" Subject: Re: [Bug #16173] After uncompressing the kernel, at boot time, the server hangs. Message-ID: <20100830113112.29b670c0@schatten.dmk.lab> In-Reply-To: References: <201008300128.34970.rjw@sisk.pl> X-Newsreader: Claws Mail 3.7.6 (GTK+ 2.20.1; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, 29 Aug 2010 16:49:06 -0700 ebiederm-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org (Eric W. Biederman) wrote: > "Rafael J. Wysocki" writes: > > > On Monday, August 30, 2010, Eric W. Biederman wrote: > >> "Rafael J. Wysocki" writes: > >> > >> > This message has been generated automatically as a part of a report > >> > of regressions introduced between 2.6.34 and 2.6.35. > >> > > >> > The following bug entry is on the current list of known regressions > >> > introduced between 2.6.34 and 2.6.35. Please verify if it still should > >> > be listed and let the tracking team know (either way). > >> > > >> > >> Looks like it's fixed. > > > > Good. What mainlne commit is that? > > commit 910081322bf9c4ed392dc8f2ecbc33fdb42a2958 > Author: Eric W. Biederman > Date: Wed Aug 4 12:51:11 2010 -0700 > > x86/apic: Map the local apic when parsing the MP table. > > This fixes a regression in 2.6.35 from 2.6.34, that is > present for select models of Intel cpus when people are > using an MP table. > > The commit cf7500c0ea133d66f8449d86392d83f840102632 > "x86, ioapic: In mpparse use mp_register_ioapic" started > calling mp_register_ioapic from MP_ioapic_info. An extremely > simple change that was obviously correct. Unfortunately > mp_register_ioapic did just a little more than the previous > hand crafted code and so we gained this call path. > > The problem call path is: > MP_ioapic_info() > mp_register_ioapic() > io_apic_unique_id() > io_apic_get_unique_id() > get_physical_broadcast() > modern_apic() > lapic_get_version() > apic_read(APIC_LVR) > > Which turned out to be a problem because the local apic > was not mapped, at that point, unlike the similar point > in the ACPI parsing code. > > This problem is fixed by mapping the local apic when > parsing the mptable as soon as we reasonably can. > > Looking at the number of places we setup the fixmap for > the local apic, I see some serious simplification opportunities. > For the moment except for not duplicating the setting up of the > fixmap in init_apic_mappings, I have not acted on them. > > The regression from 2.6.34 is tracked in bug > https://bugzilla.kernel.org/show_bug.cgi?id=16173 > > Cc: stable-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org > Reported-by: David Hill > Reported-by: Tvrtko Ursulin > Tested-by: Tvrtko Ursulin > Signed-off-by: Eric W. Biederman Closed, thx. Guan Xin reported a stable regression from 2.6.35.1->2.6.35.2 in this bugreport which I moved to https://bugzilla.kernel.org/show_bug.cgi?id=17411 . Cheers, Flo