Author: tuexen
Date: Fri May 29 12:03:02 2015
New Revision: 283708
URL: https://svnweb.freebsd.org/changeset/base/283708

Log:
  MFC r276914:
  
  Minimize the usage of SCTP_BUF_IS_EXTENDED.
  This should help Robert...

Modified:
  stable/10/sys/netinet/sctp_bsd_addr.c
  stable/10/sys/netinet/sctp_indata.c
  stable/10/sys/netinet/sctp_input.c
  stable/10/sys/netinet/sctp_output.c
  stable/10/sys/netinet/sctputil.c
  stable/10/sys/netinet/sctputil.h
  stable/10/sys/netinet6/sctp6_usrreq.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/netinet/sctp_bsd_addr.c
==============================================================================
--- stable/10/sys/netinet/sctp_bsd_addr.c       Fri May 29 11:52:32 2015        
(r283707)
+++ stable/10/sys/netinet/sctp_bsd_addr.c       Fri May 29 12:03:02 2015        
(r283708)
@@ -402,9 +402,7 @@ sctp_get_mbuf_for_msg(unsigned int space
        }
 #ifdef SCTP_MBUF_LOGGING
        if (SCTP_BASE_SYSCTL(sctp_logging_level) & SCTP_MBUF_LOGGING_ENABLE) {
-               if (SCTP_BUF_IS_EXTENDED(m)) {
-                       sctp_log_mb(m, SCTP_MBUF_IALLOC);
-               }
+               sctp_log_mb(m, SCTP_MBUF_IALLOC);
        }
 #endif
        return (m);

