Attachment 226456 Details for Bug 254695 – Patch for testing

Patch for testing

epoch.patch (text/plain), 1.93 KB, created by Michael Tuexen on 2021-07-14 12:28:45 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Michael Tuexen

Created: 2021-07-14 12:28:45 UTC

Size: 1.93 KB

patch

obsolete

>diff --git a/sys/netinet/tcp_lro.c b/sys/netinet/tcp_lro.c
>index 23e64b29b29..a9042c5f129 100644
>--- a/sys/netinet/tcp_lro.c
>+++ b/sys/netinet/tcp_lro.c
>@@ -1239,7 +1239,7 @@ tcp_lro_flush_tcphpts(struct lro_ctrl *lc, struct lro_entry *le)
> 	struct inpcb *inp;
> 	struct tcpcb *tp;
> 	struct mbuf **pp, *cmp, *mv_to;
>-	bool bpf_req, should_wake;
>+	bool bpf_req, should_wake, needs_epoch;
> 
> 	/* Check if packet doesn't belongs to our network interface. */
> 	if ((tcplro_stacks_wanting_mbufq == 0) ||
>@@ -1264,11 +1264,17 @@ tcp_lro_flush_tcphpts(struct lro_ctrl *lc, struct lro_entry *le)
> 	    IN6_IS_ADDR_UNSPECIFIED(&le->inner.data.s_addr.v6)))
> 		return (TCP_LRO_CANNOT);
> #endif
>+	needs_epoch = !(lc->ifp->if_flags & IFF_KNOWSEPOCH);
>+	if (needs_epoch)
>+		NET_EPOCH_ENTER(et);
> 	/* Lookup inp, if any. */
> 	inp = tcp_lro_lookup(lc->ifp,
> 	    (le->inner.data.lro_type == LRO_TYPE_NONE) ? &le->outer : &le->inner);
>-	if (inp == NULL)
>+	if (inp == NULL) {
>+		if (needs_epoch)
>+			NET_EPOCH_EXIT(et);
> 		return (TCP_LRO_CANNOT);
>+	}
> 
> 	counter_u64_add(tcp_inp_lro_locks_taken, 1);
> 
>@@ -1280,6 +1286,8 @@ tcp_lro_flush_tcphpts(struct lro_ctrl *lc, struct lro_entry *le)
> 	    (inp->inp_flags & (INP_DROPPED | INP_TIMEWAIT)) ||
> 	    (inp->inp_flags2 & INP_FREED)) {
> 		INP_WUNLOCK(inp);
>+		if (needs_epoch)
>+			NET_EPOCH_EXIT(et);
> 		return (TCP_LRO_CANNOT);
> 	}
> 	if ((inp->inp_irq_cpu_set == 0)  && (lc->lro_cpu_is_set == 1)) {
>@@ -1289,6 +1297,8 @@ tcp_lro_flush_tcphpts(struct lro_ctrl *lc, struct lro_entry *le)
> 	/* Check if the transport doesn't support the needed optimizations. */
> 	if ((inp->inp_flags2 & (INP_SUPPORTS_MBUFQ | INP_MBUF_ACKCMP)) == 0) {
> 		INP_WUNLOCK(inp);
>+		if (needs_epoch)
>+			NET_EPOCH_EXIT(et);
> 		return (TCP_LRO_CANNOT);
> 	}
> 
>@@ -1333,6 +1343,8 @@ tcp_lro_flush_tcphpts(struct lro_ctrl *lc, struct lro_entry *le)
> 	}
> 	if (inp != NULL)
> 		INP_WUNLOCK(inp);
>+	if (needs_epoch)
>+		NET_EPOCH_EXIT(et);
> 	return (0);	/* Success. */
> }
> #endif

Actions: View

Attachments on bug 254695: 223748 | 226456 | 226457