Re: Removing CONFIG_PCI_MSI choice

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Sep 23, 2008 at 11:46:36AM -0700, Jesse Barnes wrote:
 > On Friday, September 19, 2008 1:31 pm David Miller wrote:
 > > From: Roland Dreier <rdreier@xxxxxxxxx>
 > > Date: Fri, 19 Sep 2008 10:10:20 -0700
 > >
 > > > I investigated a little and discovered that current Fedora kernels
 > > > are built with CONFIG_PCI_MSI=y but have an extra patch added that
 > > > defaults MSI to off (and requires booting with "pci=msi" to enable
 > > > MSI).
 > >
 > > That makes me feel like a real idiot for trying to make
 > > sure all the in-kernel MSI quirks are there and sane.
 > 
 > Chuck and/or Dave (Jones), can one of you summarize the list of bugs you ran 
 > into last time you enabled MSI unconditionally on Fedora?  Maybe we can 
 > convert the errors you saw into new blacklist entries...  If things break 
 > horribly everytime you try MSI, I can't really blame you for disabling it, 
 > but it would be nice if we could make some progress here.

I don't have a list of bug reports handy, but here's one such example
I found searching for pci=nomsi in fedora bugzilla..
https://bugzilla.redhat.com/show_bug.cgi?id=438621
Where the SATA controller wouldn't boot without that option.

It was nowhere near as bad as the MMCONFIG option, which seemed to be
utterly busted everywhere, but it was still something that came up
often enough at one point, that we flipped the default state.

Every time we start a new devel branch, we turn it back to the
upstream default again, to see if things have improved, and
invariably, they haven't.  Due to an oversight, we haven't actually
disabled that patch on rawhide this time around.

	Dave

-- 
http://www.codemonkey.org.uk
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux