From owner-svn-src-all@freebsd.org Fri Nov 30 18:54:26 2018 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 9CD95114CE34; Fri, 30 Nov 2018 18:54:26 +0000 (UTC) (envelope-from brooks@spindle.one-eyed-alien.net) Received: from spindle.one-eyed-alien.net (spindle.one-eyed-alien.net [199.48.129.229]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id D88157107A; Fri, 30 Nov 2018 18:54:25 +0000 (UTC) (envelope-from brooks@spindle.one-eyed-alien.net) Received: by spindle.one-eyed-alien.net (Postfix, from userid 3001) id CD9A33C475F; Fri, 30 Nov 2018 18:54:18 +0000 (UTC) Date: Fri, 30 Nov 2018 18:54:18 +0000 From: Brooks Davis To: Andrew Rybchenko Cc: John Baldwin , gnn@FreeBSD.org, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org, philip@FreeBSD.org Subject: Re: svn commit: r341327 - head/sys/dev/sfxge Message-ID: <20181130185418.GA4183@spindle.one-eyed-alien.net> References: <201811300711.wAU7B5R6084752@repo.freebsd.org> <7545b110-4930-d49f-f99f-7ec17db7e7a7@oktetlabs.ru> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="ReaqsoxgOBHFXBhH" Content-Disposition: inline In-Reply-To: <7545b110-4930-d49f-f99f-7ec17db7e7a7@oktetlabs.ru> User-Agent: Mutt/1.9.4 (2018-02-28) X-Rspamd-Queue-Id: D88157107A X-Spamd-Result: default: False [-4.04 / 15.00]; ARC_NA(0.00)[]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; TO_MATCH_ENVRCPT_ALL(0.00)[]; NEURAL_HAM_LONG(-1.00)[-0.999,0]; MIME_GOOD(-0.20)[multipart/signed,text/plain]; DMARC_NA(0.00)[freebsd.org]; AUTH_NA(1.00)[]; IP_SCORE(-3.64)[ip: (-9.48), ipnet: 199.48.128.0/22(-4.74), asn: 36236(-3.92), country: US(-0.09)]; MX_GOOD(-0.01)[spindle.one-eyed-alien.net]; MX_MISSING(3.50)[requested record is not found]; RCPT_COUNT_SEVEN(0.00)[7]; NEURAL_HAM_SHORT(-0.98)[-0.982,0]; R_SPF_NA(0.00)[]; SIGNED_PGP(-2.00)[]; FORGED_SENDER(0.30)[brooks@freebsd.org,brooks@spindle.one-eyed-alien.net]; R_DKIM_NA(0.00)[]; RCVD_TLS_LAST(0.00)[]; ASN(0.00)[asn:36236, ipnet:199.48.128.0/22, country:US]; FROM_NEQ_ENVFROM(0.00)[brooks@freebsd.org,brooks@spindle.one-eyed-alien.net]; RCVD_COUNT_TWO(0.00)[2] X-Rspamd-Server: mx1.freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.29 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: Fri, 30 Nov 2018 18:54:27 -0000 --ReaqsoxgOBHFXBhH Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Nov 30, 2018 at 09:15:39PM +0300, Andrew Rybchenko wrote: > On 30.11.2018 20:30, John Baldwin wrote: > > On 11/29/18 11:11 PM, Andrew Rybchenko wrote: > >> Author: arybchik > >> Date: Fri Nov 30 07:11:05 2018 > >> New Revision: 341327 > >> URL: https://svnweb.freebsd.org/changeset/base/341327 > >> > >> Log: > >> sfxge(4): rollback last seen VLAN TCI if Tx packet is dropped > >> =20 > >> Early processing of a packet on transmit may change last seen > >> VLAN TCI in the queue context. If such a packet is eventually > >> dropped, last seen VLAN TCI must be set to its previous value. > >> =20 > >> Submitted by: Ivan Malov > >> Sponsored by: Solarflare Communications, Inc. > >> MFC after: 1 week > >> Differential Revision: https://reviews.freebsd.org/D18288 > > Just as a general comment. There's no point in creating a review in > > phabricator if you aren't going to get any actual review feedback via > > the tool. That just adds noise. (I've spotchecked a few of the recent > > sfxge commits and they all seem to create a review that then gets commi= tted > > a few hours later without any feedback, etc.) >=20 > All these changesets is the result of development in Solarflare. > All these changesets were reviewed internally and in fact many > have later fixes which are simply squashed in. >=20 > We have discussed it with George (gnn@) some time ago and > he asked to submit reviews anyway and wait at least a day or > two before commit. Yes in this particular case these 2 hundreds > of patches is the result of 2 years of development. So, I'd > waited some time and started to commit in blocks. >=20 > This time I've not included np@ and bz@ in reviewers since I've > not got reviewed before and it would be too much spam. >=20 > We have discussed it with Philip (philip@) shortly. As I understand > he has no time now to review it. >=20 > Basically I'm ready to follow any sensible policy. I don't think it > makes to wait forever. If there are any volunteers I'll be happy > to include more people in reviewers. You can't just throw things in phab and hope someone reviews them. If you want then to be reviewed you need to work to have the reviewed. That's true regardless of technology. In this case, it seems pretty clear they have been reviewed within Solarflare so I don't think they really need to be reviewed within FreeBSD processes before they land so long as they are well tested and don't break the build. -- Brooks --ReaqsoxgOBHFXBhH Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEcBAEBAgAGBQJcAYdaAAoJEKzQXbSebgfAo/oIAJTg2TXyWH3pCJnvNtm2jAbS t5YRRPbhhU+Z/1XGbYcQBQ2isTGIiuj4NW00iQRP3JrmOyCX4P+1k8g5CyW3C22G IxoeenQflHwimuuOsp+0Yi35kSCjzEv73i6+pq/e5bZhPulzTssSYxEdClWaNIOC 95pDW/ORiMsElQ0p1cI4oM+nN5mx8ndRk2M001+QEO5Nawm+6PRXRX1Yvww+hiRC uQNCywWu21/Y1gzZxOELsMGNTiwu5Yn4W3gJP74JK2Upc3SFAkFiTXG/JuDS8HUo jjIed8j6aYcRoI8iA/E8MVznsXoLyauSaKTBKq145Ivz+Nn3byNHJ8Ma3QQut3c= =wsDy -----END PGP SIGNATURE----- --ReaqsoxgOBHFXBhH--