Semaphore.st
author Claus Gittinger <cg@exept.de>
Mon, 24 Jul 2017 23:06:52 +0200
changeset 22080 379aa47c41ad
parent 22076 a0ca167d3248
child 22093 be39998ccf1b
permissions -rw-r--r--
#REFACTORING by cg class: Semaphore duplicate code removed added: #waitUncountedWithTimeoutMs:state: comment/format in: #wait #waitUncounted #waitWithTimeoutMs: #wakeupWaiters changed: #critical:timeoutMs:ifBlocking: #waitTimeoutMSecs: #waitUncountedWithTimeoutMs: #waitWithTimeout: #waitWithTimeoutMs:state:
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
     1
"
a27a279701f8 Initial revision
claus
parents:
diff changeset
     2
 COPYRIGHT (c) 1993 by Claus Gittinger
6616
70d54ffebc91 valueOnUnwindDo: - ifCurtailed.
Claus Gittinger <cg@exept.de>
parents: 6615
diff changeset
     3
	      All Rights Reserved
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
     4
a27a279701f8 Initial revision
claus
parents:
diff changeset
     5
 This software is furnished under a license and may be used
a27a279701f8 Initial revision
claus
parents:
diff changeset
     6
 only in accordance with the terms of that license and with the
a27a279701f8 Initial revision
claus
parents:
diff changeset
     7
 inclusion of the above copyright notice.   This software may not
a27a279701f8 Initial revision
claus
parents:
diff changeset
     8
 be provided or otherwise made available to, or used by, any
a27a279701f8 Initial revision
claus
parents:
diff changeset
     9
 other person.  No title to or ownership of the software is
a27a279701f8 Initial revision
claus
parents:
diff changeset
    10
 hereby transferred.
a27a279701f8 Initial revision
claus
parents:
diff changeset
    11
"
5556
1056cc5d6ce0 category change
Claus Gittinger <cg@exept.de>
parents: 5256
diff changeset
    12
"{ Package: 'stx:libbasic' }"
1056cc5d6ce0 category change
Claus Gittinger <cg@exept.de>
parents: 5256
diff changeset
    13
19189
48f29d2c0455 #DOCUMENTATION
matilk
parents: 16903
diff changeset
    14
"{ NameSpace: Smalltalk }"
48f29d2c0455 #DOCUMENTATION
matilk
parents: 16903
diff changeset
    15
375
claus
parents: 362
diff changeset
    16
Object subclass:#Semaphore
19927
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
    17
	instanceVariableNames:'count waitingProcesses lastOwnerId name owner'
6616
70d54ffebc91 valueOnUnwindDo: - ifCurtailed.
Claus Gittinger <cg@exept.de>
parents: 6615
diff changeset
    18
	classVariableNames:''
70d54ffebc91 valueOnUnwindDo: - ifCurtailed.
Claus Gittinger <cg@exept.de>
parents: 6615
diff changeset
    19
	poolDictionaries:''
70d54ffebc91 valueOnUnwindDo: - ifCurtailed.
Claus Gittinger <cg@exept.de>
parents: 6615
diff changeset
    20
	category:'Kernel-Processes'
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
    21
!
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
    22
1812
a048a98aa299 kludge around a situation, where one process
Claus Gittinger <cg@exept.de>
parents: 1577
diff changeset
    23
!Semaphore class methodsFor:'documentation'!
69
4564b6328136 *** empty log message ***
claus
parents: 25
diff changeset
    24
88
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    25
copyright
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    26
"
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    27
 COPYRIGHT (c) 1993 by Claus Gittinger
6616
70d54ffebc91 valueOnUnwindDo: - ifCurtailed.
Claus Gittinger <cg@exept.de>
parents: 6615
diff changeset
    28
	      All Rights Reserved
88
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    29
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    30
 This software is furnished under a license and may be used
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    31
 only in accordance with the terms of that license and with the
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    32
 inclusion of the above copyright notice.   This software may not
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    33
 be provided or otherwise made available to, or used by, any
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    34
 other person.  No title to or ownership of the software is
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    35
 hereby transferred.
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    36
"
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    37
!
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    38
69
4564b6328136 *** empty log message ***
claus
parents: 25
diff changeset
    39
documentation
4564b6328136 *** empty log message ***
claus
parents: 25
diff changeset
    40
"
4564b6328136 *** empty log message ***
claus
parents: 25
diff changeset
    41
    Semaphores are used to synchronize processes providing a nonBusy wait
4564b6328136 *** empty log message ***
claus
parents: 25
diff changeset
    42
    mechanism. A process can wait for the availability of some resource by
93
e31220cb391f *** empty log message ***
claus
parents: 88
diff changeset
    43
    performing a Semaphore>>wait, which will suspend the process until the
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    44
    resource becomes available. Signalling is done by (another process performing)
93
e31220cb391f *** empty log message ***
claus
parents: 88
diff changeset
    45
    Semaphore>>signal.
159
514c749165c3 *** empty log message ***
claus
parents: 93
diff changeset
    46
    If the resource has been already available before the wait, no suspending is
69
4564b6328136 *** empty log message ***
claus
parents: 25
diff changeset
    47
    done, but the resource immediately allocated.
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
    48
    The resource internally keeps a count, the number of times the resource can be
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
    49
    allocated. If the semaphore is created with a count greater than 1, the sema
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
    50
    can be waited-upon that many times without blocking.
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
    51
    On the other hand, if initialized with a negative count, the semaphore
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
    52
    must be signalled that many times more often in order for a wait to not block.
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    53
    In other words: whenever the semaphore has a count greater than zero,
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
    54
    the wait operation will proceed. Otherwise, it will block until the count has
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
    55
    been incremented by signal operations to be greater than zero.
2234
21fe6c17fe8d commentary
Claus Gittinger <cg@exept.de>
parents: 2142
diff changeset
    56
93
e31220cb391f *** empty log message ***
claus
parents: 88
diff changeset
    57
    There are also semaphores for mutual access to a critical region
e31220cb391f *** empty log message ***
claus
parents: 88
diff changeset
    58
    (Semaphore>>forMutualExclusion and Semaphore>>critical:).
88
81dacba7a63a *** empty log message ***
claus
parents: 79
diff changeset
    59
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    60
    Additional protocol is provided for oneShot semaphores,
