diff options
| author |   Xin Long <lucien.xin@gmail.com> | 2019-07-03 18:20:20 +0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-07-03 11:41:24 -0700 |
| commit | 7af033010214f2c7cff31147d8970484d46cc14c (patch) | |
| tree | 436fefa8124f6fa86ea352fa1adee13b1fb440c7 /net/sctp | |
| parent | r8152: move calling r8153b_rx_agg_chg_indicate() (diff) | |
| download | linux-dev-7af033010214f2c7cff31147d8970484d46cc14c.tar.xz linux-dev-7af033010214f2c7cff31147d8970484d46cc14c.zip | |
sctp: count data bundling sack chunk for outctrlchunks
Now all ctrl chunks are counted for asoc stats.octrlchunks and net
SCTP_MIB_OUTCTRLCHUNKS either after queuing up or bundling, other
than the chunk maked and bundled in sctp_packet_bundle_sack, which
caused 'outctrlchunks' not consistent with 'inctrlchunks' in peer.
This issue exists since very beginning, here to fix it by increasing
both net SCTP_MIB_OUTCTRLCHUNKS and asoc stats.octrlchunks when sack
chunk is maked and bundled in sctp_packet_bundle_sack.
Reported-by: Ja Ram Jeon <jajeon@redhat.com>
Signed-off-by: Xin Long <lucien.xin@gmail.com>
Acked-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sctp')
| -rw-r--r-- | net/sctp/output.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/net/sctp/output.c b/net/sctp/output.c index e0c27477788d..dbda7e7927fd 100644 --- a/net/sctp/output.c +++ b/net/sctp/output.c @@ -282,6 +282,9 @@ static enum sctp_xmit sctp_packet_bundle_sack(struct sctp_packet *pkt, sctp_chunk_free(sack); goto out; } + SCTP_INC_STATS(sock_net(asoc->base.sk), + SCTP_MIB_OUTCTRLCHUNKS); + asoc->stats.octrlchunks++; asoc->peer.sack_needed = 0; if (del_timer(timer)) sctp_association_put(asoc); |