Page MenuHomeFreeBSD
Files F143180735

D33268.id.diff
No OneTemporary
Actions

D33268.id.diff
View Options

diff --git a/sys/net/netisr.c b/sys/net/netisr.c
--- a/sys/net/netisr.c
+++ b/sys/net/netisr.c
@@ -700,9 +700,11 @@
static void
netisr_drain_proto_vnet(struct vnet *vnet, u_int proto)
{
+ struct epoch_tracker et;
struct netisr_workstream *nwsp;
struct netisr_work *npwp;
struct mbuf *m, *mp, *n, *ne;
+ struct ifnet *ifp;
u_int i;
KASSERT(vnet != NULL, ("%s: vnet is NULL", __func__));
@@ -723,11 +725,14 @@
*/
m = npwp->nw_head;
n = ne = NULL;
+ NET_EPOCH_ENTER(et);
while (m != NULL) {
mp = m;
m = m->m_nextpkt;
mp->m_nextpkt = NULL;
- if (mp->m_pkthdr.rcvif->if_vnet != vnet) {
+ if ((ifp = ifnet_byindexgen(mp->m_pkthdr.rcvidx,
+ mp->m_pkthdr.rcvgen)) != NULL &&
+ ifp->if_vnet != vnet) {
if (n == NULL) {
n = ne = mp;
} else {
@@ -736,10 +741,12 @@
}
continue;
}
- /* This is a packet in the selected vnet. Free it. */
+ /* This is a packet in the selected vnet, or belongs
+ to destroyed interface. Free it. */
npwp->nw_len--;
m_freem(mp);
}
+ NET_EPOCH_EXIT(et);
npwp->nw_head = n;
npwp->nw_tail = ne;
NWS_UNLOCK(nwsp);
@@ -913,8 +920,10 @@
if (local_npw.nw_head == NULL)
local_npw.nw_tail = NULL;
local_npw.nw_len--;
- VNET_ASSERT(m->m_pkthdr.rcvif != NULL,
- ("%s:%d rcvif == NULL: m=%p", __func__, __LINE__, m));
+ if (__predict_false(m_rcvif_restore(m) == NULL)) {
+ m_freem(m);
+ continue;
+ }
CURVNET_SET(m->m_pkthdr.rcvif->if_vnet);
netisr_proto[proto].np_handler(m);
CURVNET_RESTORE();
@@ -986,6 +995,7 @@
*dosignalp = 0;
if (npwp->nw_len < npwp->nw_qlimit) {
+ m_rcvif_serialize(m);
m->m_nextpkt = NULL;
if (npwp->nw_head == NULL) {
npwp->nw_head = m;

File Metadata

Mime Type
text/plain
Expires
Wed, Jan 28, 1:22 AM (19 h, 2 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
28061953
Default Alt Text
D33268.id.diff (1 KB)

Event Timeline