From owner-svn-src-stable-8@FreeBSD.ORG Sat Sep 10 11:57:44 2011 Return-Path: Delivered-To: svn-src-stable-8@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 5FDCC106581F; Sat, 10 Sep 2011 11:57:44 +0000 (UTC) (envelope-from kostikbel@gmail.com) Received: from mail.zoral.com.ua (mx0.zoral.com.ua [91.193.166.200]) by mx1.freebsd.org (Postfix) with ESMTP id C89678FC15; Sat, 10 Sep 2011 11:57:43 +0000 (UTC) Received: from deviant.kiev.zoral.com.ua (root@deviant.kiev.zoral.com.ua [10.1.1.148]) by mail.zoral.com.ua (8.14.2/8.14.2) with ESMTP id p8ABVtVM043283 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sat, 10 Sep 2011 14:31:55 +0300 (EEST) (envelope-from kostikbel@gmail.com) Received: from deviant.kiev.zoral.com.ua (kostik@localhost [127.0.0.1]) by deviant.kiev.zoral.com.ua (8.14.4/8.14.4) with ESMTP id p8ABVte0058357; Sat, 10 Sep 2011 14:31:55 +0300 (EEST) (envelope-from kostikbel@gmail.com) Received: (from kostik@localhost) by deviant.kiev.zoral.com.ua (8.14.4/8.14.4/Submit) id p8ABVtA7058356; Sat, 10 Sep 2011 14:31:55 +0300 (EEST) (envelope-from kostikbel@gmail.com) X-Authentication-Warning: deviant.kiev.zoral.com.ua: kostik set sender to kostikbel@gmail.com using -f Date: Sat, 10 Sep 2011 14:31:55 +0300 From: Kostik Belousov To: "Bjoern A. Zeeb" Message-ID: <20110910113155.GJ17489@deviant.kiev.zoral.com.ua> References: <201109090744.p897iE9x027234@svn.freebsd.org> <28707761-AA24-43D5-AE8B-B0D8842C6F13@lists.zabbadoz.net> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="7m2Ta+5QzzfNUVnL" Content-Disposition: inline In-Reply-To: <28707761-AA24-43D5-AE8B-B0D8842C6F13@lists.zabbadoz.net> User-Agent: Mutt/1.4.2.3i X-Virus-Scanned: clamav-milter 0.95.2 at skuns.kiev.zoral.com.ua X-Virus-Status: Clean X-Spam-Status: No, score=-3.3 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00, DNS_FROM_OPENWHOIS autolearn=no version=3.2.5 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on skuns.kiev.zoral.com.ua Cc: src-committers@freebsd.org, Hans Petter Selasky , svn-src-stable@freebsd.org, svn-src-all@freebsd.org, Robert Watson , svn-src-stable-8@freebsd.org Subject: Re: svn commit: r225458 - in stable/8/sys: dev/usb dev/usb/quirk dev/usb/storage sys X-BeenThere: svn-src-stable-8@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for only the 8-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 10 Sep 2011 11:57:44 -0000 --7m2Ta+5QzzfNUVnL Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Sep 10, 2011 at 11:21:00AM +0000, Bjoern A. Zeeb wrote: >=20 > On Sep 10, 2011, at 10:54 AM, Robert Watson wrote: >=20 > > On Fri, 9 Sep 2011, Hans Petter Selasky wrote: > >=20 > >> Log: > >> MFC r225350 and r225400: > >>=20 > >> This patch adds automatic detection of USB mass storage devices > >> which does not support the no synchronize cache SCSI command. > >>=20 > >> The __FreeBSD_version version macro has been bumped and > >> external kernel modules needs to be recompiled after > >> this patch. > >>=20 > >> PR: usb/160299 > >=20 > > For most other classes of hardware device driver framework KPIs -- espe= cially things like PCI bus attachment, busdma, CAM, ifnet, and GEOM framewo= rks, our MFC rules would strictly disallow this sort of change, on the grou= nds that it is our KBI policy that we not break common classes of third-par= ty device drivers (i.e., require them to be recompiled). My suspicion is t= hat we should be applying the same rules to the USB framework -- however, I= don't know if we have any third-party USB device drivers? > >=20 > > (If we do, then this change should not have been MFC'd.) >=20 > We do have FreeBSD consumers with private USB drivers, yes. It seems that most of the damage can be mitigated by placing the added fields at the end of the structures, at least for stable/8. --7m2Ta+5QzzfNUVnL Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (FreeBSD) iEYEARECAAYFAk5rSqoACgkQC3+MBN1Mb4it4wCgp9EuWsJWR3iiFemfYXRCaZqE ULIAnim6ed6WfR5PsJZgPsG4Z75+IQjZ =HOTy -----END PGP SIGNATURE----- --7m2Ta+5QzzfNUVnL--