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

Hardware feature levels in WGPURequestAdapterOptions #293

Closed
kainino0x opened this issue May 21, 2024 · 3 comments
Closed

Hardware feature levels in WGPURequestAdapterOptions #293

kainino0x opened this issue May 21, 2024 · 3 comments

Comments

@kainino0x
Copy link
Collaborator

kainino0x commented May 21, 2024

I have posted a proposal for the JS API here:
gpuweb/gpuweb#4656

I would like to do the same thing in the C API, except (at least as an initial proposal) with one major difference: we require a feature level in WGPURequestAdapterOptions instead of defaulting to "FL1"/"Core".

@kainino0x
Copy link
Collaborator Author

May 9 meeting:

  • Kai’s proposal for C API: same but require minFeatureLevel (it has no default, for now)
  • CF: Feels like “compat” is now the actual baseline…
  • KN: Compat by default?
  • CF: As long as we have features to reach back up to core
  • AE: Branching feature levels? “modern mobile”/”modern desktop”. Should be OK, you can be upgraded along the same path
    • Features that will be in mobile but not desktop? ASTC, tile local storage?
  • CF: Between defaulting to compat and requiring a feature level request, don’t think I care.
  • AE: D3D allows you to request a list of feature levels.
    • CF: Weird D3D11ism, like Win7 doesn’t know what FL11_1 is, you have to try twice.
  • KN: will bring this proposal to the WG and we can talk about it again after it’s had some discussion there

@kainino0x kainino0x added the !discuss Needs discussion (at meeting or online) label May 22, 2024
@kainino0x
Copy link
Collaborator Author

One point someone on our team raised about having webgpu.h default to compat, is that compat does enable a bunch of extra validation some of which is not trivial (for example the validation that one texture is only used with one sampler).

@kainino0x kainino0x added blocked on w3c Blocked on W3C gpuweb working group and removed !discuss Needs discussion (at meeting or online) labels Jun 6, 2024
@kainino0x kainino0x added !discuss Needs discussion (at meeting or online) and removed blocked on w3c Blocked on W3C gpuweb working group labels Nov 11, 2024
@kainino0x kainino0x removed the !discuss Needs discussion (at meeting or online) label Nov 13, 2024
@kainino0x
Copy link
Collaborator Author

Closing in favor of #412 which tracks just the one remaining question here.

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

No branches or pull requests

1 participant