Skip to content
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

do not query work group params on gles < 3.1 #2166

Merged
merged 2 commits into from
Nov 10, 2021

Conversation

mrk-its
Copy link
Contributor

@mrk-its mrk-its commented Nov 8, 2021

Description

gles backend queries MAX_COMPUTE_WORK_GROUP_COUNT and MAX_COMPUTE_WORK_GROUP_SIZE parameters generating getIndexedParameter: Bad pname: Invalid enum value warnings. This PR disables these queries on wasm32 and sets max_compute_workgroup_size_* and max_compute_workgroups_per_dimension to
0

Testing
It can be tested on wgpu cube example

wgpu-hal/src/gles/adapter.rs Outdated Show resolved Hide resolved
@mrk-its mrk-its changed the title gles: do not query compute shader params on wasm32 gles: do not query work group params on wasm32 Nov 9, 2021
@mrk-its mrk-its changed the title gles: do not query work group params on wasm32 gles: do not query work group params on gles < 3.1 Nov 9, 2021
@mrk-its mrk-its changed the title gles: do not query work group params on gles < 3.1 do not query work group params on gles < 3.1 Nov 9, 2021
Copy link
Member

@kvark kvark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@kvark kvark added the PR: needs back-porting PR with a fix that needs to land on crates label Nov 10, 2021
@kvark kvark merged commit cf2fb0b into gfx-rs:master Nov 10, 2021
@mrk-its mrk-its deleted the gles_wasm32_invalid_enum branch November 10, 2021 20:35
kvark pushed a commit to kvark/wgpu that referenced this pull request Dec 2, 2021
* do not query work group params on gles < 3.1

* address reviewer comment
@kvark
Copy link
Member

kvark commented Dec 2, 2021

Published in wgpu-hal-0.11.5

@kvark kvark removed the PR: needs back-porting PR with a fix that needs to land on crates label Dec 2, 2021
kvark pushed a commit that referenced this pull request Dec 2, 2021
* do not query work group params on gles < 3.1

* address reviewer comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants