c768021b jhb Dec. 28, 2021, 5:43 p.m.
No functional change.

Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D33639
cgit
c50827a6 jhb Dec. 28, 2021, 5:42 p.m.
No functional change.

Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D33638
cgit
5a37560e jhb Dec. 28, 2021, 5:42 p.m.
No functional change.

Reviewed by:	imp, emaste
Differential Revision:	https://reviews.freebsd.org/D33637
cgit
f2b1ec3a jhb Dec. 28, 2021, 5:41 p.m.
No functional change.

Reviewed by:	imp, emaste
Differential Revision:	https://reviews.freebsd.org/D33636
cgit
0ff783dc jhb Dec. 28, 2021, 5:41 p.m.
No functional change.

Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D33635
cgit
db269074 jhb Dec. 28, 2021, 5:41 p.m.
No functional change.

Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D33634
cgit
e7d4d580 jhb Dec. 28, 2021, 5:41 p.m.
No functional change.

Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D33633
cgit
cc5aa0a4 jhb Dec. 28, 2021, 5:41 p.m.
No functional change.

Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D33632
cgit
326d5782 jhb Dec. 28, 2021, 5:41 p.m.
No functional change.

Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D33631
cgit
2cee5861 jhb Dec. 28, 2021, 5:41 p.m.
No functional change.

Reviewed by:	imp, kib
Differential Revision:	https://reviews.freebsd.org/D33630
cgit
4287aa56 glebius Dec. 28, 2021, 4:50 p.m.
While here move out one more erroneous condition out of the epoch and
common return.  The only functional change is that if we send control
on a shut down socket we would get EINVAL instead of ECONNRESET.

Reviewed by:	tuexen
Reported by:	syzbot+8388cf7f401a7b6bece6@syzkaller.appspotmail.com
Fixes:		f64dc2ab5be38e5366271ef85ea90d8cb1c7841a
cgit
f7926a6d vmaffione Dec. 28, 2021, 11:11 a.m.
The logic that sets iri_vtag and M_VLANTAG does not handle the
case where the 802.11q VLAN tag is 0. Fix this issue across
the iflib drivers. While there, also improve and align the
VLAN tag check extraction, by moving it outside the RX descriptor
loop, eliminating a local variable and additional checks.

PR:             260068
Reviewed by:    kbowling, gallatin
Reported by:	erj
MFC after:      1 month
Differential Revision:  https://reviews.freebsd.org/D33156
cgit
52f45d8a vmaffione Dec. 28, 2021, 11:03 a.m.
Since isc_capenable (private copy of ifp->if_capenable) is
now synchronized to if_capenable, use it in the drivers
when checking the IFCAP_* bits.
This results in better cache usage and avoids indirection
through the ifp pointer.

PR:             260068
Reviewed by:    kbowling, gallatin
MFC after:      1 week
Differential Revision:  https://reviews.freebsd.org/D33156
cgit
4561c4f0 vmaffione Dec. 28, 2021, 10:55 a.m.
On SIOCSIFCAP, some bits in ifp->if_capenable may be toggled.
When this happens, apply the same change to isc_capenable, which
is the iflib private copy of if_capenable (for a subset of the
IFCAP_* bits). In this way the iflib drivers can check the bits
using isc_capenable rather than if_capenable. This is convenient
because the latter access requires an additional indirection
through the ifp, and it is also less likely to be in cache.

PR:		260068
Reviewed by:	kbowling, gallatin
MFC after:	1 week
Differential Revision:	https://reviews.freebsd.org/D33156
cgit
a7ba00a4 tuexen Dec. 28, 2021, 9:23 a.m.
MFC after:	3 days
cgit