summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSebastian Huber <sebastian.huber@embedded-brains.de>2016-04-27 16:20:05 +0200
committerSebastian Huber <sebastian.huber@embedded-brains.de>2016-05-02 07:46:14 +0200
commitb0eba5ed8af2b7e877fac9205f7b09417e33cf37 (patch)
tree354342c165e9daa60d258eb4e08f29a4b2c610d2
parentscore: _CORE_message_queue_Close() (diff)
downloadrtems-b0eba5ed8af2b7e877fac9205f7b09417e33cf37.tar.bz2
score: _CORE_message_queue_Seize()
Move lock acquire to caller of _CORE_message_queue_Seize() to allow state checks during receive operations under lock protection.
-rw-r--r--cpukit/posix/src/mqueuerecvsupp.c5
-rw-r--r--cpukit/rtems/src/msgqreceive.c5
-rw-r--r--cpukit/score/src/coremsgseize.c1
3 files changed, 10 insertions, 1 deletions
diff --git a/cpukit/posix/src/mqueuerecvsupp.c b/cpukit/posix/src/mqueuerecvsupp.c
index bbbc23443c..ecdadb344e 100644
--- a/cpukit/posix/src/mqueuerecvsupp.c
+++ b/cpukit/posix/src/mqueuerecvsupp.c
@@ -96,6 +96,11 @@ ssize_t _POSIX_Message_queue_Receive_support(
else
do_wait = wait;
+ _CORE_message_queue_Acquire_critical(
+ &the_mq->Message_queue,
+ &lock_context
+ );
+
/*
* Now perform the actual message receive
*/
diff --git a/cpukit/rtems/src/msgqreceive.c b/cpukit/rtems/src/msgqreceive.c
index 3b4945e776..e3b3466954 100644
--- a/cpukit/rtems/src/msgqreceive.c
+++ b/cpukit/rtems/src/msgqreceive.c
@@ -68,6 +68,11 @@ rtems_status_code rtems_message_queue_receive(
else
wait = true;
+ _CORE_message_queue_Acquire_critical(
+ &the_message_queue->message_queue,
+ &lock_context
+ );
+
executing = _Thread_Executing;
_CORE_message_queue_Seize(
&the_message_queue->message_queue,
diff --git a/cpukit/score/src/coremsgseize.c b/cpukit/score/src/coremsgseize.c
index 5e14918d3a..2471b918a9 100644
--- a/cpukit/score/src/coremsgseize.c
+++ b/cpukit/score/src/coremsgseize.c
@@ -40,7 +40,6 @@ void _CORE_message_queue_Seize(
CORE_message_queue_Buffer_control *the_message;
executing->Wait.return_code = CORE_MESSAGE_QUEUE_STATUS_SUCCESSFUL;
- _CORE_message_queue_Acquire_critical( the_message_queue, lock_context );
the_message = _CORE_message_queue_Get_pending_message( the_message_queue );
if ( the_message != NULL ) {
the_message_queue->number_of_pending_messages -= 1;