On 30.11.2018 21:54, Brooks Davis wrote:
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
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.
Submitted by: Ivan Malov <Ivan.Malov at oktetlabs.ru>
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 committed
a few hours later without any feedback, etc.)
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.
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.
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.
We have discussed it with Philip (philip@) shortly. As I understand
he has no time now to review it.
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.
It makes sense. As I understand the argument from George
was just give it a chance to be reviewed within FreeBSD process.
Andrew.
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
