<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<p>Thank you, Marcel! :-)</p>
<div><br>
</div>
<div>Best,</div>
<div>Christoph</div>
<br>
<div style="color: rgb(0, 0, 0);">
<div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>Von:</b> Squeak-dev <squeak-dev-bounces@lists.squeakfoundation.org> im Auftrag von commits@source.squeak.org <commits@source.squeak.org><br>
<b>Gesendet:</b> Freitag, 26. März 2021 12:51 Uhr<br>
<b>An:</b> squeak-dev@lists.squeakfoundation.org; packages@lists.squeakfoundation.org<br>
<b>Betreff:</b> [squeak-dev] The Trunk: Kernel-mt.1383.mcz</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Marcel Taeumel uploaded a new version of Kernel to project The Trunk:<br>
<a href="http://source.squeak.org/trunk/Kernel-mt.1383.mcz">http://source.squeak.org/trunk/Kernel-mt.1383.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: Kernel-mt.1383<br>
Author: mt<br>
Time: 26 March 2021, 12:51:18.904073 pm<br>
UUID: 524a87b1-28d6-f044-8a9a-4ca045211ac1<br>
Ancestors: Kernel-ct.1382<br>
<br>
Merges Kernel-ct.1382<br>
<br>
=============== Diff against Kernel-mt.1381 ===============<br>
<br>
Item was changed:<br>
  ----- Method: BlockClosure>>newProcess (in category 'scheduling') -----<br>
  newProcess<br>
+        "Answer a Process running the code in the receiver. The process is not scheduled."<br>
-        "Answer a Process running the code in the receiver. The process is not <br>
-        scheduled."<br>
-        <primitive: 19> "Simulation guard"<br>
         ^Process<br>
                 forContext: <br>
                         [self value.<br>
+                        "Since control is now at the bottom, there is no need to terminate (which runs unwinds) since all unwinds have been run.  Simply suspend.<br>
+                         Note that we must use this form rather than e.g. Processor suspendActive so that isTerminated answers true.  isTerminated requires that if there is a suspended context it is the bottom-most, but using a send would result in the process's
 suspendedContext /not/ being the bottom-most."<br>
-                        "Since control is now at the bottom there is no need to terminate (which<br>
-                         runs unwinds) since all unwnds have been run.  Simply suspend.<br>
-                         Note that we must use this form rather than e.g. Processor suspendActive<br>
-                         so that isTerminated answers true.  isTerminated requires that if there is a<br>
-                         suspended context it is the bottom-most, but using a send would result in<br>
-                         the process's suspendedContext /not/ being the bottom-most."<br>
                         Processor activeProcess suspend] asContext<br>
                 priority: Processor activePriority!<br>
<br>
Item was changed:<br>
  ----- Method: BlockClosure>>newProcessWith: (in category 'scheduling') -----<br>
  newProcessWith: anArray <br>
+        "Answer a Process running the code in the receiver. The receiver's block arguments are bound to the contents of the argument, anArray. The process is not scheduled."<br>
-        "Answer a Process running the code in the receiver. The receiver's block
<br>
-        arguments are bound to the contents of the argument, anArray. The <br>
-        process is not scheduled."<br>
-        <primitive: 19> "Simulation guard"<br>
         ^Process<br>
                 forContext: <br>
                         [self valueWithArguments: anArray.<br>
+                        "Since control is now at the bottom, there is no need to terminate (which runs unwinds) since all unwinds have been run.  Simply suspend.<br>
+                         Note that we must use this form rather than e.g. Processor suspendActive so that isTerminated answers true.  isTerminated requires that if there is a suspended context it is the bottom-most, but using a send would result in the process's
 suspendedContext /not/ being the bottom-most."<br>
-                        "Since control is now at the bottom there is no need to terminate (which<br>
-                         runs unwinds) since all unwnds have been run.  Simply suspend.<br>
-                         Note that we must use this form rather than e.g. Processor suspendActive<br>
-                         so that isTerminated answers true.  isTerminated requires that if there is a<br>
-                         suspended context it is the bottom-most, but using a send would result in<br>
-                         the process's suspendedContext /not/ being the bottom-most."<br>
                         Processor activeProcess suspend] asContext<br>
                 priority: Processor activePriority!<br>
<br>
Item was changed:<br>
  ----- Method: Context>>doPrimitive:method:receiver:args: (in category 'private') -----<br>
  doPrimitive: primitiveIndex method: meth receiver: receiver args: arguments<br>
         "Simulate a primitive method whose index is primitiveIndex.  The simulated receiver and<br>
          arguments are given as arguments to this message. If successful, push result and return<br>
          resuming context, else ^ {errCode, PrimitiveFailToken}. Any primitive which provokes<br>
          execution needs to be intercepted and simulated to avoid execution running away."<br>
  <br>
         | value |<br>
         "Judicious use of primitive 19 (a null primitive that doesn't do anything) prevents<br>
          the debugger from entering various run-away activities such as spawning a new<br>
          process, etc.  Injudicious use results in the debugger not being able to debug<br>
          interesting code, such as the debugger itself.  Hence use primitive 19 with care :-)"<br>
         "SystemNavigation new browseAllSelect: [:m| m primitive = 19]"<br>
