summaryrefslogtreecommitdiffstats
path: root/sys/kern
diff options
context:
space:
mode:
authorrwatson <rwatson@FreeBSD.org>2004-12-23 01:16:21 +0000
committerrwatson <rwatson@FreeBSD.org>2004-12-23 01:16:21 +0000
commitf1732152a77581f239cb2068299c68f6cb782116 (patch)
treef356a537ab8ade4f8b3b98b421560e9d900a02da /sys/kern
parent5728709bda70f4ee28c63807da07323e0b7e8ff0 (diff)
downloadFreeBSD-src-f1732152a77581f239cb2068299c68f6cb782116.zip
FreeBSD-src-f1732152a77581f239cb2068299c68f6cb782116.tar.gz
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.
Diffstat (limited to 'sys/kern')
-rw-r--r--sys/kern/uipc_sockbuf.c10
-rw-r--r--sys/kern/uipc_socket2.c10
2 files changed, 10 insertions, 10 deletions
diff --git a/sys/kern/uipc_sockbuf.c b/sys/kern/uipc_sockbuf.c
index f82052d..d683692 100644
--- a/sys/kern/uipc_sockbuf.c
+++ b/sys/kern/uipc_sockbuf.c
@@ -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;
diff --git a/sys/kern/uipc_socket2.c b/sys/kern/uipc_socket2.c
index f82052d..d683692 100644
--- a/sys/kern/uipc_socket2.c
+++ b/sys/kern/uipc_socket2.c
@@ -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;
OpenPOWER on IntegriCloud