Date: Wed, 23 May 2007 20:07:56 +0100 From: Rui Paulo <rpaulo@fnop.net> To: Andrew Gallatin <gallatin@cs.duke.edu> Cc: Rui Paulo <rpaulo@fnop.net>, cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/pci pci_pci.c Message-ID: <86abvvs6ib.wl%rpaulo@fnop.net> In-Reply-To: <18004.36709.708317.994071@grasshopper.cs.duke.edu> References: <200705231531.l4NFV12H058289@repoman.freebsd.org> <86bqgbs8kt.wl%rpaulo@fnop.net> <18004.36709.708317.994071@grasshopper.cs.duke.edu>
next in thread | previous in thread | raw e-mail | index | archive | help
At Wed, 23 May 2007 15:00:30 -0400 (EDT), Andrew Gallatin wrote: > > > Rui Paulo writes: > > At Wed, 23 May 2007 15:31:01 +0000 (UTC), > > Andrew Gallatin wrote: > > > > > > gallatin 2007-05-23 15:31:01 UTC > > > > > > FreeBSD src repository > > > > > > Modified files: > > > sys/dev/pci pci_pci.c > > > Log: > > > Fix a typo in pcib_alloc_msi{x} which resulted in the > > > device's, not the bridge's, softc to be used to check the > > > PCIB_DISABLE_MSI flag. This resulted in randomly allowing > > > or denying MSI interrupts based on whatever value the driver > > > happened to store at sizeof(device_t) bytes into its softc. > > > > > > I noticed this when I stopped getting MSI interrupts > > > after slighly re-arranging mxge's softc yesterday. > > > > Wow, I'll have to try this out on msk(4).. > > To be clear, pci_alloc_msi() was failing, so MSI was not > used at all. It didn't affect the actual generation > of interrupts once MSI is enabled.. Oh, ok. Thanks for the explanation. -- Rui Paulo
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?86abvvs6ib.wl%rpaulo>