Closed Bug 1860826 Opened 1 year ago Closed 1 year ago

Categories

(Core :: Graphics: WebGPU, defect, P2)

defect

Tracking

()

VERIFIED FIXED
121 Branch
Tracking Status
firefox-esr115 --- disabled
firefox119 --- disabled
firefox120 --- disabled
firefox121 --- verified
firefox122 --- verified

People

(Reporter: mayankleoboy1, Assigned: ErichDonGubler)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression)

Crash Data

Attachments

(1 file)

Attached file about:support

Bug 1831120: style(webgpu): add space b/w GPUBuffer.{unmap,destroy} to match spec. r=webgpu-reviewers,webidl,saschanaz,jimb

Differential Revision: https://phabricator.services.mozilla.com/D177056

Keywords: regression
Regressed by: 1831120
Crash Signature: [@ core::num::nonzero::NonZeroU64::new ] [@ mozilla::webgpu::Queue::WriteBuffer::<T>::operator() ]
Flags: needinfo?(egubler)

Set release status flags based on info from the regressing bug 1831120

Severity: -- → S2
Depends on: webgpu-v1
Flags: needinfo?(egubler)
Priority: -- → P2

:ErichDonGubler, since you are the author of the regressor, bug 1831120, could you take a look?

For more information, please visit BugBot documentation.

Flags: needinfo?(egubler)
See Also: → 1860577

I will investigate this as I find bandwidth to do so over the next couple of weeks. I suspect that the bug marked as the regressor is not actually the regressor (adding readonly attribute and additional spacing in IDL likely has nothing to do with this), but rather something else that has unblocked logic that was stopping this bug from being observed before.

Assignee: nobody → egubler
Flags: needinfo?(egubler)
Blocks: 1859780
No longer depends on: webgpu-v1

FWIW, I dont get the crash on the latest Nightly.

The fix is bug 1856371.
:ErichDonGubler: feel free to close this bug

Depends on: 1856371
Flags: needinfo?(egubler)
Flags: needinfo?(egubler)

Mayank: I'm confused. Is that the same crashing issue? Or does your other statement about closing this bug apply still?

Lets close this bug as the original link doesnt crash anymore.

The crash signatures from the demo in comment #7 has changed now (maybe because of bug 1861751). The new signatures are captured in other bugs i have filed (bug 1863107).

Status: NEW → RESOLVED
Closed: 1 year ago
Depends on: 1861751
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
Flags: qe-verify+

Reproducible on a 2023-10-24 Nightly build on Windows 10.
Verified as fixed on Firefox 121.0b4 and Nightly 122.0a1 on Windows 10, Ubuntu 22, macOS 12.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: