RecursionLock.st
author Merge Script
Sat, 12 Sep 2015 06:54:38 +0200
branchjv
changeset 18749 d6947ad2feaf
parent 18120 e3a375d5f6a8
parent 18748 f7400cf1f488
child 20131 4118d61ddba0
permissions -rw-r--r--
Merge
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
18748
f7400cf1f488 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16904
diff changeset
     1
"{ Encoding: utf8 }"
f7400cf1f488 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16904
diff changeset
     2
330
claus
parents:
diff changeset
     3
"
claus
parents:
diff changeset
     4
 COPYRIGHT (c) 1995 by Claus Gittinger
341
claus
parents: 330
diff changeset
     5
	      All Rights Reserved
330
claus
parents:
diff changeset
     6
claus
parents:
diff changeset
     7
 This software is furnished under a license and may be used
claus
parents:
diff changeset
     8
 only in accordance with the terms of that license and with the
claus
parents:
diff changeset
     9
 inclusion of the above copyright notice.   This software may not
claus
parents:
diff changeset
    10
 be provided or otherwise made available to, or used by, any
claus
parents:
diff changeset
    11
 other person.  No title to or ownership of the software is
claus
parents:
diff changeset
    12
 hereby transferred.
claus
parents:
diff changeset
    13
"
6552
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    14
"{ Package: 'stx:libbasic' }"
5735
3c1f1f115640 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 5104
diff changeset
    15
18748
f7400cf1f488 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16904
diff changeset
    16
"{ NameSpace: Smalltalk }"
f7400cf1f488 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16904
diff changeset
    17
330
claus
parents:
diff changeset
    18
Object subclass:#RecursionLock
776
f3c0c579c0d2 commentary
Claus Gittinger <cg@exept.de>
parents: 699
diff changeset
    19
	instanceVariableNames:'process sema'
f3c0c579c0d2 commentary
Claus Gittinger <cg@exept.de>
parents: 699
diff changeset
    20
	classVariableNames:''
f3c0c579c0d2 commentary
Claus Gittinger <cg@exept.de>
parents: 699
diff changeset
    21
	poolDictionaries:''
f3c0c579c0d2 commentary
Claus Gittinger <cg@exept.de>
parents: 699
diff changeset
    22
	category:'Kernel-Processes'
330
claus
parents:
diff changeset
    23
!
claus
parents:
diff changeset
    24
claus
parents:
diff changeset
    25
!RecursionLock class methodsFor:'documentation'!
claus
parents:
diff changeset
    26
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    27
copyright
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    28
"
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    29
 COPYRIGHT (c) 1995 by Claus Gittinger
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    30
	      All Rights Reserved
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    31
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    32
 This software is furnished under a license and may be used
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    33
 only in accordance with the terms of that license and with the
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    34
 inclusion of the above copyright notice.   This software may not
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    35
 be provided or otherwise made available to, or used by, any
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    36
 other person.  No title to or ownership of the software is
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    37
 hereby transferred.
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    38
"
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    39
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    40
!
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
    41
330
claus
parents:
diff changeset
    42
documentation
claus
parents:
diff changeset
    43
"
claus
parents:
diff changeset
    44
    like a Semaphore for mutual exclusion, but avoids the deadlock
claus
parents:
diff changeset
    45
    if a critical region is reentered by the same process again.
2143
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    46
    I.e. allows reentering the critical region IFF the current process 
341
claus
parents: 330
diff changeset
    47
    is the one which did the original locking.
330
claus
parents:
diff changeset
    48
3524
b5fe623f0e4d warning in documentation
Claus Gittinger <cg@exept.de>
parents: 3326
diff changeset
    49
    WARNING:
