git: 1d8d5348645f - stable/13 - sctp: remove book keeping not needed anymore
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 01 Feb 2023 23:22:35 UTC
The branch stable/13 has been updated by tuexen:
URL: https://cgit.FreeBSD.org/src/commit/?id=1d8d5348645f3dcdc65562c4ad9765cb62156251
commit 1d8d5348645f3dcdc65562c4ad9765cb62156251
Author: Michael Tuexen <tuexen@FreeBSD.org>
AuthorDate: 2022-06-08 21:30:52 +0000
Commit: Michael Tuexen <tuexen@FreeBSD.org>
CommitDate: 2023-02-01 23:22:04 +0000
sctp: remove book keeping not needed anymore
(cherry picked from commit ee9ee699d6b5ee7613c6a166d2db5dbb59389cfc)
---
sys/netinet/sctp_output.c | 38 ++------------------------------------
1 file changed, 2 insertions(+), 36 deletions(-)
diff --git a/sys/netinet/sctp_output.c b/sys/netinet/sctp_output.c
index 7dcd2eab8ed2..6736f7e9f948 100644
--- a/sys/netinet/sctp_output.c
+++ b/sys/netinet/sctp_output.c
@@ -12440,7 +12440,6 @@ sctp_lower_sosend(struct socket *so,
bool free_cnt_applied = false;
bool some_on_control;
bool got_all_of_the_send = false;
- bool hold_tcblock = false;
bool non_blocking = false;
error = 0;
@@ -12555,7 +12554,6 @@ sctp_lower_sosend(struct socket *so,
stcb = LIST_FIRST(&inp->sctp_asoc_list);
if (stcb != NULL) {
SCTP_TCB_LOCK(stcb);
- hold_tcblock = true;
}
SCTP_INP_RUNLOCK(inp);
} else if (sinfo_assoc_id > SCTP_ALL_ASSOC) {
@@ -12563,7 +12561,6 @@ sctp_lower_sosend(struct socket *so,
SCTP_INP_RUNLOCK(inp);
if (stcb != NULL) {
SCTP_TCB_LOCK_ASSERT(stcb);
- hold_tcblock = true;
}
} else if (addr != NULL) {
/*-
@@ -12580,7 +12577,6 @@ sctp_lower_sosend(struct socket *so,
SCTP_INP_WUNLOCK(inp);
} else {
SCTP_TCB_LOCK_ASSERT(stcb);
- hold_tcblock = true;
}
} else {
SCTP_INP_RUNLOCK(inp);
@@ -12589,11 +12585,9 @@ sctp_lower_sosend(struct socket *so,
#ifdef INVARIANTS
if (stcb != NULL) {
SCTP_TCB_LOCK_ASSERT(stcb);
- KASSERT(hold_tcblock, ("tcb lock hold, hold_tcblock is false"));
- } else {
- KASSERT(!hold_tcblock, ("hold_tcblock is true, but stcb is NULL"));
}
#endif
+
if ((stcb == NULL) && (addr != NULL)) {
/* Possible implicit send? */
SCTP_ASOC_CREATE_LOCK(inp);
@@ -12624,7 +12618,6 @@ sctp_lower_sosend(struct socket *so,
SCTP_INP_WUNLOCK(inp);
} else {
SCTP_TCB_LOCK_ASSERT(stcb);
- hold_tcblock = true;
SCTP_ASOC_CREATE_UNLOCK(inp);
create_lock_applied = false;
}
@@ -12667,7 +12660,6 @@ sctp_lower_sosend(struct socket *so,
goto out_unlocked;
}
SCTP_TCB_LOCK_ASSERT(stcb);
- hold_tcblock = true;
SCTP_ASOC_CREATE_UNLOCK(inp);
create_lock_applied = false;
/*
@@ -12681,7 +12673,6 @@ sctp_lower_sosend(struct socket *so,
if (sctp_process_cmsgs_for_init(stcb, control, &error)) {
sctp_free_assoc(inp, stcb, SCTP_NORMAL_PROC,
SCTP_FROM_SCTP_OUTPUT + SCTP_LOC_6);
- hold_tcblock = false;
stcb = NULL;
KASSERT(error != 0,
("error is 0 although sctp_process_cmsgs_for_init() indicated an error"));
@@ -12701,7 +12692,6 @@ sctp_lower_sosend(struct socket *so,
KASSERT(!create_lock_applied, ("create_lock_applied is true"));
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
asoc = &stcb->asoc;
@@ -12805,7 +12795,6 @@ sctp_lower_sosend(struct socket *so,
local_soresv = sndlen;
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -12861,10 +12850,8 @@ sctp_lower_sosend(struct socket *so,
SCTP_BUF_LEN(mm) = (int)(tot_out + sizeof(struct sctp_paramhdr));
if (top == NULL) {
SCTP_TCB_UNLOCK(stcb);
- hold_tcblock = false;
error = uiomove((caddr_t)ph, (int)tot_out, uio);
SCTP_TCB_LOCK(stcb);
- hold_tcblock = true;
if ((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) ||
(asoc->state & SCTP_STATE_WAS_ABORTED)) {
sctp_m_freem(mm);
@@ -12913,7 +12900,6 @@ sctp_lower_sosend(struct socket *so,
}
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -12972,7 +12958,6 @@ sctp_lower_sosend(struct socket *so,
be.error = 0;
stcb->block_entry = &be;
SCTP_TCB_UNLOCK(stcb);
- hold_tcblock = false;
error = sbwait(&so->so_snd);
if (error == 0) {
if (so->so_error != 0) {
@@ -12984,7 +12969,6 @@ sctp_lower_sosend(struct socket *so,
}
SOCKBUF_UNLOCK(&so->so_snd);
SCTP_TCB_LOCK(stcb);
- hold_tcblock = true;
stcb->block_entry = NULL;
if (error != 0) {
goto out_unlocked;
@@ -13019,7 +13003,6 @@ sctp_lower_sosend(struct socket *so,
skip_preblock:
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -13054,10 +13037,8 @@ skip_preblock:
if (strm->last_msg_incomplete == 0) {
do_a_copy_in:
SCTP_TCB_UNLOCK(stcb);
- hold_tcblock = false;
sp = sctp_copy_it_in(stcb, asoc, sndrcvninfo, uio, net, max_len, user_marks_eor, &error);
SCTP_TCB_LOCK(stcb);
- hold_tcblock = true;
if ((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) ||
(asoc->state & SCTP_STATE_WAS_ABORTED)) {
if (asoc->state & SCTP_STATE_WAS_ABORTED) {
@@ -13142,7 +13123,6 @@ skip_preblock:
}
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -13163,12 +13143,10 @@ skip_preblock:
((max_len > 0) && (SCTP_SB_LIMIT_SND(so) < SCTP_BASE_SYSCTL(sctp_add_more_threshold))) ||
(uio->uio_resid <= max_len)) {
SCTP_TCB_UNLOCK(stcb);
- hold_tcblock = false;
sndout = 0;
new_tail = NULL;
mm = sctp_copy_resume(uio, (int)max_len, user_marks_eor, &error, &sndout, &new_tail);
SCTP_TCB_LOCK(stcb);
- hold_tcblock = true;
if ((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) ||
(asoc->state & SCTP_STATE_WAS_ABORTED)) {
/*
@@ -13224,7 +13202,6 @@ skip_preblock:
}
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -13354,7 +13331,6 @@ skip_preblock:
be.error = 0;
stcb->block_entry = &be;
SCTP_TCB_UNLOCK(stcb);
- hold_tcblock = false;
error = sbwait(&so->so_snd);
if (error == 0) {
if (so->so_error != 0)
@@ -13365,7 +13341,6 @@ skip_preblock:
}
SOCKBUF_UNLOCK(&so->so_snd);
SCTP_TCB_LOCK(stcb);
- hold_tcblock = true;
stcb->block_entry = NULL;
if ((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) ||
(asoc->state & SCTP_STATE_WAS_ABORTED)) {
@@ -13397,7 +13372,6 @@ skip_preblock:
}
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -13440,7 +13414,6 @@ skip_preblock:
dataless_eof:
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -13517,7 +13490,6 @@ dataless_eof:
sctp_abort_an_association(stcb->sctp_ep, stcb,
op_err, false, SCTP_SO_LOCKED);
NET_EPOCH_EXIT(et);
- hold_tcblock = false;
stcb = NULL;
error = ECONNABORTED;
goto out;
@@ -13531,7 +13503,6 @@ dataless_eof:
skip_out_eof:
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -13553,7 +13524,6 @@ skip_out_eof:
}
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -13603,7 +13573,6 @@ skip_out_eof:
}
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -13634,7 +13603,6 @@ skip_out_eof:
asoc->total_output_queue_size, error);
KASSERT(stcb != NULL, ("stcb is NULL"));
- KASSERT(hold_tcblock, ("hold_tcblock is false"));
SCTP_TCB_LOCK_ASSERT(stcb);
KASSERT((asoc->state & SCTP_STATE_ABOUT_TO_BE_FREED) == 0,
("Association about to be freed"));
@@ -13650,12 +13618,10 @@ out_unlocked:
if (local_soresv) {
atomic_subtract_int(&asoc->sb_send_resv, (int)sndlen);
}
- if (hold_tcblock) {
- SCTP_TCB_UNLOCK(stcb);
- }
if (free_cnt_applied) {
atomic_subtract_int(&asoc->refcnt, 1);
}
+ SCTP_TCB_UNLOCK(stcb);
}
if (top != NULL) {
sctp_m_freem(top);