Improved profiler view and inclusive profile tree

runtime/vm/profiler.cc

 // Copyright (c) 2013, 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 "platform/address_sanitizer.h"
 #include "platform/memory_sanitizer.h"
 #include "platform/utils.h"
 
 #include "vm/allocation.h"
 #include "vm/atomic.h"
@@ skipping 515 matching lines @@
   // Attempt to find a better return address.
   ReturnAddressLocator ral(sample, code);
 
   if (!ral.LocateReturnAddress(&return_address)) {
     ASSERT(return_address == sample->pc_marker());
     // Could not find a better return address than the pc_marker.
     if (code.ContainsInstructionAt(return_address)) {
       // PC marker is in the same code as pc, no missing frame.
       return;
     }
-    if (!ContainedInDartCodeHeaps(return_address)) {
-      // PC marker is not from the Dart heap. Do not insert.
-      return;
-    }
   }
 
+  if (!ContainedInDartCodeHeaps(return_address)) {
+    // return address is not from the Dart heap. Do not insert.
+    return;
+  }
+
   if (return_address != 0) {
     sample->InsertCallerForTopFrame(return_address);
   }
 }
 
 
 bool PreprocessVisitor::ContainedInDartCodeHeaps(uword pc) const {
   return isolate()->heap()->CodeContains(pc) ||
          vm_isolate()->heap()->CodeContains(pc);
 }
@@ skipping 374 matching lines @@
     const InterruptedThreadState& state,
     void* data) {
   Isolate* isolate = reinterpret_cast<Isolate*>(data);
   if ((isolate == NULL) || (Dart::vm_isolate() == NULL)) {
     // No isolate.
     return;
   }
 
   ASSERT(isolate != Dart::vm_isolate());
 
+  const bool exited_dart_code = (isolate->stub_code() != NULL) &&
+                                (isolate->top_exit_frame_info() != 0) &&
+                                (isolate->vm_tag() != VMTag::kDartTagId);
+  const bool in_dart_code = (isolate->stub_code() != NULL) &&
+                            (isolate->top_exit_frame_info() == 0) &&
+                            (isolate->vm_tag() == VMTag::kDartTagId);
+
   uintptr_t sp = 0;
-  if ((isolate->stub_code() != NULL) &&
-      (isolate->top_exit_frame_info() == 0) &&
-      (isolate->vm_tag() == VMTag::kDartTagId)) {
+
+  if (in_dart_code) {
     // If we're in Dart code, use the Dart stack pointer.
     sp = state.dsp;
   } else {
     // If we're in runtime code, use the C stack pointer.
     sp = state.csp;
   }
 
   IsolateProfilerData* profiler_data = isolate->profiler_data();
   if (profiler_data == NULL) {
     // Profiler not initialized.
@@ skipping 73 matching lines @@
   ASSERT(counters != NULL);
   counters->Increment(vm_tag);
   sample->set_vm_tag(vm_tag);
   sample->set_user_tag(isolate->user_tag());
   sample->set_sp(sp);
   sample->set_fp(state.fp);
   sample->set_lr(state.lr);
   CopyStackBuffer(sample);
   CopyPCMarkerIfSafe(sample);
 
-  // Walk the call stack.
   if (FLAG_profile_vm) {
     // Always walk the native stack collecting both native and Dart frames.
     ProfilerNativeStackWalker stackWalker(sample,
                                           stack_lower,
                                           stack_upper,
                                           state.pc,
                                           state.fp,
                                           sp);
     stackWalker.walk();
+  } else if (exited_dart_code) {
+    // We have a valid exit frame info, use the Dart stack walker.
+    ProfilerDartExitStackWalker stackWalker(isolate, sample);
+    stackWalker.walk();
+  } else if (in_dart_code) {
+    // We are executing Dart code. We have frame pointers.
+    ProfilerDartStackWalker stackWalker(isolate,
+                                        sample,
+                                        stack_lower,
+                                        stack_upper,
+                                        state.pc,
+                                        state.fp,
+                                        sp);
+    stackWalker.walk();
   } else {
-    // Attempt to walk only the Dart call stack, falling back to walking
-    // the native stack.
-    if ((isolate->stub_code() != NULL) &&
-        (isolate->top_exit_frame_info() != 0) &&
-        (isolate->vm_tag() != VMTag::kDartTagId)) {
-      // We have a valid exit frame info, use the Dart stack walker.
-      ProfilerDartExitStackWalker stackWalker(isolate, sample);
-      stackWalker.walk();
-    } else if ((isolate->stub_code() != NULL) &&
-               (isolate->top_exit_frame_info() == 0) &&
-               (isolate->vm_tag() == VMTag::kDartTagId)) {
-      // We are executing Dart code. We have frame pointers.
-      ProfilerDartStackWalker stackWalker(isolate,
-                                          sample,
-                                          stack_lower,
-                                          stack_upper,
-                                          state.pc,
-                                          state.fp,
-                                          sp);
-      stackWalker.walk();
-    } else {
-#if defined(TARGET_OS_WINDOWS) && defined(TARGET_ARCH_X64)
-      // ProfilerNativeStackWalker is known to cause crashes on Win64.
-      // BUG=20423.
-      sample->set_ignore_sample(true);
-#else
-      // Fall back to an extremely conservative stack walker.
-      ProfilerNativeStackWalker stackWalker(sample,
-                                            stack_lower,
-                                            stack_upper,
-                                            state.pc,
-                                            state.fp,
-                                            sp);
-      stackWalker.walk();
-#endif
-    }
+    sample->set_vm_tag(VMTag::kEmbedderTagId);
+    sample->SetAt(0, state.pc);
   }
 }
 
 }  // namespace dart