6552
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    50
	for now, recursionLocks are not unlocked when an image is
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    51
	restarted. You may have to recreate them to avoid a deadLock.
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    52
	(this may change in the future, but recreating a recursionLock in
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    53
	 the #earlyRestart handling does not hurt)
1294
e26bbb61f6b2 documentation
Claus Gittinger <cg@exept.de>
parents: 1273
diff changeset
    54
e26bbb61f6b2 documentation
Claus Gittinger <cg@exept.de>
parents: 1273
diff changeset
    55
    [author:]
6552
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    56
	Claus Gittinger
1294
e26bbb61f6b2 documentation
Claus Gittinger <cg@exept.de>
parents: 1273
diff changeset
    57
1273
f8449f53a6a3 commentary
Claus Gittinger <cg@exept.de>
parents: 1216
diff changeset
    58
    [see also:]
6552
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    59
	Semaphore
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    60
	Process ProcessorScheduler
1273
f8449f53a6a3 commentary
Claus Gittinger <cg@exept.de>
parents: 1216
diff changeset
    61
"
f8449f53a6a3 commentary
Claus Gittinger <cg@exept.de>
parents: 1216
diff changeset
    62
!
f8449f53a6a3 commentary
Claus Gittinger <cg@exept.de>
parents: 1216
diff changeset
    63
f8449f53a6a3 commentary
Claus Gittinger <cg@exept.de>
parents: 1216
diff changeset
    64
examples
f8449f53a6a3 commentary
Claus Gittinger <cg@exept.de>
parents: 1216
diff changeset
    65
"
2143
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    66
  example (good):
6552
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    67
									[exBegin]
2143
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    68
    |lock|
776
f3c0c579c0d2 commentary
Claus Gittinger <cg@exept.de>
parents: 699
diff changeset
    69
2143
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    70
    lock := RecursionLock new.
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    71
    lock critical:[
6552
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    72
	Transcript showCR:'in lock ...'.
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    73
	lock critical:[
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    74
	    Transcript showCR:'again ...'
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    75
	]
2143
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    76
    ]
6552
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    77
									[exEnd]
776
f3c0c579c0d2 commentary
Claus Gittinger <cg@exept.de>
parents: 699
diff changeset
    78
2143
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    79
  in contrast to (wrong example - deadlocks):
6552
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    80
									[exBegin]
2143
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    81
    |lock|
776
f3c0c579c0d2 commentary
Claus Gittinger <cg@exept.de>
parents: 699
diff changeset
    82
2143
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    83
    lock := Semaphore forMutualExclusion.
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    84
    lock critical:[
6552
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    85
	Transcript showCR:'in lock ...'.
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    86
	lock critical:[
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    87
	    '*** never reached - deadlock because sema is already locked ***'.
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    88
	    '    (press CTRL-c and abort in the debugger)'.
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    89
	    Transcript showCR:'again ...'
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    90
	]
2143
09af9c997961 commentary
Claus Gittinger <cg@exept.de>
parents: 1422
diff changeset
    91
    ]
6552
5ebab8a33da9 Moved from libbasic2 to libbasic
Stefan Vogel <sv@exept.de>
parents: 5735
diff changeset
    92
									[exEnd]
330
claus
parents:
diff changeset
    93
"
claus
parents:
diff changeset
    94
! !
claus
parents:
diff changeset
    95
claus
parents:
diff changeset
    96
!RecursionLock class methodsFor:'instance creation'!
claus
parents:
diff changeset
    97
5735
3c1f1f115640 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 5104
diff changeset
    98
forMutualExclusion
18748
f7400cf1f488 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16904
diff changeset
    99
    "same as new, for easy exchangability with regular mutual-exclusion Semaphores."
5735
3c1f1f115640 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 5104
diff changeset
   100
3c1f1f115640 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 5104
diff changeset
   101
    ^ self new 
3c1f1f115640 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 5104
diff changeset
   102
!
3c1f1f115640 checkin from browser
Claus Gittinger <cg@exept.de>
parents: 5104
diff changeset
   103
330
claus
parents:
diff changeset
   104
new
claus
parents:
diff changeset
   105
    ^ self basicNew initialize
claus
parents:
diff changeset
   106
claus
parents:
diff changeset
   107
! !
claus
parents:
diff changeset
   108
2722
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   109
!RecursionLock methodsFor:'printing & storing'!
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   110
13356
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   111
displayOn:aGCOrStream
2722
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   112
    "return a string to display the receiver - include the
13356
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   113
     count for your convenience"
2722
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   114
13356
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   115
    "/ what a kludge - Dolphin and Squeak mean: printOn: a stream;
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   116
    "/ ST/X (and some old ST80's) mean: draw-yourself on a GC.
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   117
    (aGCOrStream isStream) ifFalse:[
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   118
        ^ super displayOn:aGCOrStream
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   119
    ].
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   120
    aGCOrStream
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   121
        nextPutAll:self class name;
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   122
        nextPut:$(.
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   123
    sema count printOn:aGCOrStream.
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   124
    aGCOrStream nextPutAll:' name: '.
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   125
    (self name ? 'unnamed') printOn:aGCOrStream.
42b4aec86a14 added: #displayOn: (instead of displayString)
Stefan Vogel <sv@exept.de>
parents: 12681
diff changeset
   126
    aGCOrStream nextPut:$).
2722
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   127
!
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   128
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   129
name
5104
99233ab3f6f5 comments
Claus Gittinger <cg@exept.de>
parents: 4938
diff changeset
   130
    "return the semaphores userFriendly name"
99233ab3f6f5 comments
Claus Gittinger <cg@exept.de>
parents: 4938
diff changeset
   131
2722
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   132
    ^ sema name
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   133
5104
99233ab3f6f5 comments
Claus Gittinger <cg@exept.de>
parents: 4938
diff changeset
   134
    "Created: / 28.6.1997 / 16:19:40 / cg"
99233ab3f6f5 comments
Claus Gittinger <cg@exept.de>
parents: 4938
diff changeset
   135
    "Modified: / 14.12.1999 / 21:03:46 / cg"
2722
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   136
!
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   137
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   138
name:aString
5104
99233ab3f6f5 comments
Claus Gittinger <cg@exept.de>
parents: 4938
diff changeset
   139
    "set the semaphores userFriendly name"
99233ab3f6f5 comments
Claus Gittinger <cg@exept.de>
parents: 4938
diff changeset
   140
4938
d5640f1c8894 Add #numberOfWaitingProcesse
Stefan Vogel <sv@exept.de>
parents: 3670
diff changeset
   141
    sema name:aString
2722
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   142
5104
99233ab3f6f5 comments
Claus Gittinger <cg@exept.de>
parents: 4938
diff changeset
   143
    "Created: / 28.6.1997 / 16:19:47 / cg"
99233ab3f6f5 comments
Claus Gittinger <cg@exept.de>
parents: 4938
diff changeset
   144
    "Modified: / 14.12.1999 / 21:03:52 / cg"
2722
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   145
! !
f6c7146fced2 added naming protocol
Claus Gittinger <cg@exept.de>
parents: 2266
diff changeset
   146
7258
9ccdbee7d1ad method category rename
Claus Gittinger <cg@exept.de>
parents: 6985
diff changeset
   147
!RecursionLock methodsFor:'private-initialization'!
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   148
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   149
initialize
2266
a94af740c68a checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2263
diff changeset
   150
    sema := Semaphore forMutualExclusion name:'recursionLock'
a94af740c68a checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2263
diff changeset
   151
a94af740c68a checkin from browser
Claus Gittinger <cg@exept.de>
parents: 2263
diff changeset
   152
    "Modified: 25.1.1997 / 00:19:15 / cg"
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   153
! !
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   154
1215
43e8e17fd9f5 added #wouldBlock
Claus Gittinger <cg@exept.de>
parents: 1186
diff changeset
   155
!RecursionLock methodsFor:'queries'!
43e8e17fd9f5 added #wouldBlock
Claus Gittinger <cg@exept.de>
parents: 1186
diff changeset
   156
4938
d5640f1c8894 Add #numberOfWaitingProcesse
Stefan Vogel <sv@exept.de>
parents: 3670
diff changeset
   157
numberOfWaitingProcesses
d5640f1c8894 Add #numberOfWaitingProcesse
Stefan Vogel <sv@exept.de>
parents: 3670
diff changeset
   158
    "return the number of waiting processes"
d5640f1c8894 Add #numberOfWaitingProcesse
Stefan Vogel <sv@exept.de>
parents: 3670
diff changeset
   159
d5640f1c8894 Add #numberOfWaitingProcesse
Stefan Vogel <sv@exept.de>
parents: 3670
diff changeset
   160
    ^ sema numberOfWaitingProcesses
d5640f1c8894 Add #numberOfWaitingProcesse
Stefan Vogel <sv@exept.de>
parents: 3670
diff changeset
   161
d5640f1c8894 Add #numberOfWaitingProcesse
Stefan Vogel <sv@exept.de>
parents: 3670
diff changeset
   162
    "Created: 18.4.1996 / 17:18:08 / cg"
d5640f1c8894 Add #numberOfWaitingProcesse
Stefan Vogel <sv@exept.de>
parents: 3670
diff changeset
   163
!
d5640f1c8894 Add #numberOfWaitingProcesse
Stefan Vogel <sv@exept.de>
parents: 3670
diff changeset
   164
7428
84f4d9f08f04 owner & wouldBlock
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   165
owner
84f4d9f08f04 owner & wouldBlock
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   166
    "return the owning processes (or nil)"
84f4d9f08f04 owner & wouldBlock
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   167
84f4d9f08f04 owner & wouldBlock
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   168
    ^ process
84f4d9f08f04 owner & wouldBlock
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   169
!
1215
43e8e17fd9f5 added #wouldBlock
Claus Gittinger <cg@exept.de>
parents: 1186
diff changeset
   170
7428
84f4d9f08f04 owner & wouldBlock
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   171
wouldBlock
84f4d9f08f04 owner & wouldBlock
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   172
    "Check if the resource represented by the receiver is  
15550
8036fe8c032d class: RecursionLock
Claus Gittinger <cg@exept.de>
parents: 14779
diff changeset
   173
     already in use by another process.
8036fe8c032d class: RecursionLock
Claus Gittinger <cg@exept.de>
parents: 14779
diff changeset
   174
     Attention: if asked without some global lock (blockedInterrupts),
8036fe8c032d class: RecursionLock
Claus Gittinger <cg@exept.de>
parents: 14779
diff changeset
   175
     the returned value may be outdated right away."
1215
43e8e17fd9f5 added #wouldBlock
Claus Gittinger <cg@exept.de>
parents: 1186
diff changeset
   176
7428
84f4d9f08f04 owner & wouldBlock
Claus Gittinger <cg@exept.de>
parents: 7258
diff changeset
   177
    ^ process notNil and:[Processor activeProcess ~~ process]
1215
43e8e17fd9f5 added #wouldBlock
Claus Gittinger <cg@exept.de>
parents: 1186
diff changeset
   178
! !
43e8e17fd9f5 added #wouldBlock
Claus Gittinger <cg@exept.de>
parents: 1186
diff changeset
   179
16904
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   180
!RecursionLock methodsFor:'signaling'!
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   181
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   182
signal
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   183
    |wasBlocked|
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   184
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   185
    process ~~ Processor activeProcess ifTrue:[
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   186
        "I have already got the lock"
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   187
        self error:'RecursionLock - signaling process doesn''t own the lock'.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   188
    ].
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   189
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   190
    wasBlocked := OperatingSystem blockInterrupts.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   191
    process := nil.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   192
    sema signal.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   193
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   194
! !
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   195
8568
624bfd00371f category change
Stefan Vogel <sv@exept.de>
parents: 7590
diff changeset
   196
!RecursionLock methodsFor:'waiting'!
330
claus
parents:
diff changeset
   197
claus
parents:
diff changeset
   198
critical:aBlock
claus
parents:
diff changeset
   199
    "evaluate aBlock as a critical region, but do not block,
claus
parents:
diff changeset
   200
     if this lock is already held by the current process."
claus
parents:
diff changeset
   201
6985
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   202
    |active retVal wasBlocked gotSema|
330
claus
parents:
diff changeset
   203
claus
parents:
diff changeset
   204
    active := Processor activeProcess.
claus
parents:
diff changeset
   205
    process == active ifTrue:[
6985
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   206
        "I have already got the lock"
6603
bf9b8a0d489b Use #ifCurtailed: instead of #valueOnUnwindDo:
Stefan Vogel <sv@exept.de>
parents: 6552
diff changeset
   207
        ^ aBlock value
330
claus
parents:
diff changeset
   208
    ].
3670
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   209
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   210
    "/
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   211
    "/ sema wait & process := active
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   212
    "/ and:
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   213
    "/ proces := nil & sema signal
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   214
    "/ must both be done atomic
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   215
    "/ Scenario:
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   216
    "/   ... recLock critical
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   217
    "/         got lock
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   218
    "/         evaluated
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   219
    "/         set process to nil
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   220
    "/         -> timer interrupt
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   221
    "/              recLock critical in timeOut action
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   222
    "/              process isNil
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   223
    "/                 sema wait !!!!!! DEADLOCK
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   224
    "/
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   225
    wasBlocked := OperatingSystem blockInterrupts.
6985
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   226
    [
14697
e9ef6bbd0507 class: RecursionLock
Claus Gittinger <cg@exept.de>
parents: 13356
diff changeset
   227
        (process notNil and:[process isDead]) ifTrue:[
e9ef6bbd0507 class: RecursionLock
Claus Gittinger <cg@exept.de>
parents: 13356
diff changeset
   228
            'RecursionLock [warning]: cleanup lock from dead process' infoPrintCR.
e9ef6bbd0507 class: RecursionLock
Claus Gittinger <cg@exept.de>
parents: 13356
diff changeset
   229
            process := nil. sema signal.
e9ef6bbd0507 class: RecursionLock
Claus Gittinger <cg@exept.de>
parents: 13356
diff changeset
   230
        ].
6985
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   231
        gotSema := sema wait.
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   232
        process := active.
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   233
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   234
        retVal := aBlock value.
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   235
    ] ifCurtailed:[
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   236
        "be careful - the unwind may occur both while waiting
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   237
         AND while evaluating the block"
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   238
        gotSema notNil ifTrue:[
6603
bf9b8a0d489b Use #ifCurtailed: instead of #valueOnUnwindDo:
Stefan Vogel <sv@exept.de>
parents: 6552
diff changeset
   239
            OperatingSystem blockInterrupts.
bf9b8a0d489b Use #ifCurtailed: instead of #valueOnUnwindDo:
Stefan Vogel <sv@exept.de>
parents: 6552
diff changeset
   240
            process := nil.
6985
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   241
            sema signal.
6975
480fec265743 FIX RACE CONDITION
Stefan Vogel <sv@exept.de>
parents: 6603
diff changeset
   242
        ].
6985
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   243
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
3670
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   244
    ].
6985
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   245
    OperatingSystem blockInterrupts.
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   246
    process := nil.
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   247
    sema signal.
3670
91b4524a74b3 avoid an assignment if I already have the sema (in #critical)
Claus Gittinger <cg@exept.de>
parents: 3524
diff changeset
   248
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
6985
ca488dd4a56c Save a block in #critical:
Stefan Vogel <sv@exept.de>
parents: 6975
diff changeset
   249
    ^ retVal.
10860
cd924455cfa9 +critical:ifBlocking:
Claus Gittinger <cg@exept.de>
parents: 8568
diff changeset
   250
!
cd924455cfa9 +critical:ifBlocking:
Claus Gittinger <cg@exept.de>
parents: 8568
diff changeset
   251
cd924455cfa9 +critical:ifBlocking:
Claus Gittinger <cg@exept.de>
parents: 8568
diff changeset
   252
critical:aBlock ifBlocking:blockingBlock
cd924455cfa9 +critical:ifBlocking:
Claus Gittinger <cg@exept.de>
parents: 8568
diff changeset
   253
    "like critical:, but do not block if the lock cannot be aquired.
cd924455cfa9 +critical:ifBlocking:
Claus Gittinger <cg@exept.de>
parents: 8568
diff changeset
   254
     Instead, return the value of the second argument, blockingBlock."
cd924455cfa9 +critical:ifBlocking:
Claus Gittinger <cg@exept.de>
parents: 8568
diff changeset
   255
14779
4657463cd9f3 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 14697
diff changeset
   256
    ^ self critical:aBlock timeoutMs:0 ifBlocking:blockingBlock.
12681
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   257
!
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   258
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   259
critical:aBlock timeoutMs:timeoutMs ifBlocking:blockingBlock
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   260
    "like critical:, but do not block if the lock cannot be aquired 
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   261
     within timeoutMs milliseconds.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   262
     Instead, return the value of blockingBlock."
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   263
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   264
    |active retVal wasBlocked gotSema|
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   265
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   266
    active := Processor activeProcess.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   267
    process == active ifTrue:[
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   268
        "I have already got the lock"
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   269
        ^ aBlock value
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   270
    ].
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   271
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   272
    "/
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   273
    "/ sema wait & process := active
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   274
    "/ and:
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   275
    "/ proces := nil & sema signal
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   276
    "/ must both be done atomic
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   277
    "/ Scenario:
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   278
    "/   ... recLock critical
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   279
    "/         got lock
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   280
    "/         evaluated
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   281
    "/         set process to nil
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   282
    "/         -> timer interrupt
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   283
    "/              recLock critical in timeOut action
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   284
    "/              process isNil
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   285
    "/                 sema wait !!!!!! DEADLOCK
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   286
    "/
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   287
    wasBlocked := OperatingSystem blockInterrupts.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   288
    [
14779
4657463cd9f3 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 14697
diff changeset
   289
        (process notNil and:[process isDead]) ifTrue:[
4657463cd9f3 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 14697
diff changeset
   290
            'RecursionLock [warning]: cleanup lock from dead process' infoPrintCR.
4657463cd9f3 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 14697
diff changeset
   291
            process := nil. sema signal.
4657463cd9f3 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 14697
diff changeset
   292
        ].
12681
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   293
        gotSema := sema waitWithTimeoutMs:timeoutMs.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   294
        gotSema notNil ifTrue:[
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   295
            process := active.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   296
            wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   297
            retVal := aBlock value.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   298
        ].
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   299
    ] ifCurtailed:[
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   300
        "be careful - the unwind may occur both while waiting
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   301
         AND while evaluating the block"
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   302
        gotSema notNil ifTrue:[
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   303
            OperatingSystem blockInterrupts.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   304
            process := nil.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   305
            sema signal.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   306
        ].
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   307
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   308
    ].
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   309
    gotSema notNil ifTrue:[
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   310
        OperatingSystem blockInterrupts.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   311
        process := nil.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   312
        sema signal.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   313
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   314
    ] ifFalse:[
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   315
        wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   316
        retVal := blockingBlock value.
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   317
    ].
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   318
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   319
    ^ retVal.
16904
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   320
!
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   321
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   322
wait
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   323
    "wait, but do not block,
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   324
     if this lock is already held by the current process.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   325
     Answer false, if alread locked, true if lock has been just acquired."
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   326
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   327
    |active wasBlocked|
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   328
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   329
    active := Processor activeProcess.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   330
    process == active ifTrue:[
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   331
        "I have already got the lock"
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   332
        ^ false.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   333
    ].
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   334
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   335
    wasBlocked := OperatingSystem blockInterrupts.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   336
    (process notNil and:[process isDead]) ifTrue:[
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   337
        'RecursionLock [warning]: cleanup lock from dead process' infoPrintCR.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   338
        process := nil. sema signal.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   339
    ].
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   340
    sema wait.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   341
    process := active.
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   342
    wasBlocked ifFalse:[OperatingSystem unblockInterrupts].
838840a0be45 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16048
diff changeset
   343
    ^ true.
330
claus
parents:
diff changeset
   344
! !
claus
parents:
diff changeset
   345
699
12f456343eea checkin from browser
Claus Gittinger <cg@exept.de>
parents: 530
diff changeset
   346
!RecursionLock class methodsFor:'documentation'!
330
claus
parents:
diff changeset
   347
1216
d7cbc6eb8dd4 critical was critical (deadlock with timeouts)
Claus Gittinger <cg@exept.de>
parents: 1215
diff changeset
   348
version
18748
f7400cf1f488 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16904
diff changeset
   349
    ^ '$Header$'
12681
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   350
!
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   351
Stefan Vogel <sv@exept.de>
parents: 11769
diff changeset
   352
version_CVS
18748
f7400cf1f488 class: RecursionLock
Stefan Vogel <sv@exept.de>
parents: 16904
diff changeset
   353
    ^ '$Header$'
330
claus
parents:
diff changeset
   354
! !
14697
e9ef6bbd0507 class: RecursionLock
Claus Gittinger <cg@exept.de>
parents: 13356
diff changeset
   355