Modified: stable/10/sys/netinet/sctp_indata.c
==============================================================================
--- stable/10/sys/netinet/sctp_indata.c Fri May 29 11:52:32 2015        
(r283707)
+++ stable/10/sys/netinet/sctp_indata.c Fri May 29 12:03:02 2015        
(r283708)
@@ -1493,13 +1493,7 @@ sctp_process_a_data_chunk(struct sctp_tc
                    the_len, M_NOWAIT);
 #ifdef SCTP_MBUF_LOGGING
                if (SCTP_BASE_SYSCTL(sctp_logging_level) & 
SCTP_MBUF_LOGGING_ENABLE) {
-                       struct mbuf *mat;
-
-                       for (mat = dmbuf; mat; mat = SCTP_BUF_NEXT(mat)) {
-                               if (SCTP_BUF_IS_EXTENDED(mat)) {
-                                       sctp_log_mb(mat, SCTP_MBUF_ICOPY);
-                               }
-                       }
+                       sctp_log_mbc(dmbuf, SCTP_MBUF_ICOPY);
                }
 #endif
        } else {

Modified: stable/10/sys/netinet/sctp_input.c
==============================================================================
--- stable/10/sys/netinet/sctp_input.c  Fri May 29 11:52:32 2015        
(r283707)
+++ stable/10/sys/netinet/sctp_input.c  Fri May 29 12:03:02 2015        
(r283708)
@@ -2443,13 +2443,7 @@ sctp_handle_cookie_echo(struct mbuf *m, 
        }
 #ifdef SCTP_MBUF_LOGGING
        if (SCTP_BASE_SYSCTL(sctp_logging_level) & SCTP_MBUF_LOGGING_ENABLE) {
-               struct mbuf *mat;
-
-               for (mat = m_sig; mat; mat = SCTP_BUF_NEXT(mat)) {
-                       if (SCTP_BUF_IS_EXTENDED(mat)) {
-                               sctp_log_mb(mat, SCTP_MBUF_SPLIT);
-                       }
-               }
+               sctp_log_mbc(m_sig, SCTP_MBUF_SPLIT);
        }
 #endif
 
@@ -5527,13 +5521,7 @@ process_control_chunks:
                                                } else {
 #ifdef SCTP_MBUF_LOGGING
                                                        if 
(SCTP_BASE_SYSCTL(sctp_logging_level) & SCTP_MBUF_LOGGING_ENABLE) {
-                                                               struct mbuf 
*mat;
-
-                                                               for (mat = 
SCTP_BUF_NEXT(mm); mat; mat = SCTP_BUF_NEXT(mat)) {
-                                                                       if 
(SCTP_BUF_IS_EXTENDED(mat)) {
-                                                                               
sctp_log_mb(mat, SCTP_MBUF_ICOPY);
-                                                                       }
-                                                               }
+                                                               
sctp_log_mbc(SCTP_BUF_NEXT(mm), SCTP_MBUF_ICOPY);
                                                        }
 #endif
                                                        sctp_queue_op_err(stcb, 
mm);
@@ -6038,13 +6026,7 @@ sctp_input_with_port(struct mbuf *i_pak,
 #ifdef SCTP_MBUF_LOGGING
        /* Log in any input mbufs */
        if (SCTP_BASE_SYSCTL(sctp_logging_level) & SCTP_MBUF_LOGGING_ENABLE) {
-               struct mbuf *mat;
-
-               for (mat = m; mat; mat = SCTP_BUF_NEXT(mat)) {
-                       if (SCTP_BUF_IS_EXTENDED(mat)) {
-                               sctp_log_mb(mat, SCTP_MBUF_INPUT);
-                       }
-               }
+               sctp_log_mbc(m, SCTP_MBUF_INPUT);
        }
 #endif
 #ifdef SCTP_PACKET_LOGGING

Modified: stable/10/sys/netinet/sctp_output.c
==============================================================================
--- stable/10/sys/netinet/sctp_output.c Fri May 29 11:52:32 2015        
(r283707)
+++ stable/10/sys/netinet/sctp_output.c Fri May 29 12:03:02 2015        
(r283708)
@@ -3846,13 +3846,7 @@ sctp_add_cookie(struct mbuf *init, int i
        }
 #ifdef SCTP_MBUF_LOGGING
        if (SCTP_BASE_SYSCTL(sctp_logging_level) & SCTP_MBUF_LOGGING_ENABLE) {
-               struct mbuf *mat;
-
-               for (mat = copy_init; mat; mat = SCTP_BUF_NEXT(mat)) {
-                       if (SCTP_BUF_IS_EXTENDED(mat)) {
-                               sctp_log_mb(mat, SCTP_MBUF_ICOPY);
-                       }
-               }
+               sctp_log_mbc(copy_init, SCTP_MBUF_ICOPY);
        }
 #endif
        copy_initack = SCTP_M_COPYM(initack, initack_offset, M_COPYALL,
@@ -3864,13 +3858,7 @@ sctp_add_cookie(struct mbuf *init, int i
        }
 #ifdef SCTP_MBUF_LOGGING
        if (SCTP_BASE_SYSCTL(sctp_logging_level) & SCTP_MBUF_LOGGING_ENABLE) {
-               struct mbuf *mat;
-
-               for (mat = copy_initack; mat; mat = SCTP_BUF_NEXT(mat)) {
-                       if (SCTP_BUF_IS_EXTENDED(mat)) {
-                               sctp_log_mb(mat, SCTP_MBUF_ICOPY);
-                       }
-               }
+               sctp_log_mbc(copy_initack, SCTP_MBUF_ICOPY);
        }
 #endif
        /* easy side we just drop it on the end */
@@ -6482,13 +6470,7 @@ error_out:
                        appendchain = SCTP_M_COPYM(clonechain, 0, M_COPYALL, 
M_NOWAIT);
 #ifdef SCTP_MBUF_LOGGING
                        if (SCTP_BASE_SYSCTL(sctp_logging_level) & 
SCTP_MBUF_LOGGING_ENABLE) {
-                               struct mbuf *mat;
-
-                               for (mat = appendchain; mat; mat = 
SCTP_BUF_NEXT(mat)) {
-                                       if (SCTP_BUF_IS_EXTENDED(mat)) {
-                                               sctp_log_mb(mat, 
SCTP_MBUF_ICOPY);
-                                       }
-                               }
+                               sctp_log_mbc(appendchain, SCTP_MBUF_ICOPY);
                        }
 #endif
                }
@@ -6581,13 +6563,7 @@ sctp_sendall_iterator(struct sctp_inpcb 
                }
 #ifdef SCTP_MBUF_LOGGING
                if (SCTP_BASE_SYSCTL(sctp_logging_level) & 
SCTP_MBUF_LOGGING_ENABLE) {
-                       struct mbuf *mat;
-
-                       for (mat = m; mat; mat = SCTP_BUF_NEXT(mat)) {
-                               if (SCTP_BUF_IS_EXTENDED(mat)) {
-                                       sctp_log_mb(mat, SCTP_MBUF_ICOPY);
-                               }
-                       }
+                       sctp_log_mbc(m, SCTP_MBUF_ICOPY);
                }
 #endif
        } else {
@@ -7371,13 +7347,7 @@ dont_do_it:
                }
 #ifdef SCTP_MBUF_LOGGING
                if (SCTP_BASE_SYSCTL(sctp_logging_level) & 
SCTP_MBUF_LOGGING_ENABLE) {
-                       struct mbuf *mat;
-
-                       for (mat = chk->data; mat; mat = SCTP_BUF_NEXT(mat)) {
-                               if (SCTP_BUF_IS_EXTENDED(mat)) {
-                                       sctp_log_mb(mat, SCTP_MBUF_ICOPY);
-                               }
-                       }
+                       sctp_log_mbc(chk->data, SCTP_MBUF_ICOPY);
                }
 #endif
                /* Pull off the data */
@@ -8973,13 +8943,7 @@ sctp_send_cookie_echo(struct mbuf *m,
                        }
 #ifdef SCTP_MBUF_LOGGING
                        if (SCTP_BASE_SYSCTL(sctp_logging_level) & 
SCTP_MBUF_LOGGING_ENABLE) {
-                               struct mbuf *mat;
-
-                               for (mat = cookie; mat; mat = 
SCTP_BUF_NEXT(mat)) {
-                                       if (SCTP_BUF_IS_EXTENDED(mat)) {
-                                               sctp_log_mb(mat, 
SCTP_MBUF_ICOPY);
-                                       }
-                               }
+                               sctp_log_mbc(cookie, SCTP_MBUF_ICOPY);
                        }
 #endif
                        break;
@@ -9045,13 +9009,7 @@ sctp_send_heartbeat_ack(struct sctp_tcb 
        }
 #ifdef SCTP_MBUF_LOGGING
        if (SCTP_BASE_SYSCTL(sctp_logging_level) & SCTP_MBUF_LOGGING_ENABLE) {
-               struct mbuf *mat;
-
-               for (mat = outchain; mat; mat = SCTP_BUF_NEXT(mat)) {
-                       if (SCTP_BUF_IS_EXTENDED(mat)) {
-                               sctp_log_mb(mat, SCTP_MBUF_ICOPY);
-                       }
-               }
+               sctp_log_mbc(outchain, SCTP_MBUF_ICOPY);
        }
 #endif
        chdr = mtod(outchain, struct sctp_chunkhdr *);
@@ -9330,13 +9288,7 @@ sctp_send_asconf_ack(struct sctp_tcb *st
                }
 #ifdef SCTP_MBUF_LOGGING
                if (SCTP_BASE_SYSCTL(sctp_logging_level) & 
SCTP_MBUF_LOGGING_ENABLE) {
-                       struct mbuf *mat;
-
-                       for (mat = m_ack; mat; mat = SCTP_BUF_NEXT(mat)) {
-                               if (SCTP_BUF_IS_EXTENDED(mat)) {
-                                       sctp_log_mb(mat, SCTP_MBUF_ICOPY);
-                               }
-                       }
+                       sctp_log_mbc(m_ack, SCTP_MBUF_ICOPY);
                }
 #endif
 

Modified: stable/10/sys/netinet/sctputil.c
==============================================================================
--- stable/10/sys/netinet/sctputil.c    Fri May 29 11:52:32 2015        
(r283707)
+++ stable/10/sys/netinet/sctputil.c    Fri May 29 12:03:02 2015        
(r283708)
@@ -217,6 +217,7 @@ sctp_log_fr(uint32_t biggest_tsn, uint32
            sctp_clog.x.misc.log4);
 }
 
