Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Tue, 14 Jan 2003 18:33:07 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Tue, 14 Jan 2003 18:33:07 -0500 Received: from fmr02.intel.com ([192.55.52.25]:42702 "EHLO caduceus.fm.intel.com") by vger.kernel.org with ESMTP id ; Tue, 14 Jan 2003 18:33:06 -0500 Message-ID: From: "Grover, Andrew" To: "Martin J. Bligh" , Zwane Mwaikambo , "Nakajima, Jun" Cc: "Protasevich, Natalie" , "Pallipadi, Venkatesh" , William Lee Irwin III , Christoph Hellwig , James Cleverdon , Linux Kernel Subject: mpparse.c is a mess (was: APIC version) Date: Tue, 14 Jan 2003 15:41:49 -0800 MIME-Version: 1.0 X-Mailer: Internet Mail Service (5.5.2653.19) content-class: urn:content-classes:message Content-Type: text/plain; charset="iso-8859-1" Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org > From: Martin J. Bligh [mailto:mbligh@aracnet.com] > >> The entries in acpi_version[] are indexed by the APIC id, not > >> smp_processor_id(). So you can overwrite acpi_version[] > for a different > >> processor. > > > > Is it possible to use smp_processor_id instead to avoid > wasting memory > > for the sparse APIC id case? > > No, the array is set up in mpparse.c before we know the real > processor > numbers. I just thought I'd mention, if anyone has any spare time or needs a project, that it would be really nice if there was a more modular interface for this stuff. Right now we have MPS code and ACPI code messing with the same data structures (!!) and it is bug-prone and confusing. Every time I try to fix anything that touches my area (ACPI IRQ routing) I am paralyzed by fear that any change I make will break something, because it is a big, swampy bog of code. Regards -- Andy - 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/