diff options
| author | Corinna Vinschen <corinna@vinschen.de> | 2004-10-04 19:44:05 +0000 |
|---|---|---|
| committer | Corinna Vinschen <corinna@vinschen.de> | 2004-10-04 19:44:05 +0000 |
| commit | 1f8b30497dba80363a1e10af5dd7366156456a0e (patch) | |
| tree | a8d17990e1eae19f29c620f1e49112cc1fc33afd /winsup/cygserver/sem.cc | |
| parent | 3ea9de7644c3110f6f8394501cd26a38852c95e9 (diff) | |
| download | cygnal-1f8b30497dba80363a1e10af5dd7366156456a0e.tar.gz cygnal-1f8b30497dba80363a1e10af5dd7366156456a0e.tar.bz2 cygnal-1f8b30497dba80363a1e10af5dd7366156456a0e.zip | |
* bsd_mutex.cc (_msleep): Simplify event creation. Revert change from
2004-08-24. It should be unnecessary now.
* msg.cc (client_request_msg::serve): Release process critical
section as early as possible.
* sem.cc (client_request_sem::serve): Ditto.
* shm.cc (client_request_shm::serve): Ditto.
* process.cc: Use hold and release method calls instead of
EnterCriticalSection/LeaveCriticalSection calls throughout.
* process.h (_hold): Rename from hold. Take filename and linenumber
parameter for logging. Define matching hold macro.
(release): Ditto.
Diffstat (limited to 'winsup/cygserver/sem.cc')
| -rw-r--r-- | winsup/cygserver/sem.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/winsup/cygserver/sem.cc b/winsup/cygserver/sem.cc index f0fbbac0b..37d894ef9 100644 --- a/winsup/cygserver/sem.cc +++ b/winsup/cygserver/sem.cc @@ -77,6 +77,8 @@ client_request_sem::serve (transport_layer_base *const conn, } /* Early revert_to_self since IPC code runs in kernel mode. */ conn->revert_to_self (); + /* sysv_sem.cc takes care of itself. */ + client->release (); thread td = { client, &_parameters.in.ipcblk, {-1, -1} }; int res; switch (_parameters.in.semop) @@ -98,7 +100,6 @@ client_request_sem::serve (transport_layer_base *const conn, /* Allocated by the call to adjust_identity_info(). */ if (_parameters.in.ipcblk.gidlist) free (_parameters.in.ipcblk.gidlist); - client->release (); error_code (res); _parameters.out.ret = td.td_retval[0]; msglen (sizeof (_parameters.out)); |