+#ifdef SCTP_MBUF_LOGGING
 void
 sctp_log_mb(struct mbuf *m, int from)
 {
@@ -243,6 +244,18 @@ sctp_log_mb(struct mbuf *m, int from)
 }
 
 void
+sctp_log_mbc(struct mbuf *m, int from)
+{
+       struct mbuf *mat;
+
+       for (mat = m; mat; mat = SCTP_BUF_NEXT(mat)) {
+               sctp_log_mb(mat, from);
+       }
+}
+
+#endif
+
+void
 sctp_log_strm_del(struct sctp_queued_to_read *control, struct 
sctp_queued_to_read *poschk, int from)
 {
        struct sctp_cwnd_log sctp_clog;
@@ -413,7 +426,8 @@ sctp_log_rwnd_set(uint8_t from, uint32_t
            sctp_clog.x.misc.log4);
 }
 
-void
+#ifdef SCTP_MBCNT_LOGGING
+static void
 sctp_log_mbcnt(uint8_t from, uint32_t total_oq, uint32_t book, uint32_t 
total_mbcnt_q, uint32_t mbcnt)
 {
        struct sctp_cwnd_log sctp_clog;
@@ -431,6 +445,8 @@ sctp_log_mbcnt(uint8_t from, uint32_t to
            sctp_clog.x.misc.log4);
 }
 
