-
-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix failed request for adapter when not available #31002
Fix failed request for adapter when not available #31002
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution!
The messaging is actually quite complicated, and it's only the initial RequestAdapter
that can return None
if webgpu is unavailable, but because of the way the ipc sender is passed through to webgpu, all others still need to return an option...
b8ca785
to
6554e8e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, with one nit.
I think we may have to also update tests expectations, I assume some TIMEOUT
will change to FAIL
or PASS
. @sagudev other than pointing to https://github.com/servo/servo/blob/main/tests/wpt/README.md#updating-test-expectations, are there other instructions on how to do it? I forgot how :)
6554e8e
to
2d859ac
Compare
🔨 Triggering try run (#7445239236) with platforms=linux and layout=undefined |
Test results for linux-wpt-layout-2020 from try job (#7445239236): Flaky unexpected result (11)
Stable unexpected results that are known to be intermittent (21)
Stable unexpected results (1)
|
|
Is very flaky lately, but it's unrelated to this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a lot of CRASH [expected OK]
with Failed to get a response for BufferMapAsync (thread Script(1,1), at components/script/dom/gpubuffer.rs:369)
You can reporoduce on local with: |
I think there you need to send the option: servo/components/webgpu/lib.rs Line 445 in 2d859ac
It compiles only because of the casting here servo/components/webgpu/lib.rs Line 426 in 2d859ac
which should be changed to |
Signed-off-by: Bentaimia Haddadi <haddadi.taym@gmail.com>
2d859ac
to
0dc2fff
Compare
🔨 Triggering try run (#7457261358) with platforms=linux and layout=undefined |
new try run for webgpu: https://github.com/sagudev/servo/actions/runs/7457368386 |
Test results for linux-wpt-layout-2020 from try job (#7457261358): Flaky unexpected result (16)
Stable unexpected results that are known to be intermittent (14)
|
✨ Try run (#7457261358) succeeded. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. @sagudev I think we're good on the tests, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Webgpu try run looks good.
./mach build -d
does not report any errors./mach test-tidy
does not report any errors