Revision: 14348
Author: [email protected]
Date: Fri Apr 19 04:55:01 2013
Log: Revert r14252 as it broke --prof for some cases
R=jkummerow
BUG=v8:2642
Review URL: https://codereview.chromium.org/14367020
http://code.google.com/p/v8/source/detail?r=14348
Modified:
/branches/bleeding_edge/src/log.cc
/branches/bleeding_edge/src/profile-generator.cc
/branches/bleeding_edge/src/sampler.cc
/branches/bleeding_edge/src/sampler.h
/branches/bleeding_edge/test/cctest/test-cpu-profiler.cc
/branches/bleeding_edge/test/cctest/test-log-stack-tracer.cc
/branches/bleeding_edge/test/cctest/test-profile-generator.cc
/branches/bleeding_edge/test/mjsunit/tools/tickprocessor-test-func-info.log
/branches/bleeding_edge/test/mjsunit/tools/tickprocessor-test.log
/branches/bleeding_edge/tools/tickprocessor.js
=======================================
--- /branches/bleeding_edge/src/log.cc Thu Apr 18 02:50:46 2013
+++ /branches/bleeding_edge/src/log.cc Fri Apr 19 04:55:01 2013
@@ -1448,7 +1448,13 @@
msg.Append(',');
msg.AppendAddress(sample->sp);
msg.Append(",%ld", static_cast<int>(OS::Ticks() - epoch_));
- msg.AppendAddress(sample->external_callback);
+ if (sample->has_external_callback) {
+ msg.Append(",1,");
+ msg.AppendAddress(sample->external_callback);
+ } else {
+ msg.Append(",0,");
+ msg.AppendAddress(sample->tos);
+ }
msg.Append(",%d", static_cast<int>(sample->state));
if (overflow) {
msg.Append(",overflow");
=======================================
--- /branches/bleeding_edge/src/profile-generator.cc Fri Apr 12 04:20:22
2013
+++ /branches/bleeding_edge/src/profile-generator.cc Fri Apr 19 04:55:01
2013
@@ -894,7 +894,7 @@
if (sample.pc != NULL) {
*entry++ = code_map_.FindEntry(sample.pc);
- if (sample.external_callback) {
+ if (sample.has_external_callback) {
// Don't use PC when in external callback code, as it can point
// inside callback's code, and we will erroneously report
// that a callback calls itself.
=======================================
--- /branches/bleeding_edge/src/sampler.cc Wed Apr 17 00:53:12 2013
+++ /branches/bleeding_edge/src/sampler.cc Fri Apr 19 04:55:01 2013
@@ -636,7 +636,16 @@
return;
}
- external_callback = isolate->external_callback();
+ const Address callback = isolate->external_callback();
+ if (callback != NULL) {
+ external_callback = callback;
+ has_external_callback = true;
+ } else {
+ // Sample potential return address value for frameless invocation of
+ // stubs (we'll figure out later, if this value makes sense).
+ tos = Memory::Address_at(sp);
+ has_external_callback = false;
+ }
SafeStackTraceFrameIterator it(isolate, fp, sp, sp, js_entry_sp);
int i = 0;
=======================================
--- /branches/bleeding_edge/src/sampler.h Wed Apr 17 00:53:12 2013
+++ /branches/bleeding_edge/src/sampler.h Fri Apr 19 04:55:01 2013
@@ -51,16 +51,21 @@
sp(NULL),
fp(NULL),
external_callback(NULL),
- frames_count(0) {}
+ frames_count(0),
+ has_external_callback(false) {}
void Trace(Isolate* isolate);
StateTag state; // The state of the VM.
Address pc; // Instruction pointer.
Address sp; // Stack pointer.
Address fp; // Frame pointer.
- Address external_callback;
+ union {
+ Address tos; // Top stack value (*sp).
+ Address external_callback;
+ };
static const int kMaxFramesCount = 64;
Address stack[kMaxFramesCount]; // Call stack.
int frames_count : 8; // Number of captured frames.
+ bool has_external_callback : 1;
};
class Sampler {
=======================================
--- /branches/bleeding_edge/test/cctest/test-cpu-profiler.cc Mon Apr 15
07:45:38 2013
+++ /branches/bleeding_edge/test/cctest/test-cpu-profiler.cc Fri Apr 19
04:55:01 2013
@@ -64,6 +64,7 @@
i::Address frame3 = NULL) {
i::TickSample* sample = proc->TickSampleEvent();
sample->pc = frame1;
+ sample->tos = frame1;
sample->frames_count = 0;
if (frame2 != NULL) {
sample->stack[0] = frame2;
@@ -238,6 +239,7 @@
i::TickSample* sample = processor.TickSampleEvent();
sample->pc = ToAddress(0x1200);
+ sample->tos = 0;
sample->frames_count = i::TickSample::kMaxFramesCount;
for (int i = 0; i < sample->frames_count; ++i) {
sample->stack[i] = ToAddress(0x1200);
=======================================
--- /branches/bleeding_edge/test/cctest/test-log-stack-tracer.cc Wed Apr 17
00:53:12 2013
+++ /branches/bleeding_edge/test/cctest/test-log-stack-tracer.cc Fri Apr 19
04:55:01 2013
@@ -286,7 +286,7 @@
// DoTrace(EBP) [native]
// TickSample::Trace
- CHECK(sample.external_callback);
+ CHECK(sample.has_external_callback);
CHECK_EQ(FUNCTION_ADDR(TraceExtension::Trace), sample.external_callback);
// Stack tracing will start from the first JS function,
i.e. "JSFuncDoTrace"
@@ -336,7 +336,7 @@
// TickSample::Trace
//
- CHECK(sample.external_callback);
+ CHECK(sample.has_external_callback);
CHECK_EQ(FUNCTION_ADDR(TraceExtension::JSTrace),
sample.external_callback);
// Stack sampling will start from the caller of JSFuncDoTrace,
i.e. "JSTrace"
=======================================
--- /branches/bleeding_edge/test/cctest/test-profile-generator.cc Fri Apr
12 04:20:22 2013
+++ /branches/bleeding_edge/test/cctest/test-profile-generator.cc Fri Apr
19 04:55:01 2013
@@ -624,11 +624,13 @@
// -> ccc -> aaa - sample3
TickSample sample1;
sample1.pc = ToAddress(0x1600);
+ sample1.tos = ToAddress(0x1500);
sample1.stack[0] = ToAddress(0x1510);
sample1.frames_count = 1;
generator.RecordTickSample(sample1);
TickSample sample2;
sample2.pc = ToAddress(0x1925);
+ sample2.tos = ToAddress(0x1900);
sample2.stack[0] = ToAddress(0x1780);
sample2.stack[1] = ToAddress(0x10000); // non-existent.
sample2.stack[2] = ToAddress(0x1620);
@@ -636,6 +638,7 @@
generator.RecordTickSample(sample2);
TickSample sample3;
sample3.pc = ToAddress(0x1510);
+ sample3.tos = ToAddress(0x1500);
sample3.stack[0] = ToAddress(0x1910);
sample3.stack[1] = ToAddress(0x1610);
sample3.frames_count = 2;
=======================================
---
/branches/bleeding_edge/test/mjsunit/tools/tickprocessor-test-func-info.log
Fri Apr 12 04:20:22 2013
+++
/branches/bleeding_edge/test/mjsunit/tools/tickprocessor-test-func-info.log
Fri Apr 19 04:55:01 2013
@@ -5,7 +5,7 @@
code-creation,Stub,0,0x424260,348,"CompareStub_GE"
code-creation,LazyCompile,0,0x2a8100,18535,"DrawQube
3d-cube.js:188",0xf43abcac,
code-creation,LazyCompile,0,0x480100,3908,"DrawLine
3d-cube.js:17",0xf43abc50,
-tick,0x424284,0xbfffeea0,0,0,0,0x480600,0x2aaaa5
-tick,0x42429f,0xbfffed88,0,0,0,0x480600,0x2aacb4
-tick,0x48063d,0xbfffec7c,0,0,0,0x2d0f7c,0x2aaec6
+tick,0x424284,0xbfffeea0,0,0,0x480600,0,0x2aaaa5
+tick,0x42429f,0xbfffed88,0,0,0x480600,0,0x2aacb4
+tick,0x48063d,0xbfffec7c,0,0,0x2d0f7c,0,0x2aaec6
profiler,"end"
=======================================
--- /branches/bleeding_edge/test/mjsunit/tools/tickprocessor-test.log Fri
Apr 12 04:20:22 2013
+++ /branches/bleeding_edge/test/mjsunit/tools/tickprocessor-test.log Fri
Apr 19 04:55:01 2013
@@ -9,17 +9,17 @@
function-creation,0xf441d280,0xf541d120
code-creation,LoadIC,0,0xf541d280,117,"j"
code-creation,LoadIC,0,0xf541d360,63,"i"
-tick,0x80f82d1,0xffdfe880,0,0,0,0xf541ce5c
-tick,0x80f89a1,0xffdfecf0,0,0,0,0xf541ce5c
-tick,0x8123b5c,0xffdff1a0,0,0,0,0xf541d1a1,0xf541ceea
-tick,0x8123b65,0xffdff1a0,0,0,0,0xf541d1a1,0xf541ceea
-tick,0xf541d2be,0xffdff1e4,0,0,0
-tick,0xf541d320,0xffdff1dc,0,0,0
-tick,0xf541d384,0xffdff1d8,0,0,0
-tick,0xf7db94da,0xffdff0ec,0,0,0,0xf541d1a1,0xf541ceea
-tick,0xf7db951c,0xffdff0f0,0,0,0,0xf541d1a1,0xf541ceea
-tick,0xf7dbc508,0xffdff14c,0,0,0,0xf541d1a1,0xf541ceea
-tick,0xf7dbff21,0xffdff198,0,0,0,0xf541d1a1,0xf541ceea
-tick,0xf7edec90,0xffdff0ec,0,0,0,0xf541d1a1,0xf541ceea
-tick,0xffffe402,0xffdff488,0,0,0
+tick,0x80f82d1,0xffdfe880,0,0,0,0,0xf541ce5c
+tick,0x80f89a1,0xffdfecf0,0,0,0,0,0xf541ce5c
+tick,0x8123b5c,0xffdff1a0,0,0,0,0,0xf541d1a1,0xf541ceea
+tick,0x8123b65,0xffdff1a0,0,0,0,0,0xf541d1a1,0xf541ceea
+tick,0xf541d2be,0xffdff1e4,0,0,0,0
+tick,0xf541d320,0xffdff1dc,0,0,0,0
+tick,0xf541d384,0xffdff1d8,0,0,0,0
+tick,0xf7db94da,0xffdff0ec,0,0,0,0,0xf541d1a1,0xf541ceea
+tick,0xf7db951c,0xffdff0f0,0,0,0,0,0xf541d1a1,0xf541ceea
+tick,0xf7dbc508,0xffdff14c,0,0,0,0,0xf541d1a1,0xf541ceea
+tick,0xf7dbff21,0xffdff198,0,0,0,0,0xf541d1a1,0xf541ceea
+tick,0xf7edec90,0xffdff0ec,0,0,0,0,0xf541d1a1,0xf541ceea
+tick,0xffffe402,0xffdff488,0,0,0,0
profiler,"end"
=======================================
--- /branches/bleeding_edge/tools/tickprocessor.js Fri Apr 12 04:20:22 2013
+++ /branches/bleeding_edge/tools/tickprocessor.js Fri Apr 19 04:55:01 2013
@@ -170,7 +170,7 @@
processor: this.processSnapshotPosition },
'tick': {
parsers: [parseInt, parseInt, parseInt, parseInt,
- parseInt, 'var-args'],
+ parseInt, parseInt, 'var-args'],
processor: this.processTick },
'heap-sample-begin': { parsers: [null, null, parseInt],
processor: this.processHeapSampleBegin },
@@ -368,7 +368,8 @@
TickProcessor.prototype.processTick = function(pc,
sp,
ns_since_start,
- external_callback,
+ is_external_callback,
+ tos_or_external_callback,
vmState,
stack) {
this.distortion += this.distortion_per_entry;
@@ -382,15 +383,23 @@
this.ticks_.excluded++;
return;
}
- if (external_callback) {
+ if (is_external_callback) {
// Don't use PC when in external callback code, as it can point
// inside callback's code, and we will erroneously report
// that a callback calls itself. Instead we use
tos_or_external_callback,
// as simply resetting PC will produce unaccounted ticks.
- pc = 0;
- }
+ pc = tos_or_external_callback;
+ tos_or_external_callback = 0;
+ } else if (tos_or_external_callback) {
+ // Find out, if top of stack was pointing inside a JS function
+ // meaning that we have encountered a frameless invocation.
+ var funcEntry = this.profile_.findEntry(tos_or_external_callback);
+ if (!funcEntry || !funcEntry.isJSFunction |
| !funcEntry.isJSFunction()) {
+ tos_or_external_callback = 0;
+ }
+ }
- this.profile_.recordTick(this.processStack(pc, external_callback,
stack));
+ this.profile_.recordTick(this.processStack(pc, tos_or_external_callback,
stack));
};
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
