From owner-freebsd-current@FreeBSD.ORG Thu Jun 5 05:55:27 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 2FAAA1065672; Thu, 5 Jun 2008 05:55:27 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from harmony.bsdimp.com (bsdimp.com [199.45.160.85]) by mx1.freebsd.org (Postfix) with ESMTP id D405F8FC1B; Thu, 5 Jun 2008 05:55:26 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from localhost (localhost [127.0.0.1]) by harmony.bsdimp.com (8.14.2/8.14.1) with ESMTP id m555qZSP008230; Wed, 4 Jun 2008 23:52:36 -0600 (MDT) (envelope-from imp@bsdimp.com) Date: Wed, 04 Jun 2008 23:53:52 -0600 (MDT) Message-Id: <20080604.235352.-1384056146.imp@bsdimp.com> To: jhb@FreeBSD.org From: "M. Warner Losh" In-Reply-To: <200806041126.28347.jhb@freebsd.org> References: <20080604.001253.1683325662.imp@bsdimp.com> <20080604.011215.1723237511.imp@bsdimp.com> <200806041126.28347.jhb@freebsd.org> X-Mailer: Mew version 5.2 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: current@FreeBSD.org Subject: Re: cs(4) MPSAFE patch 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: Thu, 05 Jun 2008 05:55:27 -0000 In message: <200806041126.28347.jhb@freebsd.org> John Baldwin writes: : On Wednesday 04 June 2008 03:12:15 am M. Warner Losh wrote: : > In message: <20080604.001253.1683325662.imp@bsdimp.com> : > "M. Warner Losh" writes: : > : In message: <200805281715.39536.jhb@freebsd.org> : > : John Baldwin writes: : > : : I have a patch to make cs(4) MPSAFE. As with previous patches, : ENOHARDWARE. : > : : If you care about this driver, test the patch. : > : : : > : : http://www.FreeBSD.org/~jhb/patches/cs.patch : > : : > : This patch appears to work for me, but I've not stress tested the : > : interface. : > : : > : I did have to make a bunch of other changes to the driver to fix some : > : obvious bogons that got in my way. These bogons are in the cs driver, : > : not in your patches to the cs driver. I'm flushing those into svn and : > : looping back the changes plus your patches to make sure things still : > : work. I'll report more when I know more. Note: this is PCMCIA card : > : only (IBM EtherJet Ethernet PC Card). : > : : > : I can confirm that your patch still applies after my changes, although : > : with a fuzz factor. : > : > Upon further testing, I've discovered that the driver neglects to tear : > down the ISR. : > : > http://people.freebsd.org/~imp/cs.patch.new : > : > fixes the problem, but may also have some stray goo in it too. : : I've updated cs.patch, I just put the teardown in detach as that is the only : place it is needed (other drivers are similar). Also, I broke the watchdog : timer in the patch you have, so you may want to fetch the updated patch for : that. Thanks for testing this. : : -- : John Baldwin : : In message: <200806041126.28347.jhb@freebsd.org> John Baldwin writes: : On Wednesday 04 June 2008 03:12:15 am M. Warner Losh wrote: : > In message: <20080604.001253.1683325662.imp@bsdimp.com> : > "M. Warner Losh" writes: : > : In message: <200805281715.39536.jhb@freebsd.org> : > : John Baldwin writes: : > : : I have a patch to make cs(4) MPSAFE. As with previous patches, : ENOHARDWARE. : > : : If you care about this driver, test the patch. : > : : : > : : http://www.FreeBSD.org/~jhb/patches/cs.patch : > : : > : This patch appears to work for me, but I've not stress tested the : > : interface. : > : : > : I did have to make a bunch of other changes to the driver to fix some : > : obvious bogons that got in my way. These bogons are in the cs driver, : > : not in your patches to the cs driver. I'm flushing those into svn and : > : looping back the changes plus your patches to make sure things still : > : work. I'll report more when I know more. Note: this is PCMCIA card : > : only (IBM EtherJet Ethernet PC Card). : > : : > : I can confirm that your patch still applies after my changes, although : > : with a fuzz factor. : > : > Upon further testing, I've discovered that the driver neglects to tear : > down the ISR. : > : > http://people.freebsd.org/~imp/cs.patch.new : > : > fixes the problem, but may also have some stray goo in it too. : : I've updated cs.patch, I just put the teardown in detach as that is the only : place it is needed (other drivers are similar). Also, I broke the watchdog : timer in the patch you have, so you may want to fetch the updated patch for : that. Thanks for testing this. OK. I'm sure the new one works now. I had to fix a few other bugs in the driver to make it work. I've committed those fixes. Your fixes can go in at any time and we know that at least the PC Card parts will work (AFAIK, the IBM EtherJet was the only card to use this chip, although there appears to be at least one Japanese variant that used it too, but had the IBM EtherJet CIS loaded). Warner