From owner-svn-src-all@FreeBSD.ORG Thu Mar 5 03:43:59 2009 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id B471E1065672; Thu, 5 Mar 2009 03:43:59 +0000 (UTC) (envelope-from rnoland@FreeBSD.org) Received: from gizmo.2hip.net (gizmo.2hip.net [64.74.207.195]) by mx1.freebsd.org (Postfix) with ESMTP id 5CAA48FC1B; Thu, 5 Mar 2009 03:43:59 +0000 (UTC) (envelope-from rnoland@FreeBSD.org) Received: from [192.168.1.2] (adsl-154-199-160.ard.bellsouth.net [72.154.199.160]) (authenticated bits=0) by gizmo.2hip.net (8.14.3/8.14.3) with ESMTP id n253gaE2069589 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 4 Mar 2009 22:42:36 -0500 (EST) (envelope-from rnoland@FreeBSD.org) From: Robert Noland To: "M. Warner Losh" In-Reply-To: <20090304.194158.1159134197.imp@bsdimp.com> References: <200903041823.n24INmcc049524@svn.freebsd.org> <20090304.135606.1649768480.imp@bsdimp.com> <1236218572.1384.19.camel@widget.2hip.net> <20090304.194158.1159134197.imp@bsdimp.com> Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-S/RAOrr6zvYInzWMUC7w" Organization: FreeBSD Date: Wed, 04 Mar 2009 21:43:49 -0600 Message-Id: <1236224629.1384.22.camel@widget.2hip.net> Mime-Version: 1.0 X-Mailer: Evolution 2.24.4 FreeBSD GNOME Team Port X-Spam-Status: No, score=-1.6 required=5.0 tests=AWL,BAYES_00,RCVD_IN_PBL, RCVD_IN_SORBS_DUL,RDNS_DYNAMIC autolearn=no version=3.2.5 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on gizmo.2hip.net Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r189367 - head/sys/dev/pci X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 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: Thu, 05 Mar 2009 03:44:00 -0000 --=-S/RAOrr6zvYInzWMUC7w Content-Type: text/plain Content-Transfer-Encoding: quoted-printable On Wed, 2009-03-04 at 19:41 -0700, M. Warner Losh wrote: > In message: <1236218572.1384.19.camel@widget.2hip.net> > Robert Noland writes: > : On Wed, 2009-03-04 at 13:56 -0700, M. Warner Losh wrote: > : > In message: <200903041823.n24INmcc049524@svn.freebsd.org> > : > Robert Noland writes: > : > : Author: rnoland > : > : Date: Wed Mar 4 18:23:48 2009 > : > : New Revision: 189367 > : > : URL: http://svn.freebsd.org/changeset/base/189367 > : > :=20 > : > : Log: > : > : Extend the management of PCIM_CMD_INTxDIS. > : > : =20 > : > : We now explicitly enable INTx during bus_setup_intr() if it is ne= eded. > : > : Several of the ata drivers were managing this bit internally. Th= is is > : > : better handled in pci and it should work for all drivers now. > : > : =20 > : > : We also mask INTx during bus_teardown_intr() by setting this bit. > : > : =20 > : > : Reviewed by: jhb > : > : MFC after: 3 days > : >=20 > : > Note: the INTxDIS bit is new in PCI 3.0, and has no effect on earlier > : > devices. This should be highlighted in the comments somewhere... > :=20 > : It is documented in 2.3 as well, I'm not sure about previous versions o= f > : the spec though. >=20 > It isn't in 2.2, and even after 2.3 it is "optional". The bit should be unused if it isn't supported by a given piece of hardware. If it doesn't do anything, we are no worse off than before. I don't think this will cause any harm, only goodness when it is supported. robert. > Warner --=20 Robert Noland FreeBSD --=-S/RAOrr6zvYInzWMUC7w Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.10 (FreeBSD) iEYEABECAAYFAkmvSnUACgkQM4TrQ4qfRONodgCfXG9uEeviU8kvytHg8tIQLns3 Nn4AniorNYMTQFFuGeoxK2N4/iGz+pwJ =QsuV -----END PGP SIGNATURE----- --=-S/RAOrr6zvYInzWMUC7w--