On Thu, Jun 13, 2013 at 10:18:30AM -0400, John Baldwin wrote: > On Tuesday, June 11, 2013 1:48:22 pm Konstantin Belousov wrote: > > + count = pci_msi_count(self); > > + if (count >= 1) { > > + if (count > 1) > > + count = 1; > > You can just drop the if and always do the assignment here, yes? > (Minor nit I know, and the original patch is fine for testing)
The comparision is useless, I removed it in my repo, thank you. I remember somebody complained that xhci(4) cannot attach on the Macs with the diagnostic about missing PRT entry for routing the legacy interrupt. As if Apple does not care about proper legacy routing. Hopefully, use of MSI-X would help there too. Here is it: http://lists.freebsd.org/pipermail/freebsd-current/2013-February/039595.html http://people.freebsd.org/~dchagin/macbookpro/dmesg.generic.HEAD.txt
pgpgXj2sYflS1.pgp
Description: PGP signature