From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754985AbZEOVOX (ORCPT ); Fri, 15 May 2009 17:14:23 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753229AbZEOVOO (ORCPT ); Fri, 15 May 2009 17:14:14 -0400 Received: from hera.kernel.org ([140.211.167.34]:53935 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753227AbZEOVON (ORCPT ); Fri, 15 May 2009 17:14:13 -0400 Message-ID: <4A0DDAF0.7080005@kernel.org> Date: Fri, 15 May 2009 14:13:20 -0700 From: Yinghai Lu User-Agent: Thunderbird 2.0.0.19 (X11/20081227) MIME-Version: 1.0 To: Bjorn Helgaas CC: Ingo Molnar , Thomas Gleixner , "H. Peter Anvin" , Jesse Barnes , "linux-kernel@vger.kernel.org" , linux-pci@vger.kernel.org Subject: Re: [PATCH] x86: update_mptable need pci_routeirq References: <4A0DC7DC.7000204@kernel.org> <200905151507.29083.bjorn.helgaas@hp.com> In-Reply-To: <200905151507.29083.bjorn.helgaas@hp.com> 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 Bjorn Helgaas wrote: > On Friday 15 May 2009 01:51:56 pm Yinghai Lu wrote: >> to get all device irq routing and save them > > This changelog isn't really enough for me to understand > what's going on. > >> Index: linux-2.6/arch/x86/kernel/mpparse.c >> =================================================================== >> --- linux-2.6.orig/arch/x86/kernel/mpparse.c >> +++ linux-2.6/arch/x86/kernel/mpparse.c >> @@ -17,6 +17,7 @@ >> #include >> #include >> #include >> +#include >> >> #include >> #include >> @@ -961,6 +962,7 @@ static int __initdata enable_update_mpta >> static int __init update_mptable_setup(char *str) >> { >> enable_update_mptable = 1; >> + pci_routeirq = 1; > > I hate to see new uses of pci_routeirq. It'd be nice to remove it > completely someday. But maybe it can't be helped. > >> return 0; >> } >> early_param("update_mptable", update_mptable_setup); > > "update_mptable" is apparently a boot option, but it's not mentioned > in Documentation/kernel-parameters.txt. Is this some sort of debug > code or BIOS-writer's helper? Same with "alloc_mptable". Why do > we have this stuff? > 1. kexec path: after acpi kernel and use kexec to load kerenl with mptable support only or acpi=off 2. help to BIOS engineer to understand how to set mptable and irq routing setup by chipset. esp system have several cards and there is bridge in those cards. YH