Bug 247963 – [Ventura] Many layout tests exiting due to GPU process "crashes"

• Component: New Bugs
• Status: RESOLVED FIXED

[Ryan Haddad]

We're seeing many tests "crashing" without crashlogs on Ventura, with only the following in stderr

No crash log found for com.apple.WebKit.GPU.Development:82267.

stdout:

stderr:
com.apple.WebKit.GPU.Development terminated (pid 82267) for reason: crash

https://build.webkit.org/results/Apple-Ventura-Release-WK2-Tests/256705@main%20(193)/results.html

[Radar WebKit Bug Importer]

<rdar://problem/102393706>

[Chris Dumez]

@Miguel: I looked at the logs and the issue is that the GPUProcess is getting suspended by RunningBoard while holding locked files.

Note that as far as I know, we don't expect process suspension on macOS at the moment so this is an issue.

Also note that this is with an open source SDK, where you recently disabled our RunningBoard code at runtime.

However, I suspect this issue is with:
Source/WebKit/Scripts/update-info-plist-for-runningboard.sh

Which updates our plist to opt into running board management and this logic likely still applies to the open source SDK builds, even though you completely disabled our RunningBoard logic there.

[Chris Dumez]

(In reply to Chris Dumez from comment #2)
> @Miguel: I looked at the logs and the issue is that the GPUProcess is
> getting suspended by RunningBoard while holding locked files.
> 
> Note that as far as I know, we don't expect process suspension on macOS at
> the moment so this is an issue.
> 
> Also note that this is with an open source SDK, where you recently disabled
> our RunningBoard code at runtime.

*at build time*

[Ryan Haddad]

Created attachment 463546 [details]
test list

I can reproduce this on a (customer) Ventura install with the attached test list

./Tools/Scripts/run-webkit-tests --test-list <path-to-download>/repro-crash --no-retry

[Miguel Salinas]

Pull request: https://github.com/WebKit/WebKit/pull/6537

[EWS]

Committed 256719@main (60d2086f9b70): <https://commits.webkit.org/256719@main>

Reviewed commits have been landed. Closing PR #6537 and removing active labels.

[Ryan Haddad]

I can confirm that this has cleared up the crashes that we were seeing on the bots. Thank you!