From owner-freebsd-current@FreeBSD.ORG Fri Jul 25 20:16:51 2008 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 BA3531065679; Fri, 25 Jul 2008 20:16:51 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from server.baldwin.cx (bigknife-pt.tunnel.tserv9.chi1.ipv6.he.net [IPv6:2001:470:1f10:75::2]) by mx1.freebsd.org (Postfix) with ESMTP id 658FF8FC16; Fri, 25 Jul 2008 20:16:51 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from localhost.corp.yahoo.com (john@localhost [IPv6:::1]) (authenticated bits=0) by server.baldwin.cx (8.14.2/8.14.2) with ESMTP id m6PKGjtV073014; Fri, 25 Jul 2008 16:16:45 -0400 (EDT) (envelope-from jhb@freebsd.org) From: John Baldwin To: current@freebsd.org Date: Fri, 25 Jul 2008 16:16:30 -0400 User-Agent: KMail/1.9.7 MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200807251616.30249.jhb@freebsd.org> X-Greylist: Sender succeeded SMTP AUTH authentication, not delayed by milter-greylist-2.0.2 (server.baldwin.cx [IPv6:::1]); Fri, 25 Jul 2008 16:16:45 -0400 (EDT) X-Virus-Scanned: ClamAV 0.93.1/7827/Fri Jul 25 14:13:59 2008 on server.baldwin.cx X-Virus-Status: Clean X-Spam-Status: No, score=-2.5 required=4.2 tests=AWL,BAYES_00,NO_RELAYS autolearn=ham version=3.1.3 X-Spam-Checker-Version: SpamAssassin 3.1.3 (2006-06-01) on server.baldwin.cx Cc: arm@freebsd.org Subject: PATCH: iicbus(4) locking, please test 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: Fri, 25 Jul 2008 20:16:51 -0000 I have a first attempt at adding locking to iicbus(4). Originally my goal was to do this so if_ic(4) could be locked. However, if_ic(4) does evil things in its ioctl routine so I'm actually going to leave that alone unless someone says they want to test possible changes. Back to iicbus(4), this patch adds locking to the core of iicbus and the various related drivers. A few drivers (ixp_iic, lpbb, and bktr) just use Giant for their driver lock for now. It has been compiled on my laptop (i386) but not anything else yet. http://www.FreeBSD.org/~jhb/patches/iicbus.patch -- John Baldwin