Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(838)

Unified Diff: runtime/vm/isolate.cc

Issue 2609253002: Added isolate + thread high watermark tracking to Observatory (Closed)
Patch Set: Merge branch 'master' of github.com:dart-lang/sdk into resubmit Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: runtime/vm/isolate.cc
diff --git a/runtime/vm/isolate.cc b/runtime/vm/isolate.cc
index 622f9c117bced5e4be56cc91414330a55135cfc0..32009f68698c8db654b4e525fe4cd1c16cfab078 100644
--- a/runtime/vm/isolate.cc
+++ b/runtime/vm/isolate.cc
@@ -776,6 +776,7 @@ Isolate::Isolate(const Dart_IsolateFlags& api_flags)
single_step_(false),
thread_registry_(new ThreadRegistry()),
safepoint_handler_(new SafepointHandler(this)),
+ memory_high_watermark_(0),
message_notify_callback_(NULL),
name_(NULL),
debugger_name_(NULL),
@@ -2108,7 +2109,8 @@ void Isolate::PrintJSON(JSONStream* stream, bool ref) {
}
}
- jsobj.AddProperty("threads", thread_registry_);
+ jsobj.AddPropertyF("_memoryHighWatermark", "%u", memory_high_watermark_);
+ jsobj.AddProperty("_threads", thread_registry_);
}
#endif
@@ -2658,6 +2660,8 @@ Thread* Isolate::ScheduleThread(bool is_mutator, bool bypass_safepoint) {
thread = thread_registry()->GetFreeThreadLocked(this, is_mutator);
ASSERT(thread != NULL);
+ thread->ResetHighWatermark();
+
// Set up other values and set the TLS value.
thread->isolate_ = this;
ASSERT(heap() != NULL);
@@ -2704,6 +2708,7 @@ void Isolate::UnscheduleThread(Thread* thread,
// Ensure that the thread reports itself as being at a safepoint.
thread->EnterSafepoint();
}
+ UpdateMemoryHighWatermark();
OSThread* os_thread = thread->os_thread();
ASSERT(os_thread != NULL);
os_thread->DisableThreadInterrupts();
@@ -2724,6 +2729,15 @@ void Isolate::UnscheduleThread(Thread* thread,
}
+void Isolate::UpdateMemoryHighWatermark() {
+ const intptr_t thread_watermarks_total =
+ thread_registry()->ThreadHighWatermarksTotalLocked();
+ if (thread_watermarks_total > memory_high_watermark_) {
+ memory_high_watermark_ = thread_watermarks_total;
+ }
+}
+
+
static RawInstance* DeserializeObject(Thread* thread,
uint8_t* obj_data,
intptr_t obj_len) {
« no previous file with comments | « runtime/vm/isolate.h ('k') | runtime/vm/thread.h » ('j') | runtime/vm/thread_registry.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698