2234
21fe6c17fe8d commentary
Claus Gittinger <cg@exept.de>
parents: 2142
diff changeset
    61
    (#signalOnce) and for conditional signalling (#signalIf).
21fe6c17fe8d commentary
Claus Gittinger <cg@exept.de>
parents: 2142
diff changeset
    62
159
514c749165c3 *** empty log message ***
claus
parents: 93
diff changeset
    63
    You can also attach semaphores to external events (such as I/O arrival or
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    64
    timer events).
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    65
    This is done by telling the Processor to signal the semaphore
183
7899f380e77d signalIf
claus
parents: 159
diff changeset
    66
    under some condition.
159
514c749165c3 *** empty log message ***
claus
parents: 93
diff changeset
    67
    See 'Processor>>signal:afterSeconds:', 'Processor>>signal:onInput:' etc.
514c749165c3 *** empty log message ***
claus
parents: 93
diff changeset
    68
2142
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
    69
    See examples in doc/coding (found in the CodingExamples-nameSpace).
1273
f8449f53a6a3 commentary
Claus Gittinger <cg@exept.de>
parents: 1185
diff changeset
    70
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
    71
    Warning/Note/Hint:
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    72
	a Semaphore-forMutualExclusion does NEVER allow for the critical
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    73
	region to be entered twice - NOT EVEN by the same process.
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    74
	That means, that a recursive attempt to enter that section leads
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    75
	to a deadlock.
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    76
	Use a RecursionLock instead, to avoid this.
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
    77
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
    78
    Hint:
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    79
	now (Jul2002), Semaphores now allow for a negative count; this allows for
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    80
	a sync-point to be implemented easily (i.e. to wait for multiple other processes
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    81
	to arrive at a sync-point).
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    82
	See examples.
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
    83
2262
4c4d810f006f semaphore names
Claus Gittinger <cg@exept.de>
parents: 2235
diff changeset
    84
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
    85
    [instance variables:]
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    86
	count                   <SmallInteger>          the number of waits, that will go through
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    87
							without blocking.
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    88
							Incremented on #signal; decremented on #wait.
2262
4c4d810f006f semaphore names
Claus Gittinger <cg@exept.de>
parents: 2235
diff changeset
    89
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    90
	waitingProcesses        <OrderedCollection>     waiting processes - will be served first
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    91
							come first served when signalled.
2262
4c4d810f006f semaphore names
Claus Gittinger <cg@exept.de>
parents: 2235
diff changeset
    92
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    93
	lastOwnerId             <SmallInteger>          a debugging aid: set when count drops
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    94
							to zero to the current processes id.
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    95
							Helps in finding deadlocks.
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
    96
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    97
	name                    <String>                a debugging aid: an optional userFriendly
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
    98
							name; helps to identify a semaphore easier.
2262
4c4d810f006f semaphore names
Claus Gittinger <cg@exept.de>
parents: 2235
diff changeset
    99
1273
f8449f53a6a3 commentary
Claus Gittinger <cg@exept.de>
parents: 1185
diff changeset
   100
    [see also:]
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   101
	SemaphoreSet RecursionLock Monitor
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   102
	SharedQueue Delay
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   103
	Process ProcessorScheduler
1294
e26bbb61f6b2 documentation
Claus Gittinger <cg@exept.de>
parents: 1273
diff changeset
   104
e26bbb61f6b2 documentation
Claus Gittinger <cg@exept.de>
parents: 1273
diff changeset
   105
    [author:]
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   106
	Claus Gittinger
3392
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   107
"
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   108
!
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   109
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   110
examples
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   111
"
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   112
    two processes synchronizing on a sema:
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   113
                                                        [exBegin]
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   114
        |sema thread1 thread2|
3392
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   115
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   116
        sema := Semaphore new.
3392
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   117
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   118
        thread1 := [
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   119
                        Transcript showCR:'here is thread 1; now waiting ...'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   120
                        sema wait.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   121
                        Transcript showCR:'here is thread 1 again.'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   122
                   ] newProcess.
3392
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   123
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   124
        thread2 := [
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   125
                        Transcript showCR:'here is thread 2; delaying a bit ...'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   126
                        Delay waitForSeconds:5.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   127
                        Transcript showCR:'here is thread 2 again; now signalling the sema'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   128
                        sema signal.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   129
                        Transcript showCR:'here is thread 2 after the signalling.'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   130
                  ] newProcess.
3392
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   131
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   132
        thread1 priority:7.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   133
        thread2 priority:6.
3392
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   134
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   135
        thread1 resume.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   136
        thread2 resume.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   137
                                                        [exEnd]
3392
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   138
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   139
    semaphore for critical regions:
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   140
                                                        [exBegin]
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   141
        |accessLock|
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   142
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   143
        accessLock := Semaphore forMutualExclusion.
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   144
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   145
        [
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   146
            5 timesRepeat:[
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   147
                Delay waitForSeconds:2.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   148
                accessLock critical:[
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   149
                    Transcript showCR:'thread1 in critical region'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   150
                    Delay waitForSeconds:1.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   151
                    Transcript showCR:'thread1 leaving critical region'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   152
                ].
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   153
            ]
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   154
        ] forkAt:5.
3392
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   155
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   156
        [
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   157
            5 timesRepeat:[
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   158
                Delay waitForSeconds:1.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   159
                accessLock critical:[
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   160
                    Transcript showCR:'thread2 in critical region'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   161
                    Delay waitForSeconds:2.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   162
                    Transcript showCR:'thread2 leaving critical region'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   163
                ].
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   164
            ]
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   165
        ] forkAt:4.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   166
                                                        [exEnd]
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   167
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   168
    a deadlock due to recursive enter of a critical region:
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   169
                                                        [exBegin]
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   170
        |accessLock block|
3392
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   171
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   172
        accessLock := Semaphore forMutualExclusion.
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   173
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   174
        block := [:arg |
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   175
                    Transcript showCR:'about to enter'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   176
                    accessLock critical:[
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   177
                        Transcript showCR:'entered - doing action'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   178
                        arg value
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   179
                    ].
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   180
                    Transcript showCR:'left region'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   181
                 ].
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   182
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   183
        block value:[].                 'this works'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   184
        block value:[block value:[] ].  'this deadlocks'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   185
                                                        [exEnd]
3392
84332bb01a86 *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 3272
diff changeset
   186
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   187
    Avoid the deadlock by using a RecursionLock instead:
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   188
                                                        [exBegin]
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   189
        |accessLock block|
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   190
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   191
        accessLock := RecursionLock new.
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   192
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   193
        block := [:arg |
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   194
                    Transcript showCR:'about to enter'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   195
                    accessLock critical:[
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   196
                        Transcript showCR:'entered - doing action'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   197
                        arg value
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   198
                    ].
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   199
                    Transcript showCR:'left region'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   200
                 ].
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   201
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   202
        block value:[].                 'this works'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   203
        block value:[block value:[] ].  'this deadlocks'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   204
                                                        [exEnd]
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   205
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   206
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   207
   Wait for multiple processes to arrive at a sync-point:
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   208
                                                        [exBegin]
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   209
        |syncSema proceedSema thread1 thread2 thread3|
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   210
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   211
        syncSema := Semaphore new.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   212
        syncSema setCount:(1-3).
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   213
        proceedSema := Semaphore new.
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   214
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   215
        thread1 := [
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   216
                        Transcript showCR:'here is thread 1; now busy ...'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   217
                        Delay waitForSeconds:(2 + (Random nextIntegerBetween:2 and:4)).
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   218
                        Transcript showCR:'here is thread 1 again - now syncing.'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   219
                        syncSema signal.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   220
                        Transcript showCR:'thread 1 is waiting for all others...'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   221
                        proceedSema wait.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   222
                        Transcript showCR:'thread 1 done.'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   223
                   ] newProcess.
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   224
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   225
        thread2 := [
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   226
                        Transcript showCR:'here is thread 2; now busy ...'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   227
                        Delay waitForSeconds:(3 + (Random nextIntegerBetween:2 and:4)).
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   228
                        Transcript showCR:'here is thread 2 again - now syncing.'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   229
                        syncSema signal.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   230
                        Transcript showCR:'thread 2 is waiting for all others...'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   231
                        proceedSema wait.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   232
                        Transcript showCR:'thread 2 done.'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   233
                  ] newProcess.
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   234
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   235
        thread3 := [
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   236
                        Transcript showCR:'here is thread 3; now busy ...'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   237
                        Delay waitForSeconds:(4 + (Random nextIntegerBetween:2 and:4)).
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   238
                        Transcript showCR:'here is thread 3 again - now syncing.'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   239
                        syncSema signal.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   240
                        Transcript showCR:'thread 3 is waiting for all others...'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   241
                        proceedSema wait.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   242
                        Transcript showCR:'thread 3 done.'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   243
                  ] newProcess.
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   244
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   245
        thread1 priority:7.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   246
        thread2 priority:6.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   247
        thread3 priority:9.
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   248
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   249
        thread1 resume.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   250
        thread2 resume.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   251
        thread3 resume.
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   252
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   253
        Transcript showCR:'main thread: now waiting for other threads...'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   254
        syncSema wait.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   255
        Transcript showCR:'main thread: all other threads at syncPoint.'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   256
        Delay waitForSeconds:2.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   257
        Transcript showCR:'main thread: now let them proceed...'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   258
        proceedSema signalForAll.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   259
        Transcript showCR:'main thread: done.'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   260
                                                        [exEnd]
6999
b0f293f2780c comment
ca
parents: 6986
diff changeset
   261
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   262
   waitWithTimeout:0 can also be used to conditionally acquire the semaphore
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   263
   i.e. only acquire it if it is available.
6999
b0f293f2780c comment
ca
parents: 6986
diff changeset
   264
     |s|
b0f293f2780c comment
ca
parents: 6986
diff changeset
   265
b0f293f2780c comment
ca
parents: 6986
diff changeset
   266
     s := Semaphore new.
b0f293f2780c comment
ca
parents: 6986
diff changeset
   267
     [
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   268
          (s waitWithTimeout:0) notNil ifTrue:[
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   269
                Transcript showCR:'process1 got the sema'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   270
                Delay waitForSeconds:1.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   271
                Transcript showCR:'process1 signals sema'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   272
                s signal.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   273
          ] ifFalse:[
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   274
                Transcript showCR:'process1 has NOT got the sema'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   275
          ].
6999
b0f293f2780c comment
ca
parents: 6986
diff changeset
   276
     ] fork.
b0f293f2780c comment
ca
parents: 6986
diff changeset
   277
     [
20116
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   278
          (s waitWithTimeout:0) notNil ifTrue:[
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   279
                Transcript showCR:'process2 got the sema'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   280
                Delay waitForSeconds:1.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   281
                Transcript showCR:'process2 signals sema'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   282
                s signal.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   283
          ] ifFalse:[
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   284
                Transcript showCR:'process2 has NOT got the sema'.
c29d96c1db82 #OTHER by mawalch
mawalch
parents: 19928
diff changeset
   285
          ]
6999
b0f293f2780c comment
ca
parents: 6986
diff changeset
   286
     ] fork.
b0f293f2780c comment
ca
parents: 6986
diff changeset
   287
     s signal.
b0f293f2780c comment
ca
parents: 6986
diff changeset
   288
     Delay waitForSeconds:0.5.
b0f293f2780c comment
ca
parents: 6986
diff changeset
   289
     Transcript showCR:'master waits for sema'.
b0f293f2780c comment
ca
parents: 6986
diff changeset
   290
     s wait.
b0f293f2780c comment
ca
parents: 6986
diff changeset
   291
     Transcript showCR:'master got the sema'.
69
4564b6328136 *** empty log message ***
claus
parents: 25
diff changeset
   292
"
4564b6328136 *** empty log message ***
claus
parents: 25
diff changeset
   293
! !
4564b6328136 *** empty log message ***
claus
parents: 25
diff changeset
   294
1812
a048a98aa299 kludge around a situation, where one process
Claus Gittinger <cg@exept.de>
parents: 1577
diff changeset
   295
!Semaphore class methodsFor:'instance creation'!
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   296
14509
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   297
cleanup
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   298
    "an emergency helper: manually signal all semaphores which were held by a now dead process.
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   299
     Can only (;-?) happen, if a semaphore-holding process was hard terminated
14509
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   300
     (i.e. no ensure handling happened), and semas remain in a bad state."
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   301
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   302
    self allSubInstancesDo:[:sema |
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   303
        |lastOwner|
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   304
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   305
        (sema count == 0
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   306
         and:[(lastOwner := sema lastOwner) notNil
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   307
         and:[lastOwner isDead]]) ifTrue:[
14509
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   308
            sema signal
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   309
        ]
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   310
    ]
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   311
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   312
    "Modified: / 20-02-2017 / 15:58:34 / stefan"
14509
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   313
!
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   314
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   315
forMutualExclusion
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   316
    "create & return a new semaphore which allows exactly one process to
2142
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
   317
     wait on it without blocking. This type of semaphore is used
4437
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   318
     for mutual exclusion from critical regions (see #critical:).
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   319
     Also see RecursionLock, to avoid deadlock in case of recursive entered
Claus Gittinger <cg@exept.de>
parents: 3836
diff changeset
   320
     critical regions."
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   321
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   322
    ^ self basicNew setCount:1; name:'criticalRegionSema'
2142
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
   323
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   324
    "Modified: / 17-06-1998 / 16:23:09 / cg"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   325
    "Modified: / 18-02-2017 / 17:27:14 / stefan"
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   326
!
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   327
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   328
new
a27a279701f8 Initial revision
claus
parents:
diff changeset
   329
    "create & return a new semaphore which blocks until a signal is sent"
a27a279701f8 Initial revision
claus
parents:
diff changeset
   330
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   331
    ^ self basicNew setCount:0
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   332
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   333
    "Modified: / 20-02-2017 / 10:26:04 / stefan"
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   334
!
a27a279701f8 Initial revision
claus
parents:
diff changeset
   335
a27a279701f8 Initial revision
claus
parents:
diff changeset
   336
new:n
a27a279701f8 Initial revision
claus
parents:
diff changeset
   337
    "create & return a new semaphore which allows n waits before
a27a279701f8 Initial revision
claus
parents:
diff changeset
   338
     blocking"
a27a279701f8 Initial revision
claus
parents:
diff changeset
   339
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   340
    ^ self basicNew setCount:n
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   341
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   342
    "Modified: / 18-02-2017 / 17:27:26 / stefan"
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   343
! !
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   344
9067
a8b1ac38b89d *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 8617
diff changeset
   345
!Semaphore methodsFor:'Compatibility-Squeak'!
a8b1ac38b89d *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 8617
diff changeset
   346
a8b1ac38b89d *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 8617
diff changeset
   347
isSignaled
a8b1ac38b89d *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 8617
diff changeset
   348
    ^ self wouldBlock not
12567
eb78ade59632 added: #waitTimeoutSeconds:
Stefan Vogel <sv@exept.de>
parents: 11281
diff changeset
   349
!
eb78ade59632 added: #waitTimeoutSeconds:
Stefan Vogel <sv@exept.de>
parents: 11281
diff changeset
   350
12589
b8964a49b462 added: #waitTimeoutMSecs:
Stefan Vogel <sv@exept.de>
parents: 12567
diff changeset
   351
waitTimeoutMSecs:milliSeconds
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
   352
    ^ self waitWithTimeoutMs:milliSeconds state:#wait
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
   353
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
   354
    "Modified: / 24-07-2017 / 21:18:01 / cg"
12589
b8964a49b462 added: #waitTimeoutMSecs:
Stefan Vogel <sv@exept.de>
parents: 12567
diff changeset
   355
!
b8964a49b462 added: #waitTimeoutMSecs:
Stefan Vogel <sv@exept.de>
parents: 12567
diff changeset
   356
12567
eb78ade59632 added: #waitTimeoutSeconds:
Stefan Vogel <sv@exept.de>
parents: 11281
diff changeset
   357
waitTimeoutSeconds:seconds
eb78ade59632 added: #waitTimeoutSeconds:
Stefan Vogel <sv@exept.de>
parents: 11281
diff changeset
   358
    ^ self waitWithTimeout:seconds
9067
a8b1ac38b89d *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 8617
diff changeset
   359
! !
a8b1ac38b89d *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 8617
diff changeset
   360
19927
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
   361
!Semaphore methodsFor:'accessing'!
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
   362
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
   363
owner
19928
e2bf8e38979c #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 19927
diff changeset
   364
    "an optional reference to someone who owns this semaphore,
e2bf8e38979c #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 19927
diff changeset
   365
     typically a shared queue or a windowgroup or similar.
e2bf8e38979c #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 19927
diff changeset
   366
     This has no semantic meaning and is only used to support debugging"
e2bf8e38979c #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 19927
diff changeset
   367
     
19927
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
   368
    ^ owner
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
   369
!
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
   370
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
   371
owner:something
19928
e2bf8e38979c #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 19927
diff changeset
   372
    "an optional reference to someone who owns this semaphore,
e2bf8e38979c #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 19927
diff changeset
   373
     typically a shared queue or a windowgroup or similar.
e2bf8e38979c #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 19927
diff changeset
   374
     This has no semantic meaning and is only used to support debugging"
e2bf8e38979c #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 19927
diff changeset
   375
19927
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
   376
    owner := something.
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
   377
! !
b99e02676fb6 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 19192
diff changeset
   378
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   379
!Semaphore methodsFor:'printing & storing'!
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   380
13355
442f69dcc5b8 added: #displayOn: (replace displayString)
Stefan Vogel <sv@exept.de>
parents: 13152
diff changeset
   381
displayOn:aGCOrStream
2142
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
   382
    "return a string to display the receiver - include the
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
   383
     count for your convenience"
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
   384
13355
442f69dcc5b8 added: #displayOn: (replace displayString)
Stefan Vogel <sv@exept.de>
parents: 13152
diff changeset
   385
    "/ what a kludge - Dolphin and Squeak mean: printOn: a stream;
21527
95a27612eafe #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 21190
diff changeset
   386
    "/ old ST80 means: draw-yourself on a GC.
13355
442f69dcc5b8 added: #displayOn: (replace displayString)
Stefan Vogel <sv@exept.de>
parents: 13152
diff changeset
   387
    (aGCOrStream isStream) ifFalse:[
21527
95a27612eafe #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 21190
diff changeset
   388
        ^ super displayOn:aGCOrStream
13355
442f69dcc5b8 added: #displayOn: (replace displayString)
Stefan Vogel <sv@exept.de>
parents: 13152
diff changeset
   389
    ].
442f69dcc5b8 added: #displayOn: (replace displayString)
Stefan Vogel <sv@exept.de>
parents: 13152
diff changeset
   390
    aGCOrStream
21527
95a27612eafe #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 21190
diff changeset
   391
        nextPutAll:self class name;
95a27612eafe #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 21190
diff changeset
   392
        nextPut:$(.
13355
442f69dcc5b8 added: #displayOn: (replace displayString)
Stefan Vogel <sv@exept.de>
parents: 13152
diff changeset
   393
    count printOn:aGCOrStream.
442f69dcc5b8 added: #displayOn: (replace displayString)
Stefan Vogel <sv@exept.de>
parents: 13152
diff changeset
   394
    aGCOrStream nextPutAll:' name: '.
442f69dcc5b8 added: #displayOn: (replace displayString)
Stefan Vogel <sv@exept.de>
parents: 13152
diff changeset
   395
    (name ? 'unnamed') printOn:aGCOrStream.
442f69dcc5b8 added: #displayOn: (replace displayString)
Stefan Vogel <sv@exept.de>
parents: 13152
diff changeset
   396
    aGCOrStream nextPut:$).
442f69dcc5b8 added: #displayOn: (replace displayString)
Stefan Vogel <sv@exept.de>
parents: 13152
diff changeset
   397
21527
95a27612eafe #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 21190
diff changeset
   398
    "Modified: / 28-06-1997 / 16:21:09 / cg"
95a27612eafe #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 21190
diff changeset
   399
    "Modified (comment): / 22-02-2017 / 16:47:54 / cg"
2265
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   400
!
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   401
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   402
name
21556
73fd8daf236a #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 21527
diff changeset
   403
    "return the semaphore's userFriendly name"
2265
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   404
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   405
    ^ name
21556
73fd8daf236a #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 21527
diff changeset
   406
73fd8daf236a #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 21527
diff changeset
   407
    "Modified (comment): / 23-02-2017 / 16:54:32 / cg"
2265
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   408
!
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   409
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   410
name:aString
21556
73fd8daf236a #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 21527
diff changeset
   411
    "set the semaphore's userFriendly name"
2265
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   412
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   413
    name := aString
21556
73fd8daf236a #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 21527
diff changeset
   414
73fd8daf236a #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 21527
diff changeset
   415
    "Modified (comment): / 23-02-2017 / 16:54:35 / cg"
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   416
! !
a27a279701f8 Initial revision
claus
parents:
diff changeset
   417
6986
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   418
!Semaphore methodsFor:'private'!
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   419
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   420
addWaitingProcess:aProcess
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   421
    "add aProcess to the list of waiting processes.
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   422
     all processes are ordered first-come-first-serve.
6986
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   423
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   424
     NOTE: must be called with blocked interrupts"
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   425
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   426
    waitingProcesses isNil ifTrue:[
21190
138b90edd609 #DOCUMENTATION by stefan
Stefan Vogel <sv@exept.de>
parents: 21115
diff changeset
   427
        "for now - assume that there is probably only one waiter"
138b90edd609 #DOCUMENTATION by stefan
Stefan Vogel <sv@exept.de>
parents: 21115
diff changeset
   428
        waitingProcesses := Array with:aProcess
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   429
    ] ifFalse:[
21190
138b90edd609 #DOCUMENTATION by stefan
Stefan Vogel <sv@exept.de>
parents: 21115
diff changeset
   430
        waitingProcesses isArray ifTrue:[
138b90edd609 #DOCUMENTATION by stefan
Stefan Vogel <sv@exept.de>
parents: 21115
diff changeset
   431
            "add 2 to reserve space for additional waiters"
138b90edd609 #DOCUMENTATION by stefan
Stefan Vogel <sv@exept.de>
parents: 21115
diff changeset
   432
            waitingProcesses := (OrderedCollection new:waitingProcesses size + 2)
138b90edd609 #DOCUMENTATION by stefan
Stefan Vogel <sv@exept.de>
parents: 21115
diff changeset
   433
                                    addAll:waitingProcesses;
138b90edd609 #DOCUMENTATION by stefan
Stefan Vogel <sv@exept.de>
parents: 21115
diff changeset
   434
                                    yourself.
138b90edd609 #DOCUMENTATION by stefan
Stefan Vogel <sv@exept.de>
parents: 21115
diff changeset
   435
        ].
138b90edd609 #DOCUMENTATION by stefan
Stefan Vogel <sv@exept.de>
parents: 21115
diff changeset
   436
        waitingProcesses add:aProcess.
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   437
    ].
6986
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   438
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   439
"/    "Sort, so that higher priority process are resumed first.
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   440
"/    Processes having the same priority are ordered first-come-first-serve."
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   441
"/
6986
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   442
"/    |priority insertIndex|
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   443
"/
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   444
"/    (waitingProcesses size == 0
6986
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   445
"/     or:[(priority := aProcess priority) <= waitingProcesses last priority]) ifTrue:[
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   446
"/        waitingProcesses add:aProcess.
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   447
"/        ^ self.
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   448
"/    ].
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   449
"/
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   450
"/    insertIndex := waitingProcesses findFirst:[:process| process priority < priority].
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   451
"/    waitingProcesses add:aProcess beforeIndex:insertIndex.
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   452
!
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   453
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   454
removeWaitingProcess:aProcess
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   455
    "remove aProcess from the list of waiting processes
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   456
     NO action if it is not in the list.
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   457
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   458
     NOTE: must be called with blocked interrupts"
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   459
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   460
    |nWaiting|
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   461
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   462
    nWaiting := waitingProcesses size.
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   463
    nWaiting == 0 ifTrue:[^ self].
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   464
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   465
    nWaiting == 1 ifTrue:[
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   466
	(waitingProcesses at:1) == aProcess ifTrue:[
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   467
	    waitingProcesses := nil.
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   468
	].
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   469
	^ self.
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   470
    ].
8617
5fb5a717deff *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 8569
diff changeset
   471
    waitingProcesses removeIdentical:aProcess ifAbsent:[].
11254
a40a7313b9a8 *** empty log message ***
ca
parents: 11218
diff changeset
   472
!
a40a7313b9a8 *** empty log message ***
ca
parents: 11218
diff changeset
   473
a40a7313b9a8 *** empty log message ***
ca
parents: 11218
diff changeset
   474
wakeupWaiters
a40a7313b9a8 *** empty log message ***
ca
parents: 11218
diff changeset
   475
    "remove all waiting processes from the list of waiting processes
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   476
     and resume them.
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   477
     Answer true, if a higher priority process became ready, false if not.
11254
a40a7313b9a8 *** empty log message ***
ca
parents: 11218
diff changeset
   478
     NOTE: Must be called when known that waitingProcesses is nonNil and
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   479
           also with blocked interrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   480
           Must not perform an operation that causes a reschedule."
11254
a40a7313b9a8 *** empty log message ***
ca
parents: 11218
diff changeset
   481
15635
1b42005ee234 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 15551
diff changeset
   482
    |processes anyDead needsReschedule|
11254
a40a7313b9a8 *** empty log message ***
ca
parents: 11218
diff changeset
   483
13152
78fdc9d96bc3 comment/format in:
Stefan Vogel <sv@exept.de>
parents: 12697
diff changeset
   484
    processes := waitingProcesses.
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
   485
    "/ do not set to nil - a waiting process may be suspended and will not be resumed by #makeRunnable: ...
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
   486
    "/    waitingProcesses := nil.
11254
a40a7313b9a8 *** empty log message ***
ca
parents: 11218
diff changeset
   487
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   488
    needsReschedule := anyDead := false.
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   489
    processes do:[:eachProcess |
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   490
        (Processor makeRunnable:eachProcess) notNil ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   491
            "higher priority process became ready"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   492
            needsReschedule := true.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   493
        ] ifFalse:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   494
            "if process is nil or dead (or for other reasons) makeRunnable returns false.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   495
             So check here."
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   496
            (eachProcess isNil or:[eachProcess isDead]) ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   497
                "printing to Transcript might not be a good idea while interrupts are blocked"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   498
                'Semaphore>>wakeupWaiters: removing a dead process: ' infoPrint. eachProcess infoPrintCR.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   499
                anyDead := true.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   500
            ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   501
        ].
14509
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   502
    ].
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   503
    anyDead ifTrue:[
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   504
        "interrupts are already blocked by sender"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   505
        waitingProcesses := processes reject:[:p | p isNil or:[p isDead]]
14509
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   506
    ].
2c8531dbf456 added: #cleanup
anwild
parents: 13579
diff changeset
   507
14770
87973b889eb8 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 14763
diff changeset
   508
    ^ needsReschedule.
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   509
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   510
    "Modified: / 20-02-2017 / 11:34:42 / stefan"
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
   511
    "Modified (format): / 24-07-2017 / 18:03:47 / cg"
6986
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   512
! !
d2ef4d62861e Isolate waitProcesses (prepare for sorting by prio)
Stefan Vogel <sv@exept.de>
parents: 6984
diff changeset
   513
7258
9ccdbee7d1ad method category rename
Claus Gittinger <cg@exept.de>
parents: 6999
diff changeset
   514
!Semaphore methodsFor:'private-accessing'!
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   515
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   516
clear
21556
73fd8daf236a #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 21527
diff changeset
   517
    "clear the semaphore's count"
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   518
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   519
    count := 0
21556
73fd8daf236a #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 21527
diff changeset
   520
73fd8daf236a #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 21527
diff changeset
   521
    "Modified (comment): / 23-02-2017 / 16:54:28 / cg"
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   522
!
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   523
2407
6b0e21391915 added #initSignals for ST-80 compatibility.
Claus Gittinger <cg@exept.de>
parents: 2352
diff changeset
   524
initSignals
6b0e21391915 added #initSignals for ST-80 compatibility.
Claus Gittinger <cg@exept.de>
parents: 2352
diff changeset
   525
    "set the count of the semaphore to zero.
6b0e21391915 added #initSignals for ST-80 compatibility.
Claus Gittinger <cg@exept.de>
parents: 2352
diff changeset
   526
     provided for ST-80 compatibility."
6b0e21391915 added #initSignals for ST-80 compatibility.
Claus Gittinger <cg@exept.de>
parents: 2352
diff changeset
   527
6b0e21391915 added #initSignals for ST-80 compatibility.
Claus Gittinger <cg@exept.de>
parents: 2352
diff changeset
   528
    count := 0
6b0e21391915 added #initSignals for ST-80 compatibility.
Claus Gittinger <cg@exept.de>
parents: 2352
diff changeset
   529
6b0e21391915 added #initSignals for ST-80 compatibility.
Claus Gittinger <cg@exept.de>
parents: 2352
diff changeset
   530
    "Created: 17.2.1997 / 11:31:19 / cg"
6b0e21391915 added #initSignals for ST-80 compatibility.
Claus Gittinger <cg@exept.de>
parents: 2352
diff changeset
   531
!
6b0e21391915 added #initSignals for ST-80 compatibility.
Claus Gittinger <cg@exept.de>
parents: 2352
diff changeset
   532
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   533
setCount:n
2142
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
   534
    "set the count of the semaphore;
20426
2ec010b1a559 #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 20263
diff changeset
   535
     that's the number of possible waits, without blocking"
2142
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
   536
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   537
    waitingProcesses := nil.
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   538
    count := n
2142
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
   539
2407
6b0e21391915 added #initSignals for ST-80 compatibility.
Claus Gittinger <cg@exept.de>
parents: 2352
diff changeset
   540
    "Modified: 17.2.1997 / 11:36:40 / cg"
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   541
! !
a27a279701f8 Initial revision
claus
parents:
diff changeset
   542
5256
1209e167d63e category rename
Claus Gittinger <cg@exept.de>
parents: 4902
diff changeset
   543
!Semaphore methodsFor:'queries'!
217
a0400fdbc933 *** empty log message ***
claus
parents: 183
diff changeset
   544
2235
c6a15bd9a33c added access to count
Claus Gittinger <cg@exept.de>
parents: 2234
diff changeset
   545
count
c6a15bd9a33c added access to count
Claus Gittinger <cg@exept.de>
parents: 2234
diff changeset
   546
    "return the number of 'already-counted' trigger events.
20426
2ec010b1a559 #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 20263
diff changeset
   547
     That's the number of waits which will succeed without blocking"
