Closed Bug 1682367 Opened 4 years ago Closed 4 years ago

Include more information in the panic for [@ swgl::swgl_fns::{{impl}}::shader_source]

Categories

(Core :: Graphics: WebRender, task, P3)

Desktop
Windows 10
task

Tracking

()

RESOLVED FIXED
86 Branch
Tracking Status
firefox86 --- fixed

People

(Reporter: aosmond, Assigned: aosmond)

References

Details

Attachments

(2 files)

We don't really have an idea of where we went wrong in bug 1680452. Is the shader source empty? Or missing the header?

For bug 1609191 we wanted to annotate crash reports with which shader was being compiled during the crash. Not sure if the solution to this could help with that too

Yep looking at ways to do that. I also want to make sure they aren't doing something like setting WR_RESOURCE_PATH and overriding the default shaders.

This patch adds infrastructure for crash reporter annotations to
WebRender. This is used to expose the new annotation,
GraphicsCompileShader, to indicate which shader we are in the process of
compiling. We often see crash reports when compiling shaders, and it
would be useful to know which one it is compiling.

This also adds another annotation, IsWebRenderResourcePathOverridden,
which is useful to know if someone overrode the shader resource path for
testing purposes. We can likely ignore any crash reports that have this
annotation set.

Attached file Data review, v1 (deleted) —
Attachment #9193285 - Flags: data-review?(chutten)

Comment on attachment 9193285 [details]
Data review, v1

DATA COLLECTION REVIEW RESPONSE:

Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate?

Yes.

Is there a control mechanism that allows the user to turn the data collection on and off?

Yes. This collection is Telemetry so can be controlled through Firefox's Preferences.

If the request is for permanent data collection, is there someone who will monitor the data over time?

Yes, :aosmond is responsible.

Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 1, Technical.

Is the data collection request for default-on or default-off?

Default on for all channels.

Does the instrumentation include the addition of any new identifiers?

No.

Is the data collection covered by the existing Firefox privacy notice?

Yes.

Does there need to be a check-in in the future to determine whether to renew the data?

No. This collection is permanent.


Result: datareview+

Attachment #9193285 - Flags: data-review?(chutten) → data-review+
Pushed by aosmond@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/876e687abc99 Add crash report annotation for which WebRender shader is being compiled. r=jrmuizel,chutten
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 86 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: