Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751503AbVJZWxh (ORCPT ); Wed, 26 Oct 2005 18:53:37 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751504AbVJZWxh (ORCPT ); Wed, 26 Oct 2005 18:53:37 -0400 Received: from mail.kroah.org ([69.55.234.183]:36006 "EHLO perch.kroah.org") by vger.kernel.org with ESMTP id S1751503AbVJZWxg (ORCPT ); Wed, 26 Oct 2005 18:53:36 -0400 Date: Wed, 26 Oct 2005 15:51:40 -0700 From: Greg KH To: Matthew Wilcox Cc: Roland Dreier , gregkh@suse.de, mst@mellanox.co.il, linux-kernel@vger.kernel.org, linux-pci@atrey.karlin.mff.cuni.cz Subject: Re: AMD 8131 and MSI quirk Message-ID: <20051026225140.GD13495@kroah.com> References: <524q799p2t.fsf@cisco.com> <20051022233220.GA1463@parisc-linux.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20051022233220.GA1463@parisc-linux.org> User-Agent: Mutt/1.5.11 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1603 Lines: 33 On Sat, Oct 22, 2005 at 05:32:20PM -0600, Matthew Wilcox wrote: > On Sat, Oct 22, 2005 at 03:14:34PM -0700, Roland Dreier wrote: > > The current quirk_amd_8131_ioapic() function sets a global > > pci_msi_quirk flag, which disables MSI/MSI-X for all devices in the > > system. This is safe but suboptimal, because there may be devices on > > other buses not related to the AMD 8131 bridge, for which MSI would > > work fine. As an example, see the end of this email for a lspci -t > > from a real Opteron system that has PCI-X buses coming from an AMD > > 8131 and PCI Express buses coming from an Nforce4 bridge -- MSI works > > fine for the Mellanox InfiniBand adapter on the PCIe bus, if we allow > > it to be enabled. > > > > I guess what we really should be doing is setting the dev->no_msi flag > > for all devices below the AMD 8131 PCI-X bridge rather than turning > > off MSI globally. Of course this is somewhat tricky, since a device > > could be hotplugged onto a bus below the AMD 8131. Greg, any thoughts > > about the proper way to use the driver model infrastructure to handle > > this? > > Perhaps the right thing to do is to change pad2 (in struct pci_bus) to > bus_flags and make bit 0 PCI_BRIDGE_FLAGS_NO_MSI ? Yeah, I can't think of any way to use the device tree to do this, so this sounds as good a way as any. thanks, greg k-h - 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/