Add optional feature `texture-formats-tier1` #5160

Jiawei-Shao · 2025-04-16T07:45:25Z

Supports below new texture formats with the RENDER_ATTACHMENT,
blendable, multisampling capabilities and the STORAGE_BINDING
capability with read-only and write-only storage texture accesses:
- r16unorm
- r16snorm
- rg16unorm
- rg16snorm
- rgba16unorm
- rgba16snorm
Allows the RENDER_ATTACHMENT, blendable, multisampling and resolve
capabilities on below texture formats:
- r8snorm
- rg8snorm
- rgba8snorm
Enables the extension rg11b10ufloat-renderable.
Allows the read-only and write-only storage texture accesses on below texture formats:
- r8unorm
- r8snorm
- r8uint
- r8sint
- rg8unorm
- rg8snorm
- rg8uint
- rg8sint
- r16uint
- r16sint
- r16float
- rg16uint
- rg16sint
- rg16float
- rgb10a2uint
- rgb10a2unorm
- rg11b10ufloat

This patch adds the optional feature `storage-texture-format-tier1` that: - Allows the `read-write` storage texture access on below formats: - `rgba32float` - `rgba32uint` - `rgba32sint` - `rgba16float` - `rgba16uint` - `rgba16sint` - `rgba8unorm` - `rgba8uint` - `rgba8sint` - Allows the `STORAGE_BINDING` usage with all storage texture accesses on below formats: - `r16float` - `r16uint` - `r16sint` - `r8unorm` - `r8uint` - `r8sint`

github-actions · 2025-04-16T07:55:56Z

Previews, as seen when this build job started (298ad79):
WebGPU _{webgpu.idl | Explainer | Correspondence Reference}
WGSL _{grammar.js | wgsl.lalr.txt}

mwyrzykowski

I think it’s covered by existing restrictions, but writable textures are only allowed in compute and fragment stages (not vertex stage), correct?

kainino0x

Editorially seems fine, but can you clarify where these lists come from? They don't look the same as Teo's original proposal. (Point to a specific comment on that thread, if there is one?)

Also #3838 tracks other tiers too, so I'm unlinking this so it won't close that bug.

Jiawei-Shao · 2025-04-17T02:23:18Z

Editorially seems fine, but can you clarify where these lists come from? They don't look the same as Teo's original proposal. (Point to a specific comment on that thread, if there is one?)

Also #3838 tracks other tiers too, so I'm unlinking this so it won't close that bug.

In #3838,

The rw-storage-texture-tier-1 extension is now actually supported as WebGPU core feature,
This PR supports both rw-storage-texture-tier-2 and rw-storage-texture-tier-3 because it is the same requirement for Metal and D3D12 (Metal: Needs MTLReadWriteTextureTier2,
D3D FL11: Optional), and according to David's comment shaderStorageImageExtendedFormats is required for current WebGPU so all the formats mentioned in this PR can be supported on all Vulkan backends that can run WebPU.

Jiawei-Shao · 2025-04-17T02:23:36Z

I think it’s covered by existing restrictions, but writable textures are only allowed in compute and fragment stages (not vertex stage), correct?

Yes.

Kangz · 2025-04-17T13:35:12Z

and according to #3838 (comment) shaderStorageImageExtendedFormats is required for current WebGPU so all the formats mentioned in this PR can be supported on all Vulkan backends that can run WebPU.

I would like to have another investigation into this because that comment was challenged later by @teoxoy and doesn't match the checks that we have in Dawn at the moment.

Can we have a report from https://github.com/kainino0x/gpuinfo-vulkan-query that describes the diff between the original tier 2 and 3? What's the require

Does the list or formats supported with "tier 1" in this PR match the entirety of shaderStorageImageExtendedFormats or are there additional formats that could be supported.

teoxoy · 2025-04-17T14:20:50Z

I think we still need 2 tiers that would map to the old tier 2 & 3 in #3838. Tier 2 can even be supported by compat.

Can we have a report from https://github.com/kainino0x/gpuinfo-vulkan-query that describes the diff between the original tier 2 and 3? What's the require

