Author: hselasky
Date: Wed Nov 19 09:07:49 2014
New Revision: 274705
URL: https://svnweb.freebsd.org/changeset/base/274705
Log:
MFC r274376:
Fix some minor TSO issues:
- Improve description of TSO limits.
- Remove a not needed KASSERT()
- Remove some not needed variable casts.
Sponsored by: Mellanox Technologies
Modified:
stable/9/sys/net/if.c
stable/9/sys/net/if_var.h
stable/9/sys/netinet/tcp_output.c
Directory Properties:
stable/9/sys/ (props changed)
stable/9/sys/net/ (props changed)
Modified: stable/9/sys/net/if.c
==============================================================================
--- stable/9/sys/net/if.c Wed Nov 19 09:03:12 2014 (r274704)
+++ stable/9/sys/net/if.c Wed Nov 19 09:07:49 2014 (r274705)
@@ -747,13 +747,6 @@ if_attach_internal(struct ifnet *ifp, in
ifp->if_hw_tsomaxsegsize);
}
}
- /*
- * If the "if_hw_tsomax" limit is set, check if it is
- * too small:
- */
- KASSERT(ifp->if_hw_tsomax == 0 ||
- ifp->if_hw_tsomax >= (IP_MAXPACKET / 8),
- ("%s: if_hw_tsomax is outside of range", __func__));
#endif
}
#ifdef VIMAGE
Modified: stable/9/sys/net/if_var.h
==============================================================================
--- stable/9/sys/net/if_var.h Wed Nov 19 09:03:12 2014 (r274704)
+++ stable/9/sys/net/if_var.h Wed Nov 19 09:07:49 2014 (r274705)
@@ -211,11 +211,24 @@ struct ifnet {
* be used with care where binary compatibility is required.
*/
char if_cspare[3];
- u_int if_hw_tsomax; /* TSO total burst length
- * limit in bytes. A value of
- * zero means no limit. Have
- * to find a better place for
- * it eventually. */
+
+ /*
+ * Network adapter TSO limits:
+ * ===========================
+ *
+ * If the "if_hw_tsomax" field is zero the maximum segment
+ * length limit does not apply. If the "if_hw_tsomaxsegcount"
+ * or the "if_hw_tsomaxsegsize" field is zero the TSO segment
+ * count limit does not apply. If all three fields are zero,
+ * there is no TSO limit.
+ *
+ * NOTE: The TSO limits only apply to the data payload part of
+ * a TCP/IP packet. That means there is no need to subtract
+ * space for ethernet-, vlan-, IP- or TCP- headers from the
+ * TSO limits unless the hardware driver in question requires
+ * so.
+ */
+ u_int if_hw_tsomax;
int if_ispare[1];
/*
* TSO fields for segment limits. If a field is zero below,
Modified: stable/9/sys/netinet/tcp_output.c
==============================================================================
--- stable/9/sys/netinet/tcp_output.c Wed Nov 19 09:03:12 2014
(r274704)
+++ stable/9/sys/netinet/tcp_output.c Wed Nov 19 09:07:49 2014
(r274705)
@@ -794,9 +794,9 @@ send:
max_len = (if_hw_tsomax - hdrlen);
if (max_len <= 0) {
len = 0;
- } else if (len > (u_int)max_len) {
+ } else if (len > max_len) {
sendalot = 1;
- len = (u_int)max_len;
+ len = max_len;
}
}
@@ -809,7 +809,7 @@ send:
max_len = 0;
mb = sbsndmbuf(&so->so_snd, off, &moff);
- while (mb != NULL && (u_int)max_len < len) {
+ while (mb != NULL && max_len < len) {
u_int mlen;
u_int frags;
@@ -843,9 +843,9 @@ send:
}
if (max_len <= 0) {
len = 0;
- } else if (len > (u_int)max_len) {
+ } else if (len > max_len) {
sendalot = 1;
- len = (u_int)max_len;
+ len = max_len;
}
}
@@ -856,7 +856,7 @@ send:
*/
max_len = (tp->t_maxopd - optlen);
if ((off + len) < so->so_snd.sb_cc) {
- moff = len % (u_int)max_len;
+ moff = len % max_len;
if (moff != 0) {
len -= moff;
sendalot = 1;
@@ -867,8 +867,8 @@ send:
* In case there are too many small fragments
* don't use TSO:
*/
- if (len <= (u_int)max_len) {
- len = (u_int)max_len;
+ if (len <= max_len) {
+ len = max_len;
sendalot = 1;
tso = 0;
}
_______________________________________________
svn-src-stable-9@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-stable-9
To unsubscribe, send any mail to "svn-src-stable-9-unsubscr...@freebsd.org"
