From owner-svn-src-head@freebsd.org Fri Mar 11 17:28:31 2016 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 31D19ACB9BA; Fri, 11 Mar 2016 17:28:31 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id 15F7A914; Fri, 11 Mar 2016 17:28:31 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [IPv6:::1]) by freefall.freebsd.org (Postfix) with ESMTP id 079B0111A; Fri, 11 Mar 2016 17:28:31 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [172.31.3.2]) by mail.xzibition.com (Postfix) with ESMTP id C9CB61E09C; Fri, 11 Mar 2016 17:28:30 +0000 (UTC) X-Virus-Scanned: amavisd-new at mail.xzibition.com Received: from mail.xzibition.com ([172.31.3.2]) by mail.xzibition.com (mail.xzibition.com [172.31.3.2]) (amavisd-new, port 10026) with LMTP id q3lTgiCmS9aF; Fri, 11 Mar 2016 17:28:28 +0000 (UTC) Subject: Re: svn commit: r296589 - head/sys/dev/fdc DKIM-Filter: OpenDKIM Filter v2.9.2 mail.xzibition.com C67411E097 To: Warner Losh References: <201603100033.u2A0X6uN027771@repo.freebsd.org> <56E1F72D.7000002@FreeBSD.org> Cc: src-committers , "svn-src-head@freebsd.org" , "svn-src-all@freebsd.org" , Warner Losh From: Bryan Drewery Openpgp: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Organization: FreeBSD Message-ID: <56E30041.3020804@FreeBSD.org> Date: Fri, 11 Mar 2016 09:28:33 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="22grvUsp1kwLkApMC3HVEJH8q6leL9f61" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Mar 2016 17:28:31 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --22grvUsp1kwLkApMC3HVEJH8q6leL9f61 Content-Type: multipart/mixed; boundary="uxkN9qKeQ4cTsvmAR5wwJL38LswhouETs" From: Bryan Drewery To: Warner Losh Cc: src-committers , "svn-src-head@freebsd.org" , "svn-src-all@freebsd.org" , Warner Losh Message-ID: <56E30041.3020804@FreeBSD.org> Subject: Re: svn commit: r296589 - head/sys/dev/fdc References: <201603100033.u2A0X6uN027771@repo.freebsd.org> <56E1F72D.7000002@FreeBSD.org> In-Reply-To: --uxkN9qKeQ4cTsvmAR5wwJL38LswhouETs Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 3/10/2016 9:14 PM, Warner Losh wrote: >=20 >=20 > On Thu, Mar 10, 2016 at 6:58 PM, Warner Losh > wrote: >=20 >=20 > On Mar 10, 2016 3:37 PM, "Bryan Drewery" > wrote: > > > > On 3/9/16 4:33 PM, Warner Losh wrote: > > > Author: imp > > > Date: Thu Mar 10 00:33:06 2016 > > > New Revision: 296589 > > > URL: https://svnweb.freebsd.org/changeset/base/296589 > > > > > > Log: > > > Stop assuming that bio_cmd is a bit field. > > > > > > Differential Revision: https://reviews.freebsd.org/D5587 > > > > > > Modified: > > > head/sys/dev/fdc/fdc.c > > > > > > Modified: head/sys/dev/fdc/fdc.c > > > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D > > > --- head/sys/dev/fdc/fdc.c Thu Mar 10 00:27:10 2016 =20 > (r296588) > > > +++ head/sys/dev/fdc/fdc.c Thu Mar 10 00:33:06 2016 =20 > (r296589) > > > @@ -941,7 +941,7 @@ fdc_worker(struct fdc_data *fdc) > > > /* Disable ISADMA if we bailed while it was active */ > > > if (fd !=3D NULL && (fd->flags & FD_ISADMA)) { > > > isa_dmadone( > > > - bp->bio_cmd & BIO_READ ? ISADMA_READ : > ISADMA_WRITE, > > > + bp->bio_cmd =3D=3D BIO_READ ? ISADMA_READ : > ISADMA_WRITE, > > > > I think we should have some kind of file (like ports CHANGES) tha= t > lists > > subtle KPI changes. This and the bio bzero change were easily mi= ssed > > and could lead to who-knows-what downstream for vendors or even > > out-of-tree modules. >=20 > True. However, these have never been documented one way or another.= =2E.. >=20 > And this change isn't a change yet... >=20 > I'd love a kpi change file. This is but one of many. We'd need > someone clueful to watch the tree and remind people to add things t= o it. >=20 > I'm also working on documenting our storage api so that people know= > better what is defined, vs what's there and subject to change. >=20 > Re-reading this, I wasn't very clear: >=20 > I think we need this file. > I think we need someone else (not me) to spearhead it and police change= s > I think that the sooner we start the better. > Can I get a volunteer? I don't mind policing it. I'll bring it up on arch@ just to be sure no one objects for some reason. It was also pointed out to me that bio_cmd was never a bitfield and lacked documentation. However I still think it is worth having a file like this. Even my share/mk changes have been subtle enough to break downstream where I don't feel UPDATING is the proper place. --=20 Regards, Bryan Drewery --uxkN9qKeQ4cTsvmAR5wwJL38LswhouETs-- --22grvUsp1kwLkApMC3HVEJH8q6leL9f61 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBAgAGBQJW4wBCAAoJEDXXcbtuRpfPu7oH/0uMnK8t0dIFYwaWV9Tm/G+O X+izHjCQ/9sk7m4K0y7XX+cZS0ODepKoTdTACmFsjvk3cTmiqQ7d8rDPhUsHT+jj zch5j1wnsJOxsLtU3VWKTIJKunwhTzLo0pIPNxOceJLv1iFLv/SDMVy+0WYdCQMM 9jnxa/8Oi8mE4CfGqvQsQAG//qsAfH+TO5cXQPqMNxHslpTOzL1jhoniYjNZiSN5 +5yZ66kCMMsnwmtfRpmZEZZHq54rcVIFd2gKkgRrRw8cUBVh+gbagGpF1S1D94+m pm1shWW2OGi6RcTUUQaF5HkQuKczoHAufQAE8OyAPiWIMwUm8SmDwHGV13hlN8U= =aezR -----END PGP SIGNATURE----- --22grvUsp1kwLkApMC3HVEJH8q6leL9f61--