2235
c6a15bd9a33c added access to count
Claus Gittinger <cg@exept.de>
parents: 2234
diff changeset
   548
c6a15bd9a33c added access to count
Claus Gittinger <cg@exept.de>
parents: 2234
diff changeset
   549
    ^ count
c6a15bd9a33c added access to count
Claus Gittinger <cg@exept.de>
parents: 2234
diff changeset
   550
c6a15bd9a33c added access to count
Claus Gittinger <cg@exept.de>
parents: 2234
diff changeset
   551
    "Created: 23.1.1997 / 02:55:58 / cg"
c6a15bd9a33c added access to count
Claus Gittinger <cg@exept.de>
parents: 2234
diff changeset
   552
!
c6a15bd9a33c added access to count
Claus Gittinger <cg@exept.de>
parents: 2234
diff changeset
   553
13579
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   554
lastOwner
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   555
    "return the last owning process or nil
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   556
     (the one which counted to zero).
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   557
     May be very useful in debugging deadLock situations"
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   558
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   559
    ^ Processor processWithId:lastOwnerId.
13579
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   560
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   561
    "Created: / 11-08-2011 / 14:35:36 / cg"
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   562
!
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   563
2265
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   564
lastOwnerId
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   565
    "return the processId of the last owning process
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   566
     (the one which counted to zero).
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   567
     May be very useful in debugging deadLock situations"
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   568
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   569
    ^ lastOwnerId
2265
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   570
13579
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   571
    "Created: / 24-01-1997 / 23:09:33 / cg"
2265
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   572
!
775feb718a9d checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2262
diff changeset
   573
1329
0a025ca40653 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 1294
diff changeset
   574
numberOfWaitingProcesses
0a025ca40653 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 1294
diff changeset
   575
    "return the number of processes waiting on the receiver"
0a025ca40653 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 1294
diff changeset
   576
0a025ca40653 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 1294
diff changeset
   577
    ^ waitingProcesses size
0a025ca40653 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 1294
diff changeset
   578
0a025ca40653 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 1294
diff changeset
   579
    "Created: 3.5.1996 / 18:06:27 / cg"
0a025ca40653 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 1294
diff changeset
   580
!
0a025ca40653 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 1294
diff changeset
   581
1577
720f14f90e2c access to waitingProcesses added
Claus Gittinger <cg@exept.de>
parents: 1329
diff changeset
   582
waitingProcesses
720f14f90e2c access to waitingProcesses added
Claus Gittinger <cg@exept.de>
parents: 1329
diff changeset
   583
    "return the processes waiting on the receiver"
720f14f90e2c access to waitingProcesses added
Claus Gittinger <cg@exept.de>
parents: 1329
diff changeset
   584
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   585
    ^ waitingProcesses ? #()
1577
720f14f90e2c access to waitingProcesses added
Claus Gittinger <cg@exept.de>
parents: 1329
diff changeset
   586
720f14f90e2c access to waitingProcesses added
Claus Gittinger <cg@exept.de>
parents: 1329
diff changeset
   587
    "Created: 18.7.1996 / 20:53:33 / cg"
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   588
! !
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   589
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   590
!Semaphore methodsFor:'semaphoreSet interface'!
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   591
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   592
checkAndAddWaitingProcess:process
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   593
    "interface for SemaphoreSet.
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   594
     If the semaphore is available, decrement it and return true.
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   595
     Otherwise register our process to be wakened up once the semaphore is available
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   596
     and return false.
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   597
     ATTENTION: this must be invoked with OperatingSystem-interrupts-blocked.
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   598
    "
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   599
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   600
    count > 0 ifTrue:[
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   601
	count := count - 1.
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   602
	count == 0 ifTrue:[
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   603
	    lastOwnerId := Processor activeProcessId.
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   604
	].
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   605
	^ true
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   606
    ].
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   607
    (waitingProcesses notNil and:[(waitingProcesses includesIdentical:process)]) ifFalse:[
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
   608
	self addWaitingProcess:process.
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   609
    ].
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   610
    ^ false
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   611
13579
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   612
    "Modified: / 14-12-1995 / 10:32:17 / stefan"
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   613
    "Modified: / 11-08-2011 / 14:36:20 / cg"
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   614
! !
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   615
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   616
!Semaphore methodsFor:'signaling'!
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   617
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   618
signal
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   619
    "waking up the highest prio waiter."
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   620
14770
87973b889eb8 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 14763
diff changeset
   621
    |wasBlocked needsReschedule|
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   622
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   623
    wasBlocked := OperatingSystem blockInterrupts.
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   624
    count := count + 1.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   625
    needsReschedule := waitingProcesses notEmptyOrNil and:[self wakeupWaiters].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   626
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   627
14770
87973b889eb8 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 14763
diff changeset
   628
    needsReschedule ifTrue:[
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   629
        "now, schedule the highest priority process"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   630
        Processor reschedule.
14770
87973b889eb8 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 14763
diff changeset
   631
    ].
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   632
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   633
    "Modified: / 04-02-1998 / 21:01:07 / cg"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   634
    "Modified: / 24-07-2017 / 11:44:15 / stefan"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   635
!
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   636
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   637
signal:anInteger
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   638
    "increment semaphore by anInteger waking up the highest prio waiters."
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   639
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   640
    |wasBlocked needsReschedule|
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   641
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   642
    wasBlocked := OperatingSystem blockInterrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   643
    count := count + anInteger.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   644
    needsReschedule := waitingProcesses notEmptyOrNil and:[self wakeupWaiters].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   645
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   646
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   647
    needsReschedule ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   648
        "now, schedule the highest priority process"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   649
        Processor reschedule.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   650
    ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   651
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   652
    "Created: / 22-02-2017 / 14:38:33 / stefan"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   653
    "Modified: / 24-07-2017 / 11:44:36 / stefan"
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   654
!
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   655
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   656
signalForAll
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   657
    "signal the semaphore for all waiters.
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   658
     This can be used for process synchronization, if multiple processes are
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   659
     waiting for a common event."
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   660
14770
87973b889eb8 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 14763
diff changeset
   661
    |wasBlocked needsReschedule|
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   662
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   663
    waitingProcesses notEmptyOrNil ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   664
        needsReschedule := false.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   665
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   666
        wasBlocked := OperatingSystem blockInterrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   667
        "first, make them all runnable, but do not schedule
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   668
         (in case one has higher prio and goes into a wait immediately again.)"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   669
        waitingProcesses notEmptyOrNil ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   670
            needsReschedule := self wakeupWaiters.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   671
            "wakeupWaites may have removed dead processes from waitingProcesses!!"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   672
            count := count + waitingProcesses size.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   673
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   674
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   675
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   676
        needsReschedule ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   677
            "now, schedule the highest priority process"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   678
            Processor reschedule.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   679
        ].
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   680
    ]
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   681
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   682
    "Modified: / 05-02-1998 / 10:40:26 / cg"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   683
    "Modified: / 24-07-2017 / 11:44:49 / stefan"
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   684
!
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   685
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   686
signalIf
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   687
    "signal the semaphore, but only if being waited upon.
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   688
     This can be used for one-shot semaphores (i.e. not remembering
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   689
     previous signals)"
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   690
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   691
    |wasBlocked needsReschedule|
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   692
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   693
    waitingProcesses notNil ifTrue:[
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   694
        needsReschedule := false.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   695
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   696
        wasBlocked := OperatingSystem blockInterrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   697
        waitingProcesses notEmptyOrNil ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   698
            count := count + 1.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   699
            needsReschedule := self wakeupWaiters.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   700
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   701
        wasBlocked ifFalse:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   702
            OperatingSystem unblockInterrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   703
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   704
        needsReschedule ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   705
            "now, schedule the highest priority process"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   706
            Processor reschedule.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   707
        ].
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   708
    ]
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   709
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   710
    "Modified: / 28-02-1996 / 21:23:57 / cg"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   711
    "Modified: / 24-07-2017 / 11:45:20 / stefan"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   712
!
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   713
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   714
signalIfWithReturn
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   715
    "signal the semaphore, but only if being waited upon.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   716
     This can be used for one-shot semaphores (i.e. not remembering
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   717
     previous signals).
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   718
     Answer true if a reschedule is needed, false if not."
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   719
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   720
    |wasBlocked needsReschedule|
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   721
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   722
    needsReschedule := false.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   723
    waitingProcesses notNil ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   724
        wasBlocked := OperatingSystem blockInterrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   725
        waitingProcesses notEmptyOrNil ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   726
            count := count + 1.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   727
            needsReschedule := self wakeupWaiters.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   728
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   729
        wasBlocked ifFalse:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   730
            OperatingSystem unblockInterrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   731
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   732
    ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   733
    ^ needsReschedule.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   734
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   735
    "Created: / 19-02-2017 / 18:09:23 / stefan"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   736
    "Modified: / 24-07-2017 / 11:45:25 / stefan"
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   737
!
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   738
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   739
signalOnce
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   740
    "wakeup waiters - but only once.