+        (primitiveIndex = 19 or: [primitiveIndex >= 85 and: [primitiveIndex <= 88] "control primitives"]) ifTrue: [<br>
-        primitiveIndex = 19 ifTrue: [<br>
                 [self notify: ('The code being simulated is trying to control a process ({1}). Process controlling cannot be simulated. If you proceed, things may happen outside the observable area of the simulator.' translated format: {meth reference})]<br>
                         ifCurtailed: [self push: nil "Cheap fix of the context's internal state"]].<br>
         <br>
         ((primitiveIndex between: 201 and: 222)<br>
          and: [(self objectClass: receiver) includesBehavior: BlockClosure]) ifTrue:<br>
                 [(primitiveIndex = 206<br>
                   or: [primitiveIndex = 208]) ifTrue:                                            "[Full]BlockClosure>>valueWithArguments:"<br>
                         [^receiver simulateValueWithArguments: arguments first caller: self].<br>
                  ((primitiveIndex between: 201 and: 209)                  "[Full]BlockClosure>>value[:value:...]"<br>
                   or: [primitiveIndex between: 221 and: 222]) ifTrue: "[Full]BlockClosure>>valueNoContextSwitch[:]"<br>
                         [^receiver simulateValueWithArguments: arguments caller: self]].<br>
  <br>
         primitiveIndex = 83 ifTrue: "afr 9/11/1998 19:50" "Object>>perform:[with:...]"<br>
                 [^self send: arguments first to: receiver with: arguments allButFirst].<br>
         primitiveIndex = 84 ifTrue: "afr 9/11/1998 19:50 & eem 8/18/2009 17:04" "Object>>perform:withArguments:"<br>
                 [^self send: arguments first to: receiver with: (arguments at: 2) lookupIn: (self objectClass: receiver)].<br>
         primitiveIndex = 100 ifTrue: "eem 8/18/2009 16:57" "Object>>perform:withArguments:inSuperclass:"<br>
                 [^self send: arguments first to: receiver with: (arguments at: 2) lookupIn: (arguments at: 3)].<br>
  <br>
         "Mutex>>primitiveEnterCriticalSection<br>
          Mutex>>primitiveTestAndSetOwnershipOfCriticalSection"<br>
         (primitiveIndex = 186 or: [primitiveIndex = 187]) ifTrue:<br>
+                [value := primitiveIndex = 186<br>
+                                        ifTrue: [receiver primitiveEnterCriticalSectionOnBehalfOf: Processor activeProcess]<br>
+                                        ifFalse: [receiver primitiveTestAndSetOwnershipOfCriticalSectionOnBehalfOf: Processor activeProcess].<br>
-                [| effective |<br>
-                 effective := Processor activeProcess effectiveProcess.<br>
-                 "active == effective"<br>
-                 value := primitiveIndex = 186<br>
-                                        ifTrue: [receiver primitiveEnterCriticalSectionOnBehalfOf: effective]<br>
-                                        ifFalse: [receiver primitiveTestAndSetOwnershipOfCriticalSectionOnBehalfOf: effective].<br>
                  ^(self isPrimFailToken: value)<br>
                         ifTrue: [value]<br>
                         ifFalse: [self push: value]].<br>
  <br>
         primitiveIndex = 188 ifTrue:    "Object>>withArgs:executeMethod:<br>
                                                                         CompiledMethod class>>receiver:withArguments:executeMethod:<br>
                                                                         VMMirror>>ifFail:object:with:executeMethod: et al"<br>
                 [| n args methodArg thisReceiver |<br>
                  ((n := arguments size) between: 2 and: 4) ifFalse:<br>
                         [^self class primitiveFailTokenFor: #'unsupported operation'].<br>
                  ((self objectClass: (args := arguments at: n - 1)) == Array<br>
                   and: [(self objectClass: (methodArg := arguments at: n)) includesBehavior: CompiledMethod]) ifFalse:<br>
                         [^self class primitiveFailTokenFor: #'bad argument'].<br>
                  methodArg numArgs = args size ifFalse:<br>
                         [^self class primitiveFailTokenFor: #'bad number of arguments'].<br>
                  thisReceiver := arguments at: n - 2 ifAbsent: [receiver].<br>
                  methodArg primitive > 0 ifTrue:<br>
                         [methodArg isQuick ifTrue:<br>
                                 [^self push: (methodArg valueWithReceiver: thisReceiver arguments: args)].<br>
                          ^self doPrimitive: methodArg primitive method: meth receiver: thisReceiver args: args].<br>
                  ^Context<br>
                         sender: self<br>
                         receiver: thisReceiver<br>
                         method: methodArg<br>
                         arguments: args].<br>
  <br>
         primitiveIndex = 118 ifTrue: "[receiver:]tryPrimitive:withArgs:; avoid recursing in the VM"<br>
                 [(arguments size = 3<br>
                   and: [(self objectClass: arguments second) == SmallInteger<br>
                   and: [(self objectClass: arguments last) == Array]]) ifTrue:<br>
                         [^self doPrimitive: arguments second method: meth receiver: arguments first args: arguments last].<br>
                  (arguments size = 2<br>
                  and: [(self objectClass: arguments first) == SmallInteger<br>
                  and: [(self objectClass: arguments last) == Array]]) ifFalse:<br>
                         [^self class primitiveFailTokenFor: nil].<br>
                  ^self doPrimitive: arguments first method: meth receiver: receiver args: arguments last].<br>
  <br>
         value := primitiveIndex = 120 "FFI method"<br>
                                 ifTrue: [(meth literalAt: 1) tryInvokeWithArguments: arguments]<br>
                                 ifFalse:<br>
                                         [primitiveIndex = 117 "named primitives"<br>
                                                 ifTrue: [self tryNamedPrimitiveIn: meth for: receiver withArgs: arguments]<br>
                                                 ifFalse: "should use self receiver: receiver tryPrimitive: primitiveIndex withArgs: arguments but this is only in later VMs (and appears to be broken)"<br>
                                                         [receiver tryPrimitive: primitiveIndex withArgs: arguments]].<br>
  <br>
         ^(self isPrimFailToken: value)<br>
                 ifTrue: [value]<br>
                 ifFalse: [self push: value]!<br>
<br>
<br>
</div>
</span></font></div>
</div>
</body>
</html>