diff options
author | Gleb Smirnoff <glebius@FreeBSD.org> | 2015-01-12 18:06:22 +0000 |
---|---|---|
committer | Gleb Smirnoff <glebius@FreeBSD.org> | 2015-01-12 18:06:22 +0000 |
commit | fc2517100bc337274155f1ca7fd98456914df236 (patch) | |
tree | 0341fd44dd08540d27da1ca8e9c575e3f6c9964c /sys | |
parent | ffec6ee52747c31748c2149cb6aa7c72e8352490 (diff) | |
download | src-fc2517100bc337274155f1ca7fd98456914df236.tar.gz src-fc2517100bc337274155f1ca7fd98456914df236.zip |
Do not go one layer down to check ifqueue length. First, not all drivers
use ifqueue at all. Second, there is no point in this lockless check.
Either positive or negative result of the check could be incorrect after
a tick.
Reviewed by: tuexen
Sponsored by: Nginx, Inc.
Notes
Notes:
svn path=/head/; revision=277077
Diffstat (limited to 'sys')
-rw-r--r-- | sys/netinet/sctp_output.c | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/sys/netinet/sctp_output.c b/sys/netinet/sctp_output.c index 59e4343e1b85..86aa3afdeab5 100644 --- a/sys/netinet/sctp_output.c +++ b/sys/netinet/sctp_output.c @@ -7958,22 +7958,6 @@ again_one_more_time: } else { skip_data_for_this_net = 0; } - if ((net->ro.ro_rt) && (net->ro.ro_rt->rt_ifp)) { - /* - * if we have a route and an ifp check to see if we - * have room to send to this guy - */ - struct ifnet *ifp; - - ifp = net->ro.ro_rt->rt_ifp; - if ((ifp->if_snd.ifq_len + 2) >= ifp->if_snd.ifq_maxlen) { - SCTP_STAT_INCR(sctps_ifnomemqueued); - if (SCTP_BASE_SYSCTL(sctp_logging_level) & SCTP_LOG_MAXBURST_ENABLE) { - sctp_log_maxburst(stcb, net, ifp->if_snd.ifq_len, ifp->if_snd.ifq_maxlen, SCTP_MAX_IFP_APPLIED); - } - continue; - } - } switch (((struct sockaddr *)&net->ro._l_addr)->sa_family) { #ifdef INET case AF_INET: |