diff options
author | Vlad Yasevich <vladislav.yasevich@hp.com> | 2006-10-30 18:55:11 -0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2006-10-30 18:55:11 -0800 |
commit | de76e695a5ce19c121ba7e246b45f258be678a75 (patch) | |
tree | 6a1a715574371cf4b8b84ef1a4e84d3b14c6fd77 /net/sctp/input.c | |
parent | 4f4443088b763ca4ac7521e9b4a881b52c294dec (diff) | |
download | op-kernel-dev-de76e695a5ce19c121ba7e246b45f258be678a75.zip op-kernel-dev-de76e695a5ce19c121ba7e246b45f258be678a75.tar.gz |
[SCTP]: Remove temporary associations from backlog and hash.
Every time SCTP creates a temporary association, the stack hashes it,
puts it on a list of endpoint associations and increments the backlog.
However, the lifetime of a temporary association is the processing time
of a current packet and it's destroyed after that. In fact, we don't
really want anyone else finding this association. There is no reason to
do this extra work.
Signed-off-by: Vlad Yasevich <vladislav.yasevich@hp.com>
Signed-off-by: Sridhar Samudrala <sri@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sctp/input.c')
-rw-r--r-- | net/sctp/input.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/net/sctp/input.c b/net/sctp/input.c index 99c0501..6d82f40 100644 --- a/net/sctp/input.c +++ b/net/sctp/input.c @@ -771,6 +771,9 @@ static void __sctp_hash_established(struct sctp_association *asoc) /* Add an association to the hash. Local BH-safe. */ void sctp_hash_established(struct sctp_association *asoc) { + if (asoc->temp) + return; + sctp_local_bh_disable(); __sctp_hash_established(asoc); sctp_local_bh_enable(); @@ -804,6 +807,9 @@ static void __sctp_unhash_established(struct sctp_association *asoc) /* Remove association from the hash table. Local BH-safe. */ void sctp_unhash_established(struct sctp_association *asoc) { + if (asoc->temp) + return; + sctp_local_bh_disable(); __sctp_unhash_established(asoc); sctp_local_bh_enable(); |