• Jose Dapena Paz's avatar
    [profiler] Emit ETW events only on a trace session. · 0e7d5763
    Jose Dapena Paz authored
    If a tracing session starts after a script load has happened, then it is
    not emitted, so the source information is not visible in ETW. This is
    because we keep track of the loads already emitted, and we try to launch
    traces even when recording is not happening.
    
    To prevent this problem, this CS keeps track of when recording is
    happening. So, when it starts, it will emit all the already generated
    code immediately. This will add some overhead on tracing session start
    if system instrumentation is enabled, but this is better than not
    having instrumentation for previous symbols.
    
    There is still one problem: for each active isolate, it calls for
    replaying the event recording in an asynchronous task. So, for any
    JS task that is already running when tracing start, symbols will not
    be available. This makes this change less useful as, for a batch task
    or any long JS code that is already running when tracing start, we
    will not get symbols yet. Only after the foreground task runner runs
    the task to emit the ETW events we will get the symbols resolved in
    the trace.
    
    An specific approach was required for d8 interactive shell. As, when
    showing prompt, it is not processing the task runner queue, and it
    is only processed when the next script is launched, it first checks
    if it needs to generate ETW symbols information before running the
    script.
    
    Bug: v8:12932
    Change-Id: I8b056c69cee0350f921a01c87beb9f2d51e10583
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3705541Reviewed-by: 's avatarCamillo Bruni <cbruni@chromium.org>
    Reviewed-by: 's avatarToon Verwaest <verwaest@chromium.org>
    Commit-Queue: José Dapena Paz <jdapena@igalia.com>
    Cr-Commit-Position: refs/heads/main@{#81673}
    0e7d5763
Name
Last commit
Last update
.github Loading commit data...
bazel Loading commit data...
build_overrides Loading commit data...
custom_deps Loading commit data...
docs Loading commit data...
gni Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.bazelrc Loading commit data...
.clang-format Loading commit data...
.clang-tidy Loading commit data...
.editorconfig Loading commit data...
.flake8 Loading commit data...
.git-blame-ignore-revs Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.mailmap Loading commit data...
.style.yapf Loading commit data...
.vpython Loading commit data...
.vpython3 Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.bazel Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
COMMON_OWNERS Loading commit data...
DEPS Loading commit data...
DIR_METADATA Loading commit data...
ENG_REVIEW_OWNERS Loading commit data...
INFRA_OWNERS Loading commit data...
INTL_OWNERS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LOONG_OWNERS Loading commit data...
MIPS_OWNERS Loading commit data...
OWNERS Loading commit data...
PPC_OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
RISCV_OWNERS Loading commit data...
S390_OWNERS Loading commit data...
WATCHLISTS Loading commit data...
WORKSPACE Loading commit data...
codereview.settings Loading commit data...