9462
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   741
     I.e. if the semaphore has already been signaled, this is ignored."
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   742
14770
87973b889eb8 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 14763
diff changeset
   743
    |wasBlocked needsReschedule|
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   744
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   745
    count <= 0 ifTrue:[
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   746
        needsReschedule := false.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   747
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   748
        wasBlocked := OperatingSystem blockInterrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   749
        "/ check again - now interrupts are blocked.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   750
        count <= 0 ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   751
            count := count + 1.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   752
            needsReschedule := (count == 1
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   753
                                and:[waitingProcesses notEmptyOrNil 
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   754
                                and:[self wakeupWaiters]]).
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   755
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   756
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   757
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   758
        needsReschedule ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   759
            "now, schedule the highest priority process"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   760
            Processor reschedule.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   761
        ].
14770
87973b889eb8 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 14763
diff changeset
   762
    ].
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   763
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   764
    "Modified: / 28-02-1996 / 21:24:08 / cg"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   765
    "Modified: / 24-07-2017 / 11:46:07 / stefan"
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   766
! !
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   767
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   768
!Semaphore methodsFor:'testing'!
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   769
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   770
isEmpty
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   771
    "ST80 compatibility - return true if there are no waiters"
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   772
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   773
    ^ waitingProcesses isEmptyOrNil
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   774
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   775
    "Created: / 03-05-1996 / 18:06:27 / cg"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   776
    "Modified: / 18-06-1998 / 16:07:38 / cg"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   777
    "Modified: / 24-07-2017 / 11:43:35 / stefan"
1577
720f14f90e2c access to waitingProcesses added
Claus Gittinger <cg@exept.de>
parents: 1329
diff changeset
   778
!
720f14f90e2c access to waitingProcesses added
Claus Gittinger <cg@exept.de>
parents: 1329
diff changeset
   779
217
a0400fdbc933 *** empty log message ***
claus
parents: 183
diff changeset
   780
wouldBlock
a0400fdbc933 *** empty log message ***
claus
parents: 183
diff changeset
   781
    "return true, if the receiver would block the activeProcess
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   782
     if a wait was performed. False otherwise.
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   783
     Attention: if asked without some global lock (blockedInterrupts),
15551
8ee0431a2f39 class: Semaphore
Claus Gittinger <cg@exept.de>
parents: 14773
diff changeset
   784
     the returned value may be outdated right away."
217
a0400fdbc933 *** empty log message ***
claus
parents: 183
diff changeset
   785
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   786
    ^ count <= 0
217
a0400fdbc933 *** empty log message ***
claus
parents: 183
diff changeset
   787
! !
a0400fdbc933 *** empty log message ***
claus
parents: 183
diff changeset
   788
8569
657853ae04cd Change waitXXX
Stefan Vogel <sv@exept.de>
parents: 7718
diff changeset
   789
!Semaphore methodsFor:'waiting'!
5713
993805688c7f added #clear
Claus Gittinger <cg@exept.de>
parents: 5556
diff changeset
   790
9462
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   791
consume
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   792
    "consume the resource without waiting.
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   793
     This works even if the count is 0 (count may become negative).
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   794
     Answer the new count afterwards"
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   795
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   796
    ^ self consume:1
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   797
!
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   798
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   799
consume:n
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   800
    "consume the resource n times without waiting.
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   801
     This works even if the count is 0 (count may become negative).
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   802
     Answer the new count afterwards"
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   803
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   804
    |wasBlocked|
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   805
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   806
    wasBlocked := OperatingSystem blockInterrupts.
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   807
    count := count - n.
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   808
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   809
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   810
    ^ count
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   811
!
822bd68c3c45 new: #consume
Stefan Vogel <sv@exept.de>
parents: 9067
diff changeset
   812
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   813
critical:aBlock
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   814
    "evaluate aBlock as a critical region; the receiver must be
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   815
     created using Semaphore>>forMutualExclusion"
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   816
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   817
    |retVal wasBlocked needsReschedule gotSema|
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   818
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   819
    wasBlocked := OperatingSystem blockInterrupts.
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   820
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   821
    count > 0 ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   822
        "/ inlined common case when no wait is required - ugly kludge but helpful
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   823
        count := count - 1.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   824
        count == 0 ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   825
            lastOwnerId := Processor activeProcessId.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   826
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   827
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   828
        retVal := aBlock ifCurtailed:[self signal].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   829
    ] ifFalse:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   830
        "have to wait for Semaphore availability"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   831
        retVal := [
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   832
            gotSema := self wait.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   833
            wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   834
            aBlock value.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   835
        ] ifCurtailed:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   836
            wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   837
            gotSema notNil ifTrue:[self signal].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   838
        ].
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   839
    ].
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   840
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   841
    "this is inlined code from #signal"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   842
    OperatingSystem blockInterrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   843
    count := count + 1.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   844
    needsReschedule := waitingProcesses notEmptyOrNil and:[self wakeupWaiters].
7717
ac9f01ef7157 tuning - although ugly, the speedup of critical: is > 30%
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   845
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   846
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   847
    needsReschedule ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   848
        "now, schedule the highest priority process"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   849
        Processor reschedule.
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   850
    ].
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   851
    "end inlined code from #signal"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   852
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   853
    ^ retVal.
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   854
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   855
    "
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   856
      the example below is stupid (it should use a SharedQueue,
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   857
      or at least a Queue with critical regions).
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   858
      Anyhow, it demonstrates how two processes lock each other
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   859
      from accessing coll at the same time
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   860
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   861
     |sema coll|
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   862
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   863
     sema := Semaphore forMutualExclusion.
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   864
     coll := OrderedCollection new:10.
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   865
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   866
     [
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   867
        1 to:1000 do:[:i |
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   868
            sema critical:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   869
                coll addLast:i.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   870
                (Delay forSeconds:0.1) wait.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   871
            ]
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   872
        ]
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   873
     ] forkAt:4.
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   874
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   875
     [
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   876
        1 to:1000 do:[:i |
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   877
            sema critical:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   878
                coll removeFirst.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   879
                (Delay forSeconds:0.1) wait.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   880
            ]
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   881
        ]
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   882
     ] forkAt:4.
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   883
    "
1185
2b24258b4f24 Fix in critical: don't signal a semaphore that we didn't get when unwinding.
Stefan Vogel <sv@exept.de>
parents: 1122
diff changeset
   884
13579
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   885
    "Modified: / 11-08-2011 / 14:36:30 / cg"
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   886
    "Modified: / 24-07-2017 / 11:42:47 / stefan"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   887
!
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   888
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   889
critical:aBlock ifBlocking:blockingBlock
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   890
    "like critical:, but do not block if the lock cannot be acquired.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   891
     Instead, return the value of the second argument, blockingBlock."
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   892
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   893
    ^ self critical:aBlock timeoutMs:0 ifBlocking:blockingBlock.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   894
!
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   895
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   896
critical:aBlock timeoutMs:timeoutMs ifBlocking:blockingBlock
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   897
    "like critical:, but do not block if the lock cannot be acquired 
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   898
     within timeoutMs milliseconds.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   899
     Instead, return the value of blockingBlock."
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   900
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   901
    |retVal wasBlocked needsReschedule gotSema|
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   902
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   903
    wasBlocked := OperatingSystem blockInterrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   904
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   905
    count > 0 ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   906
        "/ inlined common case when no wait is required - ugly kludge but helpful
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   907
        count := count - 1.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   908
        count == 0 ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   909
            lastOwnerId := Processor activeProcessId.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   910
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   911
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   912
        retVal := aBlock ifCurtailed:[self signal].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   913
        gotSema := true.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   914
    ] ifFalse:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   915
        "have to wait for Semaphore availability"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   916
        timeoutMs ~= 0 ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   917
            retVal := [
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
   918
                gotSema := self waitWithTimeoutMs:timeoutMs state:#wait.
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   919
                gotSema notNil ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   920
                    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   921
                    aBlock value.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   922
                ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   923
            ] ifCurtailed:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   924
                wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   925
                gotSema notNil ifTrue:[self signal].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   926
            ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   927
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   928
    ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   929
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   930
    gotSema notNil ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   931
        "this is inlined code from #signal"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   932
        OperatingSystem blockInterrupts.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   933
        count := count + 1.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   934
        needsReschedule := waitingProcesses notEmptyOrNil and:[self wakeupWaiters].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   935
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   936
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   937
        needsReschedule ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   938
            "now, schedule the highest priority process"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   939
            Processor reschedule.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   940
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   941
        "end inlined code from #signal"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   942
    ] ifFalse:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   943
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   944
        retVal := blockingBlock value.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   945
    ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   946
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   947
    ^ retVal.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   948
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   949
    "Created: / 18-02-2017 / 22:15:25 / stefan"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   950
    "Modified: / 24-07-2017 / 11:43:19 / stefan"
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
   951
    "Modified: / 24-07-2017 / 21:17:47 / cg"
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   952
!
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
   953
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   954
wait
a27a279701f8 Initial revision
claus
parents:
diff changeset
   955
    "wait for the semaphore"
a27a279701f8 Initial revision
claus
parents:
diff changeset
   956
