From 11c40a39b67aba8a72760639162bc0367cd71a1a Mon Sep 17 00:00:00 2001 From: Robert Watson Date: Sat, 26 Jun 2004 17:12:29 +0000 Subject: [PATCH] Replace comment on spl state when calling soabort() with a comment on locking state. No socket locks should be held when calling soabort() as it will call into protocol code that may acquire socket locks. --- sys/kern/uipc_socket.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/sys/kern/uipc_socket.c b/sys/kern/uipc_socket.c index 21ed0766587..16cb49eb537 100644 --- a/sys/kern/uipc_socket.c +++ b/sys/kern/uipc_socket.c @@ -426,7 +426,10 @@ discard: } /* - * Must be called at splnet... + * soabort() must not be called with any socket locks held, as it calls + * into the protocol, which will call back into the socket code causing + * it to acquire additional socket locks that may cause recursion or lock + * order reversals. */ int soabort(so)