From 1f8b30497dba80363a1e10af5dd7366156456a0e Mon Sep 17 00:00:00 2001 From: Corinna Vinschen Date: Mon, 4 Oct 2004 19:44:05 +0000 Subject: * 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. --- winsup/cygserver/sem.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'winsup/cygserver/sem.cc') 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)); -- cgit v1.2.3