2142
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
   957
    |activeProcess wasBlocked|
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
   958
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   959
    wasBlocked := OperatingSystem blockInterrupts.
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
   960
    count <= 0 ifTrue:[
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   961
        activeProcess := Processor activeProcess.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   962
        "
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   963
         need a while-loop here, since more than one process may
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   964
         wait for it and another one may also wake up.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   965
         Thus, the count is not always non-zero after returning from
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   966
         suspend.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   967
        "
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   968
        [
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   969
            self addWaitingProcess:activeProcess.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   970
            "
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   971
             for some more descriptive info in processMonitor ...
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   972
             ... set the state to #wait (instead of #suspend)
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   973
            "
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   974
            [
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   975
                activeProcess suspendWithState:#wait
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   976
            ] ifCurtailed:[
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   977
                "interrupts are not blocked when entered through Processor>>#interruptActive"
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   978
                OperatingSystem blockInterrupts.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   979
                self removeWaitingProcess:activeProcess.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   980
                wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   981
            ].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   982
            self removeWaitingProcess:activeProcess.
16286
4fd3aa76f29e class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 15690
diff changeset
   983
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   984
            count <= 0
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   985
        ] whileTrue.
93
e31220cb391f *** empty log message ***
claus
parents: 88
diff changeset
   986
    ].
e31220cb391f *** empty log message ***
claus
parents: 88
diff changeset
   987
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   988
    "if we come here, we have acquired the semaphore"
10
claus
parents: 3
diff changeset
   989
    count := count - 1.
2262
4c4d810f006f semaphore names
Claus Gittinger <cg@exept.de>
parents: 2235
diff changeset
   990
    count == 0 ifTrue:[
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
   991
        lastOwnerId := Processor activeProcessId.
2262
4c4d810f006f semaphore names
Claus Gittinger <cg@exept.de>
parents: 2235
diff changeset
   992
    ].
93
e31220cb391f *** empty log message ***
claus
parents: 88
diff changeset
   993
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
752
0259dd855289 new suspendAction, Semaphore & ProcSched stuff from stefan
Claus Gittinger <cg@exept.de>
parents: 699
diff changeset
   994
13579
db80734f9576 changed to remember lastOwner instead of lastOwnerID
Claus Gittinger <cg@exept.de>
parents: 13355
diff changeset
   995
    "Modified: / 13-12-1995 / 13:26:33 / stefan"
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
   996
    "Modified (format): / 21-02-2017 / 15:29:51 / stefan"
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
   997
    "Modified: / 24-07-2017 / 23:04:09 / cg"
10
claus
parents: 3
diff changeset
   998
!
claus
parents: 3
diff changeset
   999
329
claus
parents: 302
diff changeset
  1000
waitUncounted
claus
parents: 302
diff changeset
  1001
    "wait for the semaphore; do not consume the resource
claus
parents: 302
diff changeset
  1002
     (i.e. do not count down)"
claus
parents: 302
diff changeset
  1003
2142
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
  1004
    |activeProcess wasBlocked|
329
claus
parents: 302
diff changeset
  1005
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
  1006
    count > 0 ifTrue:[
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1007
        ^ self
329
claus
parents: 302
diff changeset
  1008
    ].
2142
d4a21668c699 commentary
Claus Gittinger <cg@exept.de>
parents: 1812
diff changeset
  1009
    activeProcess := Processor activeProcess.
329
claus
parents: 302
diff changeset
  1010
claus
parents: 302
diff changeset
  1011
    wasBlocked := OperatingSystem blockInterrupts.
claus
parents: 302
diff changeset
  1012
    "
claus
parents: 302
diff changeset
  1013
     need a while-loop here, since more than one process may
claus
parents: 302
diff changeset
  1014
     wait for it and another one may also wake up.
claus
parents: 302
diff changeset
  1015
     Thus, the count is not always non-zero after returning from
claus
parents: 302
diff changeset
  1016
     suspend.
claus
parents: 302
diff changeset
  1017
    "
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
  1018
    [count <= 0] whileTrue:[
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1019
        self addWaitingProcess:activeProcess.
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1020
        "
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1021
         for some more descriptive info in processMonitor ...
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1022
         ... set the state to #wait (instead of #suspend)
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1023
        "
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1024
        [
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1025
            activeProcess suspendWithState:#wait
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1026
        ] ifCurtailed:[
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1027
            "interrupts are not blocked when entered through Processor>>#interruptActive"
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1028
            OperatingSystem blockInterrupts.
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1029
            self removeWaitingProcess:activeProcess.
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1030
            wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1031
        ].
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1032
        self removeWaitingProcess:activeProcess.
329
claus
parents: 302
diff changeset
  1033
    ].
claus
parents: 302
diff changeset
  1034
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
752
0259dd855289 new suspendAction, Semaphore & ProcSched stuff from stefan
Claus Gittinger <cg@exept.de>
parents: 699
diff changeset
  1035
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1036
    "Modified: / 13-12-1995 / 13:26:49 / stefan"
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1037
    "Modified: / 24-07-2017 / 23:04:33 / cg"
329
claus
parents: 302
diff changeset
  1038
!
claus
parents: 302
diff changeset
  1039
16903
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1040
waitUncountedWithTimeoutMs:milliSeconds
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1041
    "wait for the semaphore; do not consume the resource
20263
9f3ddb907b77 #OTHER by mawalch
mawalch
parents: 20116
diff changeset
  1042
     (i.e. do not count down).
16903
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1043
     Abort the wait after some time.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1044
     return the receiver if the semaphore triggered normal, nil if we return
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1045
     due to a timeout.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1046
     With zero timeout, this can be used to poll a semaphore (returning
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1047
     the receiver if the semaphore is available, nil if not).
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1048
     However, polling is not the intended use of semaphores, though.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1049
     If milliSeconds is nil, wait without timeout."
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1050
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1051
    self waitUncountedWithTimeoutMs:milliSeconds state:#wait
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1052
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1053
    "Modified: / 13-12-1995 / 13:27:24 / stefan"
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1054
    "Modified: / 24-07-2017 / 21:53:57 / cg"
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1055
!
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1056
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1057
waitUncountedWithTimeoutMs:milliSeconds state:newStateSymbol
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1058
    "wait for the semaphore; do not consume the resource
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1059
     (i.e. do not count down).
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1060
     Abort the wait after some time.
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1061
     return the receiver if the semaphore triggered normal, nil if we return
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1062
     due to a timeout.
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1063
     With zero timeout, this can be used to poll a semaphore (returning
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1064
     the receiver if the semaphore is available, nil if not).
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1065
     However, polling is not the intended use of semaphores, though.
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1066
     If milliSeconds is nil, wait without timeout."
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1067
20263
9f3ddb907b77 #OTHER by mawalch
mawalch
parents: 20116
diff changeset
  1068
    |activeProcess timeoutOccurred wasBlocked timeoutBlock now endTime|
16903
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1069
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1070
    count > 0 ifTrue:[
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1071
        ^ self
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1072
    ].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1073
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1074
    wasBlocked := OperatingSystem blockInterrupts.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1075
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1076
    count <= 0 ifTrue:[
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1077
        "with zero-timeout, this is a poll"
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1078
        milliSeconds == 0 ifTrue:[
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1079
            wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1080
            ^ nil
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1081
        ].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1082
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1083
        activeProcess := Processor activeProcess.
20263
9f3ddb907b77 #OTHER by mawalch
mawalch
parents: 20116
diff changeset
  1084
        timeoutOccurred := false.
16903
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1085
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1086
        milliSeconds notNil ifTrue:[
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1087
            "Wait with timeout: calculate the end-time"
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1088
            now := OperatingSystem getMillisecondTime.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1089
            endTime := OperatingSystem millisecondTimeAdd:now and:milliSeconds.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1090
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1091
            timeoutBlock := [
20263
9f3ddb907b77 #OTHER by mawalch
mawalch
parents: 20116
diff changeset
  1092
                    timeoutOccurred := true.
16903
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1093
                    timeoutBlock:= nil.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1094
                    Processor resume:activeProcess.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1095
                ].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1096
            Processor addTimedBlock:timeoutBlock for:activeProcess atMilliseconds:endTime.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1097
        ].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1098
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1099
        "
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1100
         need a while-loop here, since more than one process may
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1101
         wait for it and another one may also wake up.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1102
         Thus, the count is not always non-zero after returning from
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1103
         suspend.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1104
        "
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1105
        [
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1106
            self addWaitingProcess:activeProcess.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1107
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1108
            "
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1109
             for some more descriptive info in processMonitor ...
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1110
             ... set the state to #wait (instead of #suspend)
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1111
            "
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1112
            [
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1113
                "sleep until resumed..."
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1114
                activeProcess suspendWithState:newStateSymbol.
16903
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1115
            ] ifCurtailed:[
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1116
                "interrupts are not blocked when entered through Processor>>#interruptActive"
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1117
                OperatingSystem blockInterrupts.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1118
                timeoutBlock notNil ifTrue:[
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1119
                    Processor removeTimedBlock:timeoutBlock.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1120
                    timeoutBlock := nil.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1121
                ].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1122
                self removeWaitingProcess:activeProcess.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1123
                wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1124
            ].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1125
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1126
            self removeWaitingProcess:activeProcess.
20263
9f3ddb907b77 #OTHER by mawalch
mawalch
parents: 20116
diff changeset
  1127
            timeoutOccurred ifTrue:[
16903
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1128
                wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1129
                ^ nil
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1130
            ].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1131
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1132
            count <= 0
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1133
        ] whileTrue.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1134
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1135
        timeoutBlock notNil ifTrue:[
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1136
            Processor removeTimedBlock:timeoutBlock.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1137
            timeoutBlock := nil.
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1138
        ].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1139
    ].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1140
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1141
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1142
    ^ self
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1143
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1144
    "Created: / 24-07-2017 / 21:53:23 / cg"
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1145
    "Modified: / 24-07-2017 / 23:06:00 / cg"