+#endif
+
 void
 sctp_misc_ints(uint8_t from, uint32_t a, uint32_t b, uint32_t c, uint32_t d)
 {
@@ -6157,9 +6173,7 @@ struct mbuf *
 sctp_m_free(struct mbuf *m)
 {
        if (SCTP_BASE_SYSCTL(sctp_logging_level) & SCTP_MBUF_LOGGING_ENABLE) {
-               if (SCTP_BUF_IS_EXTENDED(m)) {
-                       sctp_log_mb(m, SCTP_MBUF_IFREE);
-               }
+               sctp_log_mb(m, SCTP_MBUF_IFREE);
        }
        return (m_free(m));
 }

Modified: stable/10/sys/netinet/sctputil.h
==============================================================================
--- stable/10/sys/netinet/sctputil.h    Fri May 29 11:52:32 2015        
(r283707)
+++ stable/10/sys/netinet/sctputil.h    Fri May 29 12:03:02 2015        
(r283708)
@@ -349,10 +349,16 @@ void sctp_log_strm_del_alt(struct sctp_t
 void sctp_log_nagle_event(struct sctp_tcb *stcb, int action);
 
 
+#ifdef SCTP_MBUF_LOGGING
 void
      sctp_log_mb(struct mbuf *m, int from);
 
 void
+     sctp_log_mbc(struct mbuf *m, int from);
+
+#endif
+
+void
 sctp_sblog(struct sockbuf *sb,
     struct sctp_tcb *stcb, int from, int incr);
 
@@ -369,7 +375,6 @@ void sctp_log_lock(struct sctp_inpcb *in
 void sctp_log_maxburst(struct sctp_tcb *stcb, struct sctp_nets *, int, int, 
uint8_t);
 void sctp_log_block(uint8_t, struct sctp_association *, int);
 void sctp_log_rwnd(uint8_t, uint32_t, uint32_t, uint32_t);
-void sctp_log_mbcnt(uint8_t, uint32_t, uint32_t, uint32_t, uint32_t);
 void sctp_log_rwnd_set(uint8_t, uint32_t, uint32_t, uint32_t, uint32_t);
 int sctp_fill_stat_log(void *, size_t *);
 void sctp_log_fr(uint32_t, uint32_t, uint32_t, int);

Modified: stable/10/sys/netinet6/sctp6_usrreq.c
==============================================================================
--- stable/10/sys/netinet6/sctp6_usrreq.c       Fri May 29 11:52:32 2015        
(r283707)
+++ stable/10/sys/netinet6/sctp6_usrreq.c       Fri May 29 12:03:02 2015        
(r283708)
@@ -94,13 +94,7 @@ sctp6_input_with_port(struct mbuf **i_pa
 #ifdef SCTP_MBUF_LOGGING
        /* Log in any input mbufs */
        if (SCTP_BASE_SYSCTL(sctp_logging_level) & SCTP_MBUF_LOGGING_ENABLE) {
-               struct mbuf *mat;
-
-               for (mat = m; mat; mat = SCTP_BUF_NEXT(mat)) {
-                       if (SCTP_BUF_IS_EXTENDED(mat)) {
-                               sctp_log_mb(mat, SCTP_MBUF_INPUT);
-                       }
-               }
+               sctp_log_mbc(m, SCTP_MBUF_INPUT);
        }
 #endif
 #ifdef SCTP_PACKET_LOGGING
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to