- On store buffer block overflow call a VM leaf function to

vm/stub_code_x64.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/globals.h"
 #if defined(TARGET_ARCH_X64)
 
 #include "vm/assembler.h"
 #include "vm/assembler_macros.h"
 #include "vm/compiler.h"
@@ skipping 73 matching lines @@
 
   // Cache Context pointer into CTX while executing Dart code.
   __ movq(CTX, RBX);
 
   __ LeaveFrame();
   __ ret();
 }
 
 
 // Print the stop message.
-static void PrintStopMessage(const char* message) {
+DEFINE_LEAF_RUNTIME_ENTRY(void, PrintStopMessage, const char* message) {
   OS::Print("Stop message: %s\n", message);
 }
+END_LEAF_RUNTIME_ENTRY
 
 
 // Input parameters:
 //   RSP : points to return address.
 //   RDI : stop message (const char*).
-// Must preserve all registers, except RDI and TMP.
+// Must preserve all registers.
 void StubCode::GeneratePrintStopMessageStub(Assembler* assembler) {
-  // Preserve caller-saved registers.
-  __ pushq(RAX);
-  __ pushq(RCX);
-  __ pushq(RDX);
-  __ pushq(RSI);
-  __ pushq(R8);
-  __ pushq(R9);
-  __ pushq(R10);
+  __ enter(Immediate(0));
+  __ PreserveCallerSavedRegisters();
 
-  AssemblerMacros::EnterStubFrame(assembler);
+  // Call the runtime leaf function. RDI already contains the parameter.
+  __ ReserveAlignedFrameSpace(kWordSize);

[siva, 2012/07/09 22:51:07]

Nothing is being pushed on the stack as parameters so why not
__ ReserveAlignedFrameSpace(0);

[Ivan Posva, 2012/07/09 22:59:35]

Done.

+  __ CallRuntime(kPrintStopMessageRuntimeEntry);
 
-  // Align frame before entering C++ world.
-  if (OS::ActivationFrameAlignment() > 0) {
-    __ andq(RSP, Immediate(~(OS::ActivationFrameAlignment() - 1)));
-  }
-
-  // Stop message is already in RDI.
-  __ movq(TMP, Immediate(reinterpret_cast<uword>(&PrintStopMessage)));
-  __ call(TMP);
-
-  __ LeaveFrame();
-
-  // Restore caller-saved registers.
-  __ popq(R10);
-  __ popq(R9);
-  __ popq(R8);
-  __ popq(RSI);
-  __ popq(RDX);
-  __ popq(RCX);
-  __ popq(RAX);
-
+  __ RestoreCallerSavedRegisters();
+  __ leave();
   __ ret();
 }
 
 
 // Input parameters:
 //   RSP : points to return address.
 //   RSP + 8 : address of return value.
 //   RAX : address of first argument in argument array.
 //   RAX - 8*R10 + 8 : address of last argument in argument array.
 //   RBX : address of the native function to call.
@@ skipping 914 matching lines @@
   __ CallRuntime(kAllocateContextRuntimeEntry);  // Allocate context.
   __ popq(RAX);  // Pop number of context variables argument.
   __ popq(RAX);  // Pop the new context object.
   // RAX: new object
   // Restore the frame pointer.
   __ LeaveFrame();
   __ ret();
 }
 
 
+DECLARE_LEAF_RUNTIME_ENTRY(void, StoreBufferBlockProcess, Isolate* isolate);
+
 // Helper stub to implement Assembler::StoreIntoObject.
 // Input parameters:
 //   RAX: Address being stored
 void StubCode::GenerateUpdateStoreBufferStub(Assembler* assembler) {
   // Save registers being destroyed.
-  __ pushq(CTX);
-  __ pushq(RBX);
+  __ pushq(RDX);
+  __ pushq(RCX);
 
   // Load the isolate out of the context.
   // RAX: Address being stored
-  __ movq(CTX, FieldAddress(CTX, Context::isolate_offset()));
+  __ movq(RDX, FieldAddress(CTX, Context::isolate_offset()));
 
   // Load top_ out of the StoreBufferBlock and add the address to the pointers_.
   // RAX: Address being stored
-  // CTX: Isolate
+  // RDX: Isolate
   intptr_t store_buffer_offset = Isolate::store_buffer_block_offset();
-  __ movl(RBX,
-          Address(CTX, store_buffer_offset + StoreBufferBlock::top_offset()));
-  __ movq(Address(CTX,
-                  RBX, TIMES_8,
+  __ movl(RCX,
+          Address(RDX, store_buffer_offset + StoreBufferBlock::top_offset()));
+  __ movq(Address(RDX,
+                  RCX, TIMES_8,
                   store_buffer_offset + StoreBufferBlock::pointers_offset()),
           RAX);
 
   // Increment top_ and check for overflow.
-  // RBX: top_
-  // CTX: Isolate
+  // RCX: top_
+  // RDX: Isolate
   Label L;
-  __ incq(RBX);
-  __ movl(Address(CTX, store_buffer_offset + StoreBufferBlock::top_offset()),
-          RBX);
-  __ cmpl(RBX, Immediate(StoreBufferBlock::kSize));
-  __ j(NOT_EQUAL, &L, Assembler::kNearJump);
+  __ incq(RCX);
+  __ movl(Address(RDX, store_buffer_offset + StoreBufferBlock::top_offset()),
+          RCX);
+  __ cmpl(RCX, Immediate(StoreBufferBlock::kSize));
+  // Restore values.
+  __ popq(RCX);
+  __ popq(RDX);
+  __ j(EQUAL, &L, Assembler::kNearJump);
+  __ ret();
 
-  // Handle overflow: Reset the top_ pointer.
-  // CTX: Isolate
-  __ movl(RBX, Immediate(0));
-  __ movl(Address(CTX, store_buffer_offset + StoreBufferBlock::top_offset()),
-          RBX);
-
+  // Handle overflow: Call the runtime leaf function.
   __ Bind(&L);
-  // Restore values.
-  __ popq(RBX);
-  __ popq(CTX);
+  // Setup frame, push callee-saved registers.
+  __ enter(Immediate(0));
+  __ PreserveCallerSavedRegisters();
+  __ ReserveAlignedFrameSpace(0);
+  __ movq(RDI, FieldAddress(CTX, Context::isolate_offset()));
+  __ CallRuntime(kStoreBufferBlockProcessRuntimeEntry);
+  // Restore callee-saved registers, tear down frame.
+  __ RestoreCallerSavedRegisters();
+  __ leave();
   __ ret();
 }
 
 
 // Called for inline allocation of objects.
 // Input parameters:
 //   RSP + 16 : type arguments object (only if class is parameterized).
 //   RSP + 8 : type arguments of instantiator (only if class is parameterized).
 //   RSP : points to return address.
 void StubCode::GenerateAllocationStubForClass(Assembler* assembler,
@@ skipping 791 matching lines @@
 void StubCode::GenerateJumpToErrorHandlerStub(Assembler* assembler) {
   __ movq(RAX, RCX);  // error object.
   __ movq(RBP, RDX);  // target frame_pointer.
   __ movq(RSP, RSI);  // target stack_pointer.
   __ jmp(RDI);  // Jump to the exception handler code.
 }
 
 }  // namespace dart
 
 #endif  // defined TARGET_ARCH_X64