Cleanup native, collected, and stub code handling in profiler

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/utils.h"
 
 #include "vm/allocation.h"
 #include "vm/atomic.h"
 #include "vm/code_patcher.h"
 #include "vm/isolate.h"
@@ skipping 311 matching lines @@
   }
 
   void AddCaller(intptr_t index) {
     AddCallEntry(callers_table_, index);
   }
 
   void AddCallee(intptr_t index) {
     AddCallEntry(callees_table_, index);
   }
 
-  void PrintToJSONArray(JSONArray* events, bool full) {
+  void PrintNativeCode(JSONObject* profile_code_obj) {
+    ASSERT(kind() == kNativeCode);
+    JSONObject obj(profile_code_obj, "code");
+    obj.AddProperty("type", "@Code");
+    obj.AddProperty("kind", "Native");
+    obj.AddProperty("name", name());
+    obj.AddProperty("user_name", name());
+    obj.AddPropertyF("start", "%" Px "", start());
+    obj.AddPropertyF("end", "%" Px "", end());
+    obj.AddPropertyF("id", "code/native/%" Px "", start());
+  }
+
+  void PrintCollectedCode(JSONObject* profile_code_obj) {
+    ASSERT(kind() == kCollectedCode);
+    JSONObject obj(profile_code_obj, "code");
+    obj.AddProperty("type", "@Code");
+    obj.AddProperty("kind", "Collected");
+    obj.AddProperty("name", name());
+    obj.AddProperty("user_name", name());
+    obj.AddPropertyF("start", "%" Px "", start());
+    obj.AddPropertyF("end", "%" Px "", end());
+    obj.AddPropertyF("id", "code/collected/%" Px "", start());
+  }
+
+  void PrintToJSONArray(Isolate* isolate, JSONArray* events, bool full) {
     JSONObject obj(events);
     obj.AddProperty("type", "ProfileCode");
     obj.AddProperty("kind", KindToCString(kind()));
     obj.AddPropertyF("inclusive_ticks", "%" Pd "", inclusive_ticks());
     obj.AddPropertyF("exclusive_ticks", "%" Pd "", exclusive_ticks());
     if (kind() == kDartCode) {
       // Look up code in Dart heap.
-      Code& code = Code::Handle(Code::LookupCode(start()));
-      Function& func = Function::Handle();
+      Code& code = Code::Handle();
+      code ^= Code::LookupCode(start());
+      if (code.IsNull()) {
+        // Code is a stub in the Vm isolate.
+        code ^= Code::LookupCodeInVmIsolate(start());
+      }
       ASSERT(!code.IsNull());
-      func ^= code.function();
-      if (func.IsNull()) {
-        if (name() == NULL) {
-          const char* stub_name = StubCode::NameOfStub(start());
-          GenerateAndSetSymbolName(stub_name == NULL ? "Stub" : stub_name);
-        }
-        obj.AddPropertyF("start", "%" Px "", start());
-        obj.AddPropertyF("end", "%" Px "", end());
-        obj.AddProperty("name", name());
-      } else {
-        obj.AddProperty("code", code, !full);
-      }
+      obj.AddProperty("code", code, !full);
     } else if (kind() == kCollectedCode) {
       if (name() == NULL) {
+        // Lazily set generated name.
         GenerateAndSetSymbolName("Collected");
       }
-      obj.AddPropertyF("start", "%" Px "", start());
-      obj.AddPropertyF("end", "%" Px "", end());
-      obj.AddProperty("name", name());
+      PrintCollectedCode(&obj);
     } else {
       ASSERT(kind() == kNativeCode);
       if (name() == NULL) {
+        // Lazily set generated name.
         GenerateAndSetSymbolName("Native");
       }
-      obj.AddPropertyF("start", "%" Px "", start());
-      obj.AddPropertyF("end", "%" Px "", end());
-      obj.AddProperty("name", name());
+      PrintNativeCode(&obj);
     }
     {
       JSONArray ticks(&obj, "ticks");
       for (intptr_t i = 0; i < address_table_->length(); i++) {
         const AddressEntry& entry = (*address_table_)[i];
         ticks.AddValueF("%" Px "", entry.pc);
         ticks.AddValueF("%" Pd "", entry.exclusive_ticks);
         ticks.AddValueF("%" Pd "", entry.inclusive_ticks);
       }
     }
@@ skipping 162 matching lines @@
 
   static bool CompareUpperBound(uword pc, uword start, uword end) {
     return pc >= end;
   }
 
   static bool CompareLowerBound(uword pc, uword start, uword end) {
     return end <= pc;
   }
 
   CodeRegion* CreateCodeRegion(uword pc) {
-    Code& code = Code::Handle(Code::LookupCode(pc));
+    Code& code = Code::Handle();
+    code ^= Code::LookupCode(pc);
+    if (!code.IsNull()) {
+      return new CodeRegion(CodeRegion::kDartCode, code.EntryPoint(),
+                            code.EntryPoint() + code.Size());
+    }
+    code ^= Code::LookupCodeInVmIsolate(pc);
     if (!code.IsNull()) {
       return new CodeRegion(CodeRegion::kDartCode, code.EntryPoint(),
                             code.EntryPoint() + code.Size());
     }
     if (heap_->CodeContains(pc)) {
-      const intptr_t kDartCodeAlignment = 0x10;
+      const intptr_t kDartCodeAlignment = OS::PreferredCodeAlignment();
       const intptr_t kDartCodeAlignmentMask = ~(kDartCodeAlignment - 1);
       return new CodeRegion(CodeRegion::kCollectedCode, pc,
                             (pc & kDartCodeAlignmentMask) + kDartCodeAlignment);
     }
     uintptr_t native_start = 0;
     char* native_name = NativeSymbolResolver::LookupSymbolName(pc,
                                                                &native_start);
     if (native_name == NULL) {
       return new CodeRegion(CodeRegion::kNativeCode, pc, pc + 1);
     }
@@ skipping 214 matching lines @@
       {
         ScopeStopwatch sw("CodeTableStream");
         // Serialize to JSON.
         JSONObject obj(stream);
         obj.AddProperty("type", "Profile");
         obj.AddProperty("samples", samples);
         JSONArray codes(&obj, "codes");
         for (intptr_t i = 0; i < code_region_table.Length(); i++) {
           CodeRegion* region = code_region_table.At(i);
           ASSERT(region != NULL);
-          region->PrintToJSONArray(&codes, full);
+          region->PrintToJSONArray(isolate, &codes, full);
         }
       }
     }
   }
   // Enable profile interrupts.
   BeginExecution(isolate);
 }
 
 
 void Profiler::WriteProfile(Isolate* isolate) {
@@ skipping 207 matching lines @@
     stack_lower = 0;
     stack_upper = 0;
   }
   ProfilerSampleStackWalker stackWalker(sample, stack_lower, stack_upper,
                                         state.pc, state.fp, state.sp);
   stackWalker.walk(isolate->heap());
 }
 
 
 }  // namespace dart