Attachment 370142 Details for Bug 197998 – Patch

[patch] Patch

bug-197998-20190517223940.patch (text/plain), 4.48 KB, created by Tadeu Zagallo on 2019-05-17 13:39:42 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Tadeu Zagallo

Created: 2019-05-17 13:39:42 PDT

Size: 4.48 KB

patch

obsolete

>Subversion Revision: 245317
>diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog
>index 312863185dc8be5ea8d89570fd5919333e6c814d..2b1c4e5399d405aae180eb9c43241b527fb6c642 100644
>--- a/Source/JavaScriptCore/ChangeLog
>+++ b/Source/JavaScriptCore/ChangeLog
>@@ -1,3 +1,16 @@
>+2019-05-17  Tadeu Zagallo  <tzagallo@apple.com>
>+
>+        Add extra information to dumpJITMemory
>+        https://bugs.webkit.org/show_bug.cgi?id=197998
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add ktrace events around the memory dump and mach_absolute_time to link the
>+        events with the entries in the dump.
>+
>+        * jit/ExecutableAllocator.cpp:
>+        (JSC::dumpJITMemory):
>+
> 2019-05-14  Keith Miller  <keith_miller@apple.com>
> 
>         Fix issue with byteOffset on ARM64E
>diff --git a/Source/WTF/ChangeLog b/Source/WTF/ChangeLog
>index 6bc60aa7e4bf74aa275461c4ac185caa4d982601..287422512745a6dc1088c4ff6bdc55f6eb21c74c 100644
>--- a/Source/WTF/ChangeLog
>+++ b/Source/WTF/ChangeLog
>@@ -1,3 +1,14 @@
>+2019-05-17  Tadeu Zagallo  <tzagallo@apple.com>
>+
>+        Add extra information to dumpJITMemory
>+        https://bugs.webkit.org/show_bug.cgi?id=197998
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add a new trace point code for JSC::dumpJITMemory
>+
>+        * wtf/SystemTracing.h:
>+
> 2019-05-14  Commit Queue  <commit-queue@webkit.org>
> 
>         Unreviewed, rolling out r245281.
>diff --git a/Source/JavaScriptCore/jit/ExecutableAllocator.cpp b/Source/JavaScriptCore/jit/ExecutableAllocator.cpp
>index 7230cb7eef8c9f53cbb1a91248cbcc8ce06706eb..79a17808b386f59c90c6262dd507ca21d609e7f7 100644
>--- a/Source/JavaScriptCore/jit/ExecutableAllocator.cpp
>+++ b/Source/JavaScriptCore/jit/ExecutableAllocator.cpp
>@@ -33,8 +33,10 @@
> #include "JSCInlines.h"
> #include <wtf/MetaAllocator.h>
> #include <wtf/PageReservation.h>
>+#include <wtf/SystemTracing.h>
> 
> #if OS(DARWIN)
>+#include <mach/mach_time.h>
> #include <sys/mman.h>
> #endif
> 
>@@ -587,9 +589,12 @@ void dumpJITMemory(const void* dst, const void* src, size_t size)
>     };
> 
>     LockHolder locker(dumpJITMemoryLock);
>+    uint64_t time = mach_absolute_time();
>     uint64_t dst64 = bitwise_cast<uintptr_t>(dst);
>-    write(&dst64, sizeof(dst64));
>     uint64_t size64 = size;
>+    TraceScope(DumpJITMemoryStart, DumpJITMemoryStop, time, dst64, size64);
>+    write(&time, sizeof(time));
>+    write(&dst64, sizeof(dst64));
>     write(&size64, sizeof(size64));
>     write(src, size);
> #else
>diff --git a/Source/WTF/wtf/SystemTracing.h b/Source/WTF/wtf/SystemTracing.h
>index bcce0f9c77d157104d84f63e42cd50cbb2d7836c..75fdea986b8e92bd9821cab03d8f0fce56feb828 100644
>--- a/Source/WTF/wtf/SystemTracing.h
>+++ b/Source/WTF/wtf/SystemTracing.h
>@@ -47,6 +47,8 @@ enum TracePointCode {
>     WebAssemblyCompileEnd,
>     WebAssemblyExecuteStart,
>     WebAssemblyExecuteEnd,
>+    DumpJITMemoryStart,
>+    DumpJITMemoryStop,
> 
>     WebCoreRange = 5000,
>     MainResourceLoadDidStartProvisional,
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 40a60b502837efb3b0b6ea60ebf362d4c5cf61d9..0482bff9ed04be7c5b7cbfa76926387b82d8d68e 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,14 @@
>+2019-05-17  Tadeu Zagallo  <tzagallo@apple.com>
>+
>+        Add extra information to dumpJITMemory
>+        https://bugs.webkit.org/show_bug.cgi?id=197998
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add description for the new dumpJITMemory trace point code.
>+
>+        * Tracing/SystemTracePoints.plist:
>+
> 2019-05-14  Andy Estes  <aestes@apple.com>
> 
>         [Apple Pay] Payment APIs should be completely disabled in web views into which clients have injected user scripts
>diff --git a/Tools/Tracing/SystemTracePoints.plist b/Tools/Tracing/SystemTracePoints.plist
>index c140e91dd0f82d8be1e8e3e6af1da55fcd156304..ad281569215a0d8ad1892d223ae00e046788f5ea 100644
>--- a/Tools/Tracing/SystemTracePoints.plist
>+++ b/Tools/Tracing/SystemTracePoints.plist
>@@ -43,6 +43,18 @@
>                  <key>CodeEnd</key>
>                  <string>2506</string>
>              </dict>
>+             <dict>
>+                 <key>Name</key>
>+                 <string>Dump JIT Memory</string>
>+                 <key>Type</key>
>+                 <string>Interval</string>
>+                 <key>Component</key>
>+                 <string>47</string>
>+                 <key>CodeBegin</key>
>+                 <string>2507</string>
>+                 <key>CodeEnd</key>
>+                 <string>2508</string>
>+             </dict>
>              <dict>
>                  <key>Name</key>
>                  <string>Main Resource Load</string>

Actions: View | Formatted Diff | Diff

Attachments on bug 197998: 370142 | 370151 | 370188