Title: [290345] trunk/Source/WebKit
Revision
290345
Author
[email protected]
Date
2022-02-22 20:04:05 -0800 (Tue, 22 Feb 2022)

Log Message

Use non-inline messages for layer volatility-related IPC in RemoteRenderingBackend
https://bugs.webkit.org/show_bug.cgi?id=237061

Reviewed by Wenson Hsieh.

There appears to be a rare issue where StreamClientConnection::sendSync() can hang sometimes,
which makes it impossible to do an entire MotionMark run (rdar://89313626). So avoid this
by making the MarkSurfaceNonVolatile and SwapToValidFrontBuffer use out-of-line encoding.

* GPUProcess/graphics/RemoteRenderingBackend.messages.in:

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (290344 => 290345)


--- trunk/Source/WebKit/ChangeLog	2022-02-23 03:54:11 UTC (rev 290344)
+++ trunk/Source/WebKit/ChangeLog	2022-02-23 04:04:05 UTC (rev 290345)
@@ -1,3 +1,16 @@
+2022-02-22  Simon Fraser  <[email protected]>
+
+        Use non-inline messages for layer volatility-related IPC in RemoteRenderingBackend
+        https://bugs.webkit.org/show_bug.cgi?id=237061
+
+        Reviewed by Wenson Hsieh.
+
+        There appears to be a rare issue where StreamClientConnection::sendSync() can hang sometimes,
+        which makes it impossible to do an entire MotionMark run (rdar://89313626). So avoid this
+        by making the MarkSurfaceNonVolatile and SwapToValidFrontBuffer use out-of-line encoding.
+
+        * GPUProcess/graphics/RemoteRenderingBackend.messages.in:
+
 2022-02-22  Simon Lewis  <[email protected]>
 
         Further restrict received IPC boolean values to 0 or 1

Modified: trunk/Source/WebKit/GPUProcess/graphics/RemoteRenderingBackend.messages.in (290344 => 290345)


--- trunk/Source/WebKit/GPUProcess/graphics/RemoteRenderingBackend.messages.in	2022-02-23 03:54:11 UTC (rev 290344)
+++ trunk/Source/WebKit/GPUProcess/graphics/RemoteRenderingBackend.messages.in	2022-02-23 04:04:05 UTC (rev 290345)
@@ -37,8 +37,8 @@
     DeleteAllFonts()
     ReleaseRemoteResource(WebCore::RenderingResourceIdentifier renderingResourceIdentifier)
 
-    MarkSurfaceNonVolatile(WebCore::RenderingResourceIdentifier renderingResourceIdentifier) -> (std::optional<WebKit::ImageBufferBackendHandle> backendHandle, bool bufferWasEmpty) Synchronous NotStreamEncodableReply
-    SwapToValidFrontBuffer(struct WebKit::BufferIdentifierSet bufferSet) -> (struct WebKit::BufferIdentifierSet swappedBufferSet, std::optional<WebKit::ImageBufferBackendHandle> frontBufferHandle, bool frontBufferWasEmpty) Synchronous NotStreamEncodableReply
+    MarkSurfaceNonVolatile(WebCore::RenderingResourceIdentifier renderingResourceIdentifier) -> (std::optional<WebKit::ImageBufferBackendHandle> backendHandle, bool bufferWasEmpty) Synchronous NotStreamEncodable NotStreamEncodableReply
+    SwapToValidFrontBuffer(struct WebKit::BufferIdentifierSet bufferSet) -> (struct WebKit::BufferIdentifierSet swappedBufferSet, std::optional<WebKit::ImageBufferBackendHandle> frontBufferHandle, bool frontBufferWasEmpty) Synchronous NotStreamEncodable NotStreamEncodableReply
 
     MarkSurfacesVolatile(Vector<WebCore::RenderingResourceIdentifier> renderingResourceIdentifiers) -> (Vector<WebCore::RenderingResourceIdentifier> inUseBufferIdentifiers) Synchronous
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to