Commit c0fca4e8 authored by yurys@chromium.org's avatar yurys@chromium.org

Revert r14252 as it broke --prof for some cases

R=jkummerow
BUG=v8:2642

Review URL: https://codereview.chromium.org/14367020

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14348 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent 1938b1de
...@@ -1448,7 +1448,13 @@ void Logger::TickEvent(TickSample* sample, bool overflow) { ...@@ -1448,7 +1448,13 @@ void Logger::TickEvent(TickSample* sample, bool overflow) {
msg.Append(','); msg.Append(',');
msg.AppendAddress(sample->sp); msg.AppendAddress(sample->sp);
msg.Append(",%ld", static_cast<int>(OS::Ticks() - epoch_)); msg.Append(",%ld", static_cast<int>(OS::Ticks() - epoch_));
if (sample->has_external_callback) {
msg.Append(",1,");
msg.AppendAddress(sample->external_callback); msg.AppendAddress(sample->external_callback);
} else {
msg.Append(",0,");
msg.AppendAddress(sample->tos);
}
msg.Append(",%d", static_cast<int>(sample->state)); msg.Append(",%d", static_cast<int>(sample->state));
if (overflow) { if (overflow) {
msg.Append(",overflow"); msg.Append(",overflow");
......
...@@ -894,7 +894,7 @@ void ProfileGenerator::RecordTickSample(const TickSample& sample) { ...@@ -894,7 +894,7 @@ void ProfileGenerator::RecordTickSample(const TickSample& sample) {
if (sample.pc != NULL) { if (sample.pc != NULL) {
*entry++ = code_map_.FindEntry(sample.pc); *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 // Don't use PC when in external callback code, as it can point
// inside callback's code, and we will erroneously report // inside callback's code, and we will erroneously report
// that a callback calls itself. // that a callback calls itself.
......
...@@ -636,7 +636,16 @@ DISABLE_ASAN void TickSample::Trace(Isolate* isolate) { ...@@ -636,7 +636,16 @@ DISABLE_ASAN void TickSample::Trace(Isolate* isolate) {
return; 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); SafeStackTraceFrameIterator it(isolate, fp, sp, sp, js_entry_sp);
int i = 0; int i = 0;
......
...@@ -51,16 +51,21 @@ struct TickSample { ...@@ -51,16 +51,21 @@ struct TickSample {
sp(NULL), sp(NULL),
fp(NULL), fp(NULL),
external_callback(NULL), external_callback(NULL),
frames_count(0) {} frames_count(0),
has_external_callback(false) {}
void Trace(Isolate* isolate); void Trace(Isolate* isolate);
StateTag state; // The state of the VM. StateTag state; // The state of the VM.
Address pc; // Instruction pointer. Address pc; // Instruction pointer.
Address sp; // Stack pointer. Address sp; // Stack pointer.
Address fp; // Frame pointer. Address fp; // Frame pointer.
union {
Address tos; // Top stack value (*sp).
Address external_callback; Address external_callback;
};
static const int kMaxFramesCount = 64; static const int kMaxFramesCount = 64;
Address stack[kMaxFramesCount]; // Call stack. Address stack[kMaxFramesCount]; // Call stack.
int frames_count : 8; // Number of captured frames. int frames_count : 8; // Number of captured frames.
bool has_external_callback : 1;
}; };
class Sampler { class Sampler {
......
...@@ -64,6 +64,7 @@ static void EnqueueTickSampleEvent(ProfilerEventsProcessor* proc, ...@@ -64,6 +64,7 @@ static void EnqueueTickSampleEvent(ProfilerEventsProcessor* proc,
i::Address frame3 = NULL) { i::Address frame3 = NULL) {
i::TickSample* sample = proc->TickSampleEvent(); i::TickSample* sample = proc->TickSampleEvent();
sample->pc = frame1; sample->pc = frame1;
sample->tos = frame1;
sample->frames_count = 0; sample->frames_count = 0;
if (frame2 != NULL) { if (frame2 != NULL) {
sample->stack[0] = frame2; sample->stack[0] = frame2;
...@@ -238,6 +239,7 @@ TEST(Issue1398) { ...@@ -238,6 +239,7 @@ TEST(Issue1398) {
i::TickSample* sample = processor.TickSampleEvent(); i::TickSample* sample = processor.TickSampleEvent();
sample->pc = ToAddress(0x1200); sample->pc = ToAddress(0x1200);
sample->tos = 0;
sample->frames_count = i::TickSample::kMaxFramesCount; sample->frames_count = i::TickSample::kMaxFramesCount;
for (int i = 0; i < sample->frames_count; ++i) { for (int i = 0; i < sample->frames_count; ++i) {
sample->stack[i] = ToAddress(0x1200); sample->stack[i] = ToAddress(0x1200);
......
...@@ -286,7 +286,7 @@ TEST(CFromJSStackTrace) { ...@@ -286,7 +286,7 @@ TEST(CFromJSStackTrace) {
// DoTrace(EBP) [native] // DoTrace(EBP) [native]
// TickSample::Trace // TickSample::Trace
CHECK(sample.external_callback); CHECK(sample.has_external_callback);
CHECK_EQ(FUNCTION_ADDR(TraceExtension::Trace), sample.external_callback); CHECK_EQ(FUNCTION_ADDR(TraceExtension::Trace), sample.external_callback);
// Stack tracing will start from the first JS function, i.e. "JSFuncDoTrace" // Stack tracing will start from the first JS function, i.e. "JSFuncDoTrace"
...@@ -336,7 +336,7 @@ TEST(PureJSStackTrace) { ...@@ -336,7 +336,7 @@ TEST(PureJSStackTrace) {
// TickSample::Trace // TickSample::Trace
// //
CHECK(sample.external_callback); CHECK(sample.has_external_callback);
CHECK_EQ(FUNCTION_ADDR(TraceExtension::JSTrace), sample.external_callback); CHECK_EQ(FUNCTION_ADDR(TraceExtension::JSTrace), sample.external_callback);
// Stack sampling will start from the caller of JSFuncDoTrace, i.e. "JSTrace" // Stack sampling will start from the caller of JSFuncDoTrace, i.e. "JSTrace"
......
...@@ -624,11 +624,13 @@ TEST(RecordTickSample) { ...@@ -624,11 +624,13 @@ TEST(RecordTickSample) {
// -> ccc -> aaa - sample3 // -> ccc -> aaa - sample3
TickSample sample1; TickSample sample1;
sample1.pc = ToAddress(0x1600); sample1.pc = ToAddress(0x1600);
sample1.tos = ToAddress(0x1500);
sample1.stack[0] = ToAddress(0x1510); sample1.stack[0] = ToAddress(0x1510);
sample1.frames_count = 1; sample1.frames_count = 1;
generator.RecordTickSample(sample1); generator.RecordTickSample(sample1);
TickSample sample2; TickSample sample2;
sample2.pc = ToAddress(0x1925); sample2.pc = ToAddress(0x1925);
sample2.tos = ToAddress(0x1900);
sample2.stack[0] = ToAddress(0x1780); sample2.stack[0] = ToAddress(0x1780);
sample2.stack[1] = ToAddress(0x10000); // non-existent. sample2.stack[1] = ToAddress(0x10000); // non-existent.
sample2.stack[2] = ToAddress(0x1620); sample2.stack[2] = ToAddress(0x1620);
...@@ -636,6 +638,7 @@ TEST(RecordTickSample) { ...@@ -636,6 +638,7 @@ TEST(RecordTickSample) {
generator.RecordTickSample(sample2); generator.RecordTickSample(sample2);
TickSample sample3; TickSample sample3;
sample3.pc = ToAddress(0x1510); sample3.pc = ToAddress(0x1510);
sample3.tos = ToAddress(0x1500);
sample3.stack[0] = ToAddress(0x1910); sample3.stack[0] = ToAddress(0x1910);
sample3.stack[1] = ToAddress(0x1610); sample3.stack[1] = ToAddress(0x1610);
sample3.frames_count = 2; sample3.frames_count = 2;
......
...@@ -5,7 +5,7 @@ profiler,"begin",1 ...@@ -5,7 +5,7 @@ profiler,"begin",1
code-creation,Stub,0,0x424260,348,"CompareStub_GE" code-creation,Stub,0,0x424260,348,"CompareStub_GE"
code-creation,LazyCompile,0,0x2a8100,18535,"DrawQube 3d-cube.js:188",0xf43abcac, code-creation,LazyCompile,0,0x2a8100,18535,"DrawQube 3d-cube.js:188",0xf43abcac,
code-creation,LazyCompile,0,0x480100,3908,"DrawLine 3d-cube.js:17",0xf43abc50, code-creation,LazyCompile,0,0x480100,3908,"DrawLine 3d-cube.js:17",0xf43abc50,
tick,0x424284,0xbfffeea0,0,0,0,0x480600,0x2aaaa5 tick,0x424284,0xbfffeea0,0,0,0x480600,0,0x2aaaa5
tick,0x42429f,0xbfffed88,0,0,0,0x480600,0x2aacb4 tick,0x42429f,0xbfffed88,0,0,0x480600,0,0x2aacb4
tick,0x48063d,0xbfffec7c,0,0,0,0x2d0f7c,0x2aaec6 tick,0x48063d,0xbfffec7c,0,0,0x2d0f7c,0,0x2aaec6
profiler,"end" profiler,"end"
...@@ -9,17 +9,17 @@ code-creation,LazyCompile,0,0xf541d120,145,"exp native math.js:41" ...@@ -9,17 +9,17 @@ code-creation,LazyCompile,0,0xf541d120,145,"exp native math.js:41"
function-creation,0xf441d280,0xf541d120 function-creation,0xf441d280,0xf541d120
code-creation,LoadIC,0,0xf541d280,117,"j" code-creation,LoadIC,0,0xf541d280,117,"j"
code-creation,LoadIC,0,0xf541d360,63,"i" code-creation,LoadIC,0,0xf541d360,63,"i"
tick,0x80f82d1,0xffdfe880,0,0,0,0xf541ce5c tick,0x80f82d1,0xffdfe880,0,0,0,0,0xf541ce5c
tick,0x80f89a1,0xffdfecf0,0,0,0,0xf541ce5c tick,0x80f89a1,0xffdfecf0,0,0,0,0,0xf541ce5c
tick,0x8123b5c,0xffdff1a0,0,0,0,0xf541d1a1,0xf541ceea tick,0x8123b5c,0xffdff1a0,0,0,0,0,0xf541d1a1,0xf541ceea
tick,0x8123b65,0xffdff1a0,0,0,0,0xf541d1a1,0xf541ceea tick,0x8123b65,0xffdff1a0,0,0,0,0,0xf541d1a1,0xf541ceea
tick,0xf541d2be,0xffdff1e4,0,0,0 tick,0xf541d2be,0xffdff1e4,0,0,0,0
tick,0xf541d320,0xffdff1dc,0,0,0 tick,0xf541d320,0xffdff1dc,0,0,0,0
tick,0xf541d384,0xffdff1d8,0,0,0 tick,0xf541d384,0xffdff1d8,0,0,0,0
tick,0xf7db94da,0xffdff0ec,0,0,0,0xf541d1a1,0xf541ceea tick,0xf7db94da,0xffdff0ec,0,0,0,0,0xf541d1a1,0xf541ceea
tick,0xf7db951c,0xffdff0f0,0,0,0,0xf541d1a1,0xf541ceea tick,0xf7db951c,0xffdff0f0,0,0,0,0,0xf541d1a1,0xf541ceea
tick,0xf7dbc508,0xffdff14c,0,0,0,0xf541d1a1,0xf541ceea tick,0xf7dbc508,0xffdff14c,0,0,0,0,0xf541d1a1,0xf541ceea
tick,0xf7dbff21,0xffdff198,0,0,0,0xf541d1a1,0xf541ceea tick,0xf7dbff21,0xffdff198,0,0,0,0,0xf541d1a1,0xf541ceea
tick,0xf7edec90,0xffdff0ec,0,0,0,0xf541d1a1,0xf541ceea tick,0xf7edec90,0xffdff0ec,0,0,0,0,0xf541d1a1,0xf541ceea
tick,0xffffe402,0xffdff488,0,0,0 tick,0xffffe402,0xffdff488,0,0,0,0
profiler,"end" profiler,"end"
...@@ -170,7 +170,7 @@ function TickProcessor( ...@@ -170,7 +170,7 @@ function TickProcessor(
processor: this.processSnapshotPosition }, processor: this.processSnapshotPosition },
'tick': { 'tick': {
parsers: [parseInt, parseInt, parseInt, parseInt, parsers: [parseInt, parseInt, parseInt, parseInt,
parseInt, 'var-args'], parseInt, parseInt, 'var-args'],
processor: this.processTick }, processor: this.processTick },
'heap-sample-begin': { parsers: [null, null, parseInt], 'heap-sample-begin': { parsers: [null, null, parseInt],
processor: this.processHeapSampleBegin }, processor: this.processHeapSampleBegin },
...@@ -368,7 +368,8 @@ TickProcessor.prototype.includeTick = function(vmState) { ...@@ -368,7 +368,8 @@ TickProcessor.prototype.includeTick = function(vmState) {
TickProcessor.prototype.processTick = function(pc, TickProcessor.prototype.processTick = function(pc,
sp, sp,
ns_since_start, ns_since_start,
external_callback, is_external_callback,
tos_or_external_callback,
vmState, vmState,
stack) { stack) {
this.distortion += this.distortion_per_entry; this.distortion += this.distortion_per_entry;
...@@ -382,15 +383,23 @@ TickProcessor.prototype.processTick = function(pc, ...@@ -382,15 +383,23 @@ TickProcessor.prototype.processTick = function(pc,
this.ticks_.excluded++; this.ticks_.excluded++;
return; return;
} }
if (external_callback) { if (is_external_callback) {
// Don't use PC when in external callback code, as it can point // Don't use PC when in external callback code, as it can point
// inside callback's code, and we will erroneously report // inside callback's code, and we will erroneously report
// that a callback calls itself. Instead we use tos_or_external_callback, // that a callback calls itself. Instead we use tos_or_external_callback,
// as simply resetting PC will produce unaccounted ticks. // 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));
}; };
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment