From owner-svn-src-all@FreeBSD.ORG Fri Mar 22 20:04:45 2013 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 6A98CF87; Fri, 22 Mar 2013 20:04:45 +0000 (UTC) (envelope-from marius@alchemy.franken.de) Received: from alchemy.franken.de (alchemy.franken.de [194.94.249.214]) by mx1.freebsd.org (Postfix) with ESMTP id E305C225; Fri, 22 Mar 2013 20:04:44 +0000 (UTC) Received: from alchemy.franken.de (localhost [127.0.0.1]) by alchemy.franken.de (8.14.6/8.14.6/ALCHEMY.FRANKEN.DE) with ESMTP id r2MJpHpZ095985; Fri, 22 Mar 2013 20:51:17 +0100 (CET) (envelope-from marius@alchemy.franken.de) Received: (from marius@localhost) by alchemy.franken.de (8.14.6/8.14.6/Submit) id r2MJpHM8095984; Fri, 22 Mar 2013 20:51:17 +0100 (CET) (envelope-from marius) Date: Fri, 22 Mar 2013 20:51:17 +0100 From: Marius Strobl To: Ryan Stone Subject: Re: svn commit: r247570 - head/sys/dev/aac Message-ID: <20130322195117.GZ955@alchemy.franken.de> References: <201303011955.r21JtAqn069123@svn.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 22 Mar 2013 20:04:45 -0000 On Fri, Mar 22, 2013 at 03:26:25PM -0400, Ryan Stone wrote: > On Fri, Mar 1, 2013 at 2:55 PM, Marius Strobl wrote: > > > Author: marius > > Date: Fri Mar 1 19:55:10 2013 > > New Revision: 247570 > > URL: http://svnweb.freebsd.org/changeset/base/247570 > > > > Log: > > - Make tables, device ID strings etc const. This includes #ifdef'ing 0 > > aac_command_status_table, which is actually unused since r111532. > > While at it, make aac_if a pointer to the now const interface tables > > instead of copying them over to the softc (this alone already reduces > > the > > size of aac.ko on amd64 by ~1 KiB). > > - Remove redundant softc members. > > - Use DEVMETHOD_END. > > - Use NULL instead of 0 for pointers. > > - Remove redundant bzero(9)'ing of the softc. > > - Use pci_enable_busmaster(9) instead of duplicating it. > > - Remove redundant checking for PCIM_CMD_MEMEN (resource allocation will > > just fail). > > - Canonicalize the error messages in case of resource allocation > > failures. > > - Add support for using MSI instead of INTx, controllable via the tunable > > hw.aac.enable_msi (defaulting to on). > > > > This doesn't seem to be working for the 2230S controller. It panics at > mountroot with "panic: run_interrupt_driven_config_hooks". It works if I > set hw.aac.enable_msi=0. This is happening with two different > motherboards, so I'm not entirely sure if the problem is the BIOS or the > driver. I put a verbose boot log here: > http://people.freebsd.org/~rstone/aac_msi.log Hrm, I had checked the corresponding linux driver and it does nothing special for enabling the use of an MSI and neither excludes certain controller variants in this regard. On the other hand, it doesn't default to using an MSI either. It's unfortunate that nothing else in that system uses an MSI so far. Do you happen to have an easy way of testing whether MSIs generally work with that machine, e.g. by giving a sufficiently new Ethernet card a try? Marius