diff options
author | tanimura <tanimura@FreeBSD.org> | 2002-04-27 08:24:29 +0000 |
---|---|---|
committer | tanimura <tanimura@FreeBSD.org> | 2002-04-27 08:24:29 +0000 |
commit | dbb4756491715a06ce4578841f6eba43fc62fa70 (patch) | |
tree | 982f8e96f9de9e219deaa4ea138a8334dc086c3c /sys/netinet/accf_http.c | |
parent | bcaaa89ad095d7a175927dd7e531fa6f278ed18e (diff) | |
download | FreeBSD-src-dbb4756491715a06ce4578841f6eba43fc62fa70.zip FreeBSD-src-dbb4756491715a06ce4578841f6eba43fc62fa70.tar.gz |
Add a global sx sigio_lock to protect the pointer to the sigio object
of a socket. This avoids lock order reversal caused by locking a
process in pgsigio().
sowakeup() and the callers of it (sowwakeup, soisconnected, etc.) now
require sigio_lock to be locked. Provide sowwakeup_locked(),
soisconnected_locked(), and so on in case where we have to modify a
socket and wake up a process atomically.
Diffstat (limited to 'sys/netinet/accf_http.c')
-rw-r--r-- | sys/netinet/accf_http.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/sys/netinet/accf_http.c b/sys/netinet/accf_http.c index a9a8fb0..73c9af2 100644 --- a/sys/netinet/accf_http.c +++ b/sys/netinet/accf_http.c @@ -197,9 +197,11 @@ sohashttpget(struct socket *so, void *arg, int waitflag) fallout: DPRINT("fallout"); + SIGIO_SLOCK(); so->so_upcall = NULL; so->so_rcv.sb_flags &= ~SB_UPCALL; - soisconnected(so); + soisconnected_locked(so); + SIGIO_SUNLOCK(); return; } @@ -269,9 +271,11 @@ readmore: fallout: DPRINT("fallout"); + SIGIO_SLOCK(); so->so_upcall = NULL; so->so_rcv.sb_flags &= ~SB_UPCALL; - soisconnected(so); + soisconnected_locked(so); + SIGIO_SUNLOCK(); return; } @@ -339,8 +343,10 @@ readmore: return; gotit: + SIGIO_SLOCK(); so->so_upcall = NULL; so->so_rcv.sb_flags &= ~SB_UPCALL; - soisconnected(so); + soisconnected_locked(so); + SIGIO_SUNLOCK(); return; } |