shaderStorageImageExtendedFormats loses all Adreno 5XX devices (see folded section of #3837 (comment)).

Does the list or formats supported with "tier 1" in this PR match the entirety of shaderStorageImageExtendedFormats or are there additional formats that could be supported.

IIRC, #3837 covers all formats enabled by shaderStorageImageExtendedFormats.

Jiawei-Shao · 2025-04-18T02:25:19Z

Hi @Kangz,

I've updated Myles' comment in #3838:

Format	Proposed tier in #3838	Metal support	D3D12 support	Vulkan support	OpenGL ES 3.1 Support
rgba32float	2	MTLReadWriteTextureTier2	Supported as a set	All	All
rgba32uint	2	MTLReadWriteTextureTier2	Supported as a set	All	All
rgba32sint	2	MTLReadWriteTextureTier2	Supported as a set	All	All
rgba16float	2	MTLReadWriteTextureTier2	Supported as a set	All	All
rgba16uint	2	MTLReadWriteTextureTier2	Supported as a set	All	All
rgba16sint	2	MTLReadWriteTextureTier2	Supported as a set	All	All
rgba8unorm	2	MTLReadWriteTextureTier2	Supported as a set	All	All
rgba8uint	2	MTLReadWriteTextureTier2	Supported as a set	All	All
rgba8sint	2	MTLReadWriteTextureTier2	Supported as a set	All	All
r16float	3	MTLReadWriteTextureTier2	Supported as a set	shaderStorageImageExtendedFormats	None
r16uint	3	MTLReadWriteTextureTier2	Supported as a set	shaderStorageImageExtendedFormats	None
r16sint	3	MTLReadWriteTextureTier2	Supported as a set	shaderStorageImageExtendedFormats	None
r8unorm	3	MTLReadWriteTextureTier2	Supported as a set	shaderStorageImageExtendedFormats	None
r8uint	3	MTLReadWriteTextureTier2	Supported as a set	shaderStorageImageExtendedFormats	None
r8sint	3	MTLReadWriteTextureTier2	Supported as a set	shaderStorageImageExtendedFormats	None

Note that:

The formats with MTLReadWriteTextureTier2 are listed in the Metal doc.
Supported as a set means "if any one of the format listed is supported, all are supported" According to the D3D12 doc.
According to David's comment shaderStorageImageExtendedFormats is now required to support WebGPU, so actually the supports of read-only or write-only STORAGE_BINDING of these formats should be in WebGPU core feature.
Other formats in shaderStorageImageExtendedFormats cannot support MTLReadWrite in Metal so I don't add them in this PR.

kainino0x

The table doesn't explain the difference between the formats that gain RW support (rgba*) and the ones that gain basic STORAGE support (r*). Could you clarify where that comes from?

According to David's comment shaderStorageImageExtendedFormats is now required to support WebGPU, so actually the supports of read-only or write-only STORAGE_BINDING of these formats should be in WebGPU core feature.

I don't think this comment was correct. From the following comments it sounds like that wasn't intended, and there was a spec bug we fixed in #4741.

spec/index.bs

Jiawei-Shao · 2025-04-22T03:29:59Z

The table doesn't explain the difference between the formats that gain RW support (rgba*) and the ones that gain basic STORAGE support (r*). Could you clarify where that comes from?

According to David's comment shaderStorageImageExtendedFormats is now required to support WebGPU, so actually the supports of read-only or write-only STORAGE_BINDING of these formats should be in WebGPU core feature.

I don't think this comment was correct. From the following comments it sounds like that wasn't intended, and there was a spec bug we fixed in #4741.

Now this PR only includes Tier 2 formats which are less controversial. PTAL.

We can talk about Tier 3 formats in another PR.

Kangz · 2025-04-22T14:17:50Z

Thank you for updating this table, it is very useful! Looking at it, IMHO we could merge tier2/3 into a single optional feature (but that doesn't do the same thing on compat vs. not) (it would drop Adreno 500 support but IMHO that's ok). If we want to add support for more formats in the future we can add more tiers.

mwyrzykowski · 2025-04-22T17:30:01Z

Thank you for updating this table, it is very useful! Looking at it, IMHO we could merge tier2/3 into a single optional feature (but that doesn't do the same thing on compat vs. not) (it would drop Adreno 500 support but IMHO that's ok). If we want to add support for more formats in the future we can add more tiers.

Merging tier2+3 into a single tier2 would be nice, it would reduce fragmentation

spec/index.bs

LGTM but holding until working group approval

kainino0x

Conclusions from the WG meeting:

We should include the tier3 formats after all; sorry I was confused about what they supported.
Compat won't allow them, though, so it will add restrictions to this feature (the same way it adds restrictions to core capabilities). We should add this to the Compat proposal doc in the same PR.
The WG decided it should be called something like "storage-texture-formats-1" or "rw-storage-texture-formats-1" - editors need to settle on something @jimblandy @toji. I think the criteria are: this one is numbered 1, and the feature is named in such a way that any future version n+1 is superset of version n.

Jiawei-Shao · 2025-05-20T02:32:03Z

The table looks good to me, thanks for working on this!

One nit: For the 16bit normalized formats should the boxes be ticked instead of saying "If texture-formats-tier1 is enabled"? since those formats are already gated behind texture-formats-tier1 (in the Required Feature column).

Done @teoxoy

alan-baker

WGSL lgtm

dneto0

WGSL change LGTM

kainino0x

Apologies for the delay!

spec/index.bs

Kangz · 2025-05-27T17:06:23Z

GPU Web WG 2025-05-14 Atlantic-time

CW: Jiawei wrote up tables for texture-formats-tier1 and texture-formats-tier2: Add optional feature texture-formats-tier1 #5160 (comment)
CW: Please review for next meeting. (don't think PR is updated yet, so please review these comments) Any comments so far? (Not yet.)

This PR reflects changes from the following spec PRs: * gpuweb/gpuweb#5193 * gpuweb/gpuweb#5160 Please publish new version after it's merged so that we can add tests in the CTS as in gpuweb/cts#4390

This patch adds the optional feature `texture-formats-tier2` with below new features based on [the previous investigation](#5160 (comment)): - Allows the `read-write` storage texture access on below texture formats: - `r8unorm` - `r8uint` - `r8sint` - `rgba8unorm` - `rgba8uint` - `rgba8sint` - `r16uint` - `r16sint` - `r16float` - `rgba16uint` - `rgba16sint` - `rgba16float` - `rgba32uint` - `rgba32sint` - `rgba32float` - Enabling `texture-formats-tier2` will also enable `texture-formats-tier1`. Fixed: #3838

These were added in the upstream WebGPU API and should be reflected here. - gpuweb/gpuweb#5147 Ordered at the top for consistency even though it changes the numbering of everything else. Also fixed the ordering of TimestampQuery to match since the numbering is changing anyway. (Even though the ordering doesn't really matter because it's nonobservable upstream and there isn't really an ordering.) - gpuweb/gpuweb#5160 - gpuweb/gpuweb#5226 The renumbering is technically a breaking change because it changes the ABI (which shouldn't matter right now) and could affect code that depends on comparing enum values or hard-coded integer versions of them.

Tier 1 PRs for WebGPU spec.: [`gpuweb`gfx-rs#5213](gpuweb/gpuweb#5213) and [`gpuweb`gfx-rs#5160](gpuweb/gpuweb#5160). Tier 2 PR for WebGPU spec.: [`gpuweb`gfx-rs#5226](gpuweb/gpuweb#5226)

…ture r=#webgpu-reviewers! Added to spec. with [gpuweb#5160](gpuweb/gpuweb#5160). Differential Revision: https://phabricator.services.mozilla.com/D260594

Tier 1 PRs for WebGPU spec.: [`gpuweb`gfx-rs#5213](gpuweb/gpuweb#5213) and [`gpuweb`gfx-rs#5160](gpuweb/gpuweb#5160). Tier 2 PR for WebGPU spec.: [`gpuweb`gfx-rs#5226](gpuweb/gpuweb#5226)

Jiawei-Shao requested review from Kangz, alan-baker, kainino0x, kdashg and mwyrzykowski April 16, 2025 07:45

Fix typo

63b92e5

mwyrzykowski approved these changes Apr 16, 2025

View reviewed changes

kainino0x previously requested changes Apr 17, 2025

View reviewed changes

Merge branch 'main' into add-rw-storage-texture-tier1

53f6e9b

Merge branch 'main' into add-rw-storage-texture-tier1

2288345

kainino0x reviewed Apr 22, 2025

View reviewed changes

spec/index.bs Outdated Show resolved Hide resolved

spec/index.bs Show resolved Hide resolved

Remove the formats that requires shaderStorageImageExtendedFormats

31c35fe

Jiawei-Shao changed the title ~~Add optional feature storage-texture-format-tier1~~ Add optional feature storage-texture-format-tier2 Apr 22, 2025

Small fix

3713478

Use "read-write-storage-texture-format-tier2" as the extension name

7c398e7

kainino0x reviewed Apr 22, 2025

View reviewed changes

spec/index.bs Outdated Show resolved Hide resolved

spec/index.bs Outdated Show resolved Hide resolved

Jiawei-Shao added 2 commits April 23, 2025 14:29

Address reviewer's comments

ffffdbc

Merge branch 'main' into add-rw-storage-texture-tier1

886bfa4

kainino0x reviewed Apr 23, 2025

View reviewed changes

Enable the capabilities of 16-bit snorm and norm formats by default

86a2fe6

alan-baker approved these changes May 20, 2025

View reviewed changes

dneto0 approved these changes May 21, 2025

View reviewed changes

kainino0x reviewed May 22, 2025

View reviewed changes

spec/index.bs Outdated Show resolved Hide resolved

spec/index.bs Outdated Show resolved Hide resolved

spec/index.bs Outdated Show resolved Hide resolved

kainino0x added the api resolved Resolved - waiting for a change to the API specification label May 22, 2025

Address reviewers' comments

298ad79

kainino0x approved these changes May 22, 2025

View reviewed changes

kainino0x mentioned this pull request May 22, 2025

Auto-enable rg11b10ufloat-renderable with texture-formats-tier1 #5200

Closed

kainino0x merged commit 20b5f62 into gpuweb:main May 22, 2025
4 checks passed

kainino0x added the needs-cts-issue This change requires tests (or would need tests if accepted), but may not have a CTS issue filed yet label May 22, 2025

This was referenced May 26, 2025

Roll gpuweb gpuweb/types#182

Merged

Add 16-bit normalized texture format feature #4938

Closed

teoxoy mentioned this pull request May 31, 2025

Proposal: formats-tier-1 extension #3837

Closed

greggman mentioned this pull request Jun 14, 2025

Fix work group count assertion greggman/webgpu-debug-helper#4

Merged

kainino0x mentioned this pull request Jun 18, 2025

Proposal: read-write storage textures #3838

Closed

Jiawei-Shao mentioned this pull request Jun 19, 2025

Add optional feature texture-formats-tier2 #5226

Merged

kainino0x mentioned this pull request Jun 26, 2025

[minor breaking] Add new feature names from upstream JS API webgpu-native/webgpu-headers#551

Merged

JMS55 mentioned this pull request Jul 29, 2025

SSAO/SSGI followup tracking issue bevyengine/bevy#8888

Open

13 tasks

This was referenced Aug 27, 2025

copyExternalImageToTexture should allow copying to the new 16unorm formats #5289

Closed

copyExternalImageToTexture/importExternalTexture: Add HDR headroom parameter #5236

Open

juj added a commit to juj/wasm_webgpu that referenced this pull request Sep 12, 2025

IDL: Add optional feature texture-formats-tier1. gpuweb/gpuweb#5160

95a0fc3

Add optional feature texture-formats-tier1 #5160

Add optional feature texture-formats-tier1 #5160

Uh oh!

Conversation

Jiawei-Shao commented Apr 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Apr 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mwyrzykowski left a comment

Choose a reason for hiding this comment

Uh oh!

kainino0x left a comment

Choose a reason for hiding this comment

Uh oh!

Jiawei-Shao commented Apr 17, 2025

Uh oh!

Jiawei-Shao commented Apr 17, 2025

Uh oh!

Kangz commented Apr 17, 2025

Uh oh!

teoxoy commented Apr 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Jiawei-Shao commented Apr 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kainino0x left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Jiawei-Shao commented Apr 22, 2025

Uh oh!

Kangz commented Apr 22, 2025

Uh oh!

mwyrzykowski commented Apr 22, 2025

Uh oh!

Uh oh!

Uh oh!

kainino0x left a comment

Choose a reason for hiding this comment

Uh oh!

Jiawei-Shao commented May 20, 2025

Uh oh!

alan-baker left a comment

Choose a reason for hiding this comment

Uh oh!

dneto0 left a comment

Choose a reason for hiding this comment

Uh oh!

kainino0x left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Kangz commented May 27, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Add optional feature `texture-formats-tier1` #5160

Add optional feature `texture-formats-tier1` #5160

Jiawei-Shao commented Apr 16, 2025 •

edited

Loading

github-actions bot commented Apr 16, 2025 •

edited

Loading

teoxoy commented Apr 17, 2025 •

edited

Loading

Jiawei-Shao commented Apr 18, 2025 •

edited

Loading