In sonewconn(), the s/if/while/ change to wait for room at the tail of

the accept queue is a feature, not a bug/issue, so remove the XXXRW
from the comment.
This commit is contained in:
Robert Watson 2004-12-23 01:16:21 +00:00
parent ba65391172
commit 1ef121cf6b
2 changed files with 10 additions and 10 deletions

View File

@ -256,11 +256,11 @@ sonewconn(head, connstatus)
head->so_qlen++;
} else {
/*
* XXXRW: Keep removing sockets from the head until there's
* room for us to insert on the tail. In pre-locking
* revisions, this was a simple if(), but as we could be
* racing with other threads and soabort() requires dropping
* locks, we must loop waiting for the condition to be true.
* Keep removing sockets from the head until there's room for
* us to insert on the tail. In pre-locking revisions, this
* was a simple if(), but as we could be racing with other
* threads and soabort() requires dropping locks, we must
* loop waiting for the condition to be true.
*/
while (head->so_incqlen > head->so_qlimit) {
struct socket *sp;

View File

@ -256,11 +256,11 @@ sonewconn(head, connstatus)
head->so_qlen++;
} else {
/*
* XXXRW: Keep removing sockets from the head until there's
* room for us to insert on the tail. In pre-locking
* revisions, this was a simple if(), but as we could be
* racing with other threads and soabort() requires dropping
* locks, we must loop waiting for the condition to be true.
* Keep removing sockets from the head until there's room for
* us to insert on the tail. In pre-locking revisions, this
* was a simple if(), but as we could be racing with other
* threads and soabort() requires dropping locks, we must
* loop waiting for the condition to be true.
*/
while (head->so_incqlen > head->so_qlimit) {
struct socket *sp;