From owner-freebsd-acpi@FreeBSD.ORG Wed Nov 17 20:17:46 2004 Return-Path: Delivered-To: freebsd-acpi@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id E85CD16A4CE; Wed, 17 Nov 2004 20:17:46 +0000 (GMT) Received: from ylpvm29.prodigy.net (ylpvm29-ext.prodigy.net [207.115.57.60]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9B74643D1D; Wed, 17 Nov 2004 20:17:46 +0000 (GMT) (envelope-from nate@root.org) Received: from [10.0.0.34] (adsl-67-119-74-222.dsl.sntc01.pacbell.net [67.119.74.222])iAHKHcDU004633; Wed, 17 Nov 2004 15:17:38 -0500 Message-ID: <419BB1DB.3090002@root.org> Date: Wed, 17 Nov 2004 12:17:31 -0800 From: Nate Lawson User-Agent: Mozilla Thunderbird 0.7.3 (X11/20040901) X-Accept-Language: en-us, en MIME-Version: 1.0 To: John Baldwin References: <200411111737.00537.jhb@FreeBSD.org> <200411161339.13818.jhb@FreeBSD.org> <419A8CEB.4070305@root.org> <200411170905.34296.jhb@FreeBSD.org> In-Reply-To: <200411170905.34296.jhb@FreeBSD.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit cc: freebsd-acpi@FreeBSD.org cc: gallatin@FreeBSD.org cc: scottl@FreeBSD.org Subject: Re: New ACPI PCI Link Routing code X-BeenThere: freebsd-acpi@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: ACPI and power management development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 17 Nov 2004 20:17:47 -0000 John Baldwin wrote: > On Tuesday 16 November 2004 06:27 pm, Nate Lawson wrote: > >>John Baldwin wrote: >> >>>On Thursday 11 November 2004 05:37 pm, John Baldwin wrote: >>> >>>>I've done some work on the ACPI PCI link code to make it a bit more like >>>>$PIR in that it is link centric and uses actual new-bus devices for each >>>>device link. One benefit of this is that unused links will be disabled >>>>now which might help with interrupt aliasing problems on machines using >>>>APICs. Also, instead of routing IRQs for links via PCI device numbers >>>>using tunables, they are now routed via the link name ala $PIR. Thus, >>>>one uses 'hw.pci.link.LNKA.irq=X' to route LNKA to IRQ X. Also, when >>>>choosing a virgin interrupt, we no longer try to guess at which IRQs >>>>might be used by ISA devices. Instead, we only use known-good IRQs >>>>including IRQs that the BIOS has already used and the SCI if the link is >>>>being routed via ISA IRQs. The patch is at >>>>http://www.FreeBSD.org/~jhb/patches/acpi_pci_link.patch Please test and >>>>let me know if there are any problems, thanks. >>> >>>I've updated this to the latest current and verified that it compiles ok >>>(since I had at least one report that it didn't patch cleanly and/or >>>compile). I plan to commit this in a couple of days unless I hear some >>>sort of negative feedback. >> >>Oh, please s/style(msmith)/style(9) in some parts you added. > > > Heh, I can do that in the new code, sure. I just ask because acpi_pci_link.c was already closer to style(9) and I'd like all new modules to do that as well. Changes to style(msmith) files should conform to the existing file unless they rewrite it so much that it can be brought into compliance easily. -Nate