Attachment 357860 Details for Bug 192943 – Patch

[patch] Patch

bug-192943-20181220133853.patch (text/plain), 5.34 KB, created by Suresh Koppisetty on 2018-12-20 13:38:53 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Suresh Koppisetty

Created: 2018-12-20 13:38:53 PST

Size: 5.34 KB

patch

obsolete

>Subversion Revision: 239457
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index c813612d50edff8f23239c0474b923a720666348..b93df0f8d5d2dea1bca1eb9f6b8e294122e4310e 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,54 @@
>+2018-12-20  Suresh Koppisetty  <skoppisetty@apple.com>
>+
>+        Moving non-critical initializations to a parallel thread can speed up process launch time by 15%.
>+        https://bugs.webkit.org/show_bug.cgi?id=192943
>+        <rdar://problem/46877677>.
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        _RegisterApplication, _LSSetApplicationInformationItem and _accessibilityInitialize are non-critical work
>+        and they can be done in a parallel thread to speed up process launch time. Following times are measured
>+        from artraces captured under different scenarios. See <rdar://problem/46877677>.
>+
>+        System Config:
>+        âââââââââââââ
>+        MacBook Pro (13-inch, Late 2013 model running 18E158)
>+
>+        Clean system:
>+        âââââââââââââ
>+        Mean total process launch time: 166.56 ms
>+        Mean Process Launch time: 58.88 ms
>+        Mean Init New Web Process time: 74.64 ms
>+
>+        Root with a clean trunk:
>+        âââââââââââââ
>+        Safari : 32656d6f
>+        Webkit: r239297
>+
>+        Mean total process launch time: 176.53 ms
>+        Mean Process Launch time: 62.72 ms
>+        Mean Init New Web Process time: 83.6 ms
>+
>+        Root with proposed changes on top of trunk:
>+        âââââââââââââ
>+        Safari : 32656d6f
>+        Webkit: r239297
>+
>+        Mean total process launch time: 150.12 ms
>+        Mean Process Launch time: 64.19 ms
>+        Mean Init New Web Process time: 74.18 ms
>+
>+        Perf Improvement:
>+        âââââââââââââ
>+        176.53 - 150.12 = 26.41 ms (~15%)
>+
>+        * WebProcess/WebProcess.h:
>+        * WebProcess/cocoa/WebProcessCocoa.mm:
>+        (WebKit::WebProcess::platformInitializeWebProcess):
>+        (WebKit::WebProcess::updateProcessName):
>+        (WebKit::WebProcess::platformInitializeProcess):
>+        (WebKit::WebProcess::platformTerminate):
>+
> 2018-12-20  Keith Rollin  <krollin@apple.com>
> 
>         Improve release-level page-load logging
>diff --git a/Source/WebKit/WebProcess/WebProcess.h b/Source/WebKit/WebProcess/WebProcess.h
>index ad7d1d11367858639353a422d0d3f2ff20dc77f3..4e72beea9190d68ffddbfe653ab6b44f71892d92 100644
>--- a/Source/WebKit/WebProcess/WebProcess.h
>+++ b/Source/WebKit/WebProcess/WebProcess.h
>@@ -468,6 +468,7 @@ private:
> #if PLATFORM(MAC)
>     std::unique_ptr<WebCore::CPUMonitor> m_cpuMonitor;
>     Optional<double> m_cpuLimit;
>+    OSObjectPtr<dispatch_queue_t> m_internalQueue;
> 
>     enum class ProcessType { Inspector, ServiceWorker, PrewarmedWebContent, WebContent };
>     ProcessType m_processType { ProcessType::WebContent };
>diff --git a/Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm b/Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm
>index ef32528f6899cffda85207b7ad76a0c2d2fde6f1..7b5f8511b2412c6d926d65527a09e5bcb75fc50b 100644
>--- a/Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm
>+++ b/Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm
>@@ -185,7 +185,10 @@ void WebProcess::platformInitializeWebProcess(WebProcessCreationParameters&& par
> #if PLATFORM(MAC) && ENABLE(WEBPROCESS_NSRUNLOOP)
>     // Need to initialize accessibility for VoiceOver to work when the WebContent process is using NSRunLoop.
>     // Currently, it is also needed to allocate and initialize an NSApplication object.
>-    [NSApplication _accessibilityInitialize];
>+    dispatch_async(m_internalQueue, ^{
>+        // _accessibilityInitialize would have to use the same (serial) queue as _RegisterApplication(), otherwise, _accessibilityInitialize can also end up calling _RegisterApplication.
>+        [NSApplication _accessibilityInitialize];
>+    });
> #endif
> 
> #if TARGET_OS_IPHONE
>@@ -235,8 +238,8 @@ void WebProcess::updateProcessName()
>         break;
>     }
> 
>-    dispatch_async(dispatch_get_global_queue(QOS_CLASS_BACKGROUND, 0), ^{
>-        // Note that it is important for _RegisterApplication() to have been called before setting the display name.
>+    dispatch_async(m_internalQueue, ^{
>+        // _LSSetApplicationInformationItem would have to use the same (serial) queue as _RegisterApplication(), otherwise, the call to _RegisterApplication() may happen *after* we've tried to set the process name and setting the process name would fail.
>         auto error = _LSSetApplicationInformationItem(kLSDefaultSessionID, _LSGetCurrentApplicationASN(), _kLSDisplayNameKey, (CFStringRef)applicationName, nullptr);
>         ASSERT(!error);
>         if (error) {
>@@ -362,9 +365,11 @@ void WebProcess::platformInitializeProcess(const ChildProcessInitializationParam
> 
>     SwitchingGPUClient::setSingleton(WebSwitchingGPUClient::singleton());
> 
>-    // This is necessary so that we are able to set the process' display name.
>-    _RegisterApplication(nullptr, nullptr);
>-
>+    m_internalQueue = dispatch_queue_create([NSString stringWithFormat:@"com.apple.WebKit.WebProcessCocoa"].UTF8String, DISPATCH_QUEUE_SERIAL);
>+    dispatch_async(m_internalQueue, ^{
>+        // This is necessary so that we are able to set the process' display name.
>+        _RegisterApplication(nullptr, nullptr);
>+    });
> #else
> 
>     if (![NSApp isRunning]) {

Actions: View | Formatted Diff | Diff

Attachments on bug 192943: 357845 | 357860 | 357870 | 357902 | 357915