16903
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1146
!
b8fc32d0daf3 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16631
diff changeset
  1147
19192
62dba7ef04eb #FEATURE
Stefan Vogel <sv@exept.de>
parents: 19189
diff changeset
  1148
waitWithTimeout:secondsOrNilOrTimeDuration
12697
c70a0ec49f43 comment/format in: #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 12589
diff changeset
  1149
    "wait for the semaphore, but abort the wait after some time (seconds).
2690
f79d685e06af checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2407
diff changeset
  1150
     return the receiver if the semaphore triggered normal, nil if we return
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
  1151
     due to a timeout.
926
101239898989 unwind block over wait (in critical)
Claus Gittinger <cg@exept.de>
parents: 770
diff changeset
  1152
     The seconds-argument may be a float (i.e. use 0.1 for a 100ms timeout).
2690
f79d685e06af checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2407
diff changeset
  1153
     With zero timeout, this can be used to poll a semaphore (returning
f79d685e06af checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2407
diff changeset
  1154
     the receiver if the semaphore is available, nil if not).
4902
caf73c9821c5 Allow nil as argument to #waitWithTimeout: (no timeout) (stefan)
Claus Gittinger <cg@exept.de>
parents: 4633
diff changeset
  1155
     However, polling is not the intended use of semaphores, though.
caf73c9821c5 Allow nil as argument to #waitWithTimeout: (no timeout) (stefan)
Claus Gittinger <cg@exept.de>
parents: 4633
diff changeset
  1156
     If seconds is nil, wait without timeout."
159
514c749165c3 *** empty log message ***
claus
parents: 93
diff changeset
  1157
6061
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1158
    |millis|
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1159
19192
62dba7ef04eb #FEATURE
Stefan Vogel <sv@exept.de>
parents: 19189
diff changeset
  1160
    secondsOrNilOrTimeDuration notNil ifTrue:[
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1161
        secondsOrNilOrTimeDuration isNumber ifTrue:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1162
            millis := secondsOrNilOrTimeDuration.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1163
        ] ifFalse:[
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1164
            "converts a TimeDuration to a Fraction"
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1165
            millis := secondsOrNilOrTimeDuration asFraction.
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1166
        ].
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1167
        millis := (millis * 1000) asInteger
6061
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1168
    ].
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1169
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1170
    ^ self waitWithTimeoutMs:millis state:#wait.
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1171
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1172
    "Modified: / 21-02-2017 / 14:49:08 / stefan"
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1173
    "Modified: / 24-07-2017 / 21:15:39 / cg"
6061
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1174
!
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1175
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1176
waitWithTimeoutMs:milliSeconds
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1177
    "wait for the semaphore, but abort the wait after some time.
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1178
     return the receiver if the semaphore triggered normal, nil if we return
16625
09b045c29e25 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16286
diff changeset
  1179
     due to a timeout.
6061
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1180
     With zero timeout, this can be used to poll a semaphore (returning
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1181
     the receiver if the semaphore is available, nil if not).
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1182
     However, polling is not the intended use of semaphores, though.
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1183
     If milliSeconds is nil, wait without timeout."
9e266783d5c2 care for nil arg in #waitWithTimeout:
Claus Gittinger <cg@exept.de>
parents: 5713
diff changeset
  1184
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1185
    ^ self waitWithTimeoutMs:milliSeconds state:#wait
6615
2b56a43d0e89 allow for negative counts.
Claus Gittinger <cg@exept.de>
parents: 6601
diff changeset
  1186
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1187
    "Modified: / 21-02-2017 / 15:21:30 / stefan"
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1188
    "Modified: / 24-07-2017 / 21:15:25 / cg"
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1189
!
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1190
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1191
waitWithTimeoutMs:milliSeconds state:waitStateSymbol
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1192
    "wait for the semaphore, but abort the wait after some time.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1193
     return the receiver if the semaphore triggered normal, nil if we return
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1194
     due to a timeout.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1195
     With zero timeout, this can be used to poll a semaphore (returning
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1196
     the receiver if the semaphore is available, nil if not).
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1197
     However, polling is not the intended use of semaphores, though.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1198
     If milliSeconds is nil, wait without timeout.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1199
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1200
     THIS IS A COPY of #waitWithTimeoutMs - the only difference is setting waitStateSymbol.
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1201
     waitStateSymbol is the state the process is set to while waiting - normally #wait."
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1202
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1203
    |activeProcess timeoutOccurred wasBlocked timeoutBlock endTime|
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1204
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1205
    wasBlocked := OperatingSystem blockInterrupts.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1206
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1207
    count <= 0 ifTrue:[
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1208
        "with zero-timeout, this is a poll"
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1209
        milliSeconds == 0 ifTrue:[
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1210
            wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1211
            ^ nil
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1212
        ].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1213
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1214
        activeProcess := Processor activeProcess.
20263
9f3ddb907b77 #OTHER by mawalch
mawalch
parents: 20116
diff changeset
  1215
        timeoutOccurred := false.
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1216
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1217
        milliSeconds notNil ifTrue:[
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1218
            "Wait with timeout: calculate the end-time"
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1219
            endTime := OperatingSystem 
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1220
                        millisecondTimeAdd:OperatingSystem getMillisecondTime
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1221
                        and:milliSeconds.
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1222
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1223
            timeoutBlock := [
20263
9f3ddb907b77 #OTHER by mawalch
mawalch
parents: 20116
diff changeset
  1224
                    timeoutOccurred := true.
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1225
                    timeoutBlock:= nil.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1226
                    Processor resume:activeProcess.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1227
                ].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1228
            Processor addTimedBlock:timeoutBlock for:activeProcess atMilliseconds:endTime.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1229
        ].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1230
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1231
        "
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1232
         need a while-loop here, since more than one process may
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1233
         wait for it and another one may also wake up.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1234
         Thus, the count is not always non-zero after returning from
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1235
         suspend.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1236
        "
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1237
        [
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1238
            self addWaitingProcess:activeProcess.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1239
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1240
            "
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1241
             for some more descriptive info in processMonitor ...
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1242
             ... set the state to waitStateSymbol (instead of #suspend)
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1243
            "
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1244
            [
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1245
                "sleep until resumed..."
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1246
                activeProcess suspendWithState:waitStateSymbol.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1247
            ] ifCurtailed:[
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1248
                "interrupts are not blocked when entered through Processor>>#interruptActive"
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1249
                OperatingSystem blockInterrupts.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1250
                timeoutBlock notNil ifTrue:[
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1251
                    Processor removeTimedBlock:timeoutBlock.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1252
                    timeoutBlock := nil.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1253
                ].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1254
                self removeWaitingProcess:activeProcess.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1255
                wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1256
            ].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1257
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1258
            self removeWaitingProcess:activeProcess.
20263
9f3ddb907b77 #OTHER by mawalch
mawalch
parents: 20116
diff changeset
  1259
            timeoutOccurred ifTrue:[
16631
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1260
                wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1261
                ^ nil
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1262
            ].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1263
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1264
            count <= 0
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1265
        ] whileTrue.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1266
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1267
        timeoutBlock notNil ifTrue:[
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1268
            Processor removeTimedBlock:timeoutBlock.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1269
            timeoutBlock := nil.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1270
        ].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1271
    ].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1272
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1273
    "if we come here, we have acquired the semaphore"
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1274
    count := count - 1.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1275
    count == 0 ifTrue:[
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1276
        lastOwnerId := Processor activeProcessId.
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1277
    ].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1278
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
ba77908696d0 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 16625
diff changeset
  1279
    ^ self
22076
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1280
a0ca167d3248 #REFACTORING by stefan
Stefan Vogel <sv@exept.de>
parents: 21556
diff changeset
  1281
    "Modified: / 21-02-2017 / 15:21:39 / stefan"
22080
379aa47c41ad #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 22076
diff changeset
  1282
    "Modified: / 24-07-2017 / 23:04:57 / cg"
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
  1283
! !
77
6c38ca59927f *** empty log message ***
claus
parents: 69
diff changeset
  1284
1812
a048a98aa299 kludge around a situation, where one process
Claus Gittinger <cg@exept.de>
parents: 1577
diff changeset
  1285
!Semaphore class methodsFor:'documentation'!
183
7899f380e77d signalIf
claus
parents: 159
diff changeset
  1286
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 560
diff changeset
  1287
version
19189
48f29d2c0455 #DOCUMENTATION
matilk
parents: 16903
diff changeset
  1288
    ^ '$Header$'
12567
eb78ade59632 added: #waitTimeoutSeconds:
Stefan Vogel <sv@exept.de>
parents: 11281
diff changeset
  1289
!
eb78ade59632 added: #waitTimeoutSeconds:
Stefan Vogel <sv@exept.de>
parents: 11281
diff changeset
  1290
eb78ade59632 added: #waitTimeoutSeconds:
Stefan Vogel <sv@exept.de>
parents: 11281
diff changeset
  1291
version_CVS
19189
48f29d2c0455 #DOCUMENTATION
matilk
parents: 16903
diff changeset
  1292
    ^ '$Header$'
1
a27a279701f8 Initial revision
claus
parents:
diff changeset
  1293
! !
14763
db01620704a8 class: Semaphore
Stefan Vogel <sv@exept.de>
parents: 14537
diff changeset
  1294