From owner-p4-projects@FreeBSD.ORG Sat Mar 3 20:52:50 2007 Return-Path: X-Original-To: p4-projects@freebsd.org Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 4021B16A403; Sat, 3 Mar 2007 20:52:50 +0000 (UTC) X-Original-To: perforce@freebsd.org Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id E1E7F16A400; Sat, 3 Mar 2007 20:52:49 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from harmony.bsdimp.com (vc4-2-0-87.dsl.netrack.net [199.45.160.85]) by mx1.freebsd.org (Postfix) with ESMTP id A1FB213C442; Sat, 3 Mar 2007 20:52:49 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from localhost (localhost [127.0.0.1]) by harmony.bsdimp.com (8.13.4/8.13.4) with ESMTP id l23KoW9C044476; Sat, 3 Mar 2007 13:50:33 -0700 (MST) (envelope-from imp@bsdimp.com) Date: Sat, 03 Mar 2007 13:50:40 -0700 (MST) Message-Id: <20070303.135040.179932401.imp@bsdimp.com> To: piso@freebsd.org From: "M. Warner Losh" In-Reply-To: <200703031614.l23GE0QR013965@repoman.freebsd.org> References: <200703031614.l23GE0QR013965@repoman.freebsd.org> X-Mailer: Mew version 4.2 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-2.0 (harmony.bsdimp.com [127.0.0.1]); Sat, 03 Mar 2007 13:50:33 -0700 (MST) Cc: perforce@freebsd.org Subject: Re: PERFORCE change 115279 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 03 Mar 2007 20:52:50 -0000 In message: <200703031614.l23GE0QR013965@repoman.freebsd.org> Paolo Pisati writes: : Restore pccard ability to support filtered drivers: : : o check for filter and ithread handlers : o fix pccard_intr() return code Good catch... I think we should re-evaluate how CSC interrupts are handled for CardBus and PC Card in general, which might simplify things in this area of the code. It was originally put into place to handle sudden, surprise removal of the cards in a (mostly) sane way. Now that the underlying reasons for having these checks are gone, it would make things better if this could be investigated... I'll have to add it to my list... Warner