From owner-freebsd-current@FreeBSD.ORG Mon Aug 6 12:20:40 2012 Return-Path: Delivered-To: current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C214C106564A for ; Mon, 6 Aug 2012 12:20:40 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from bigwig.baldwin.cx (bigknife-pt.tunnel.tserv9.chi1.ipv6.he.net [IPv6:2001:470:1f10:75::2]) by mx1.freebsd.org (Postfix) with ESMTP id 9A79F8FC17 for ; Mon, 6 Aug 2012 12:20:40 +0000 (UTC) Received: from jhbbsd.localnet (unknown [209.249.190.124]) by bigwig.baldwin.cx (Postfix) with ESMTPSA id 0B383B94A for ; Mon, 6 Aug 2012 08:20:40 -0400 (EDT) From: John Baldwin To: current@freebsd.org Date: Mon, 6 Aug 2012 08:20:39 -0400 User-Agent: KMail/1.13.5 (FreeBSD/8.2-CBSD-20110714-p17; KDE/4.5.5; amd64; ; ) MIME-Version: 1.0 Content-Type: Text/Plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Message-Id: <201208060820.39553.jhb@freebsd.org> X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.7 (bigwig.baldwin.cx); Mon, 06 Aug 2012 08:20:40 -0400 (EDT) Cc: Subject: [PATCH] Make ida(4) MPSAFE and a host of other fixes X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 06 Aug 2012 12:20:40 -0000 ida(4) is a driver for older Compaq RAID adapters (PCI and EISA). It is one of the few remaining non-MPSAFE storage drivers. I have a patch to add locking to it, but while doing that I fixed several other issues including incorrect bus_dma support (it didn't handle deferred callbacks and EINPROGRESS at all). It also did not pre-allocate dma maps but attempted to create them on the fly (despite a comment claiming that it did pre-allocate them). It now probes disks using a configintrhook rather than using polled commands. The drive number for each logical disk is passed to the child disk device via ivars. I've added bus_dma sync ops for the hardware QCB accesses. I've also reworked it's queuing mechanism so that it should now queue multiple commands to the controller (before it only queued one command at a time even though it had free QCBs). The patch compiles, but I have no hardware to test it. Given that this is an old driver, if I can't find anyone to test it, I will remove it from HEAD after committing the fixes (so if someone shows up in the future there is a better base to start from). http://www.FreeBSD.org/~jhb/patches/ida_locking_dma.patch -- John Baldwin