Closed Bug 1970429 Opened 1 year ago Closed 1 year ago

Categories

(Toolkit :: Crash Reporting, defect)

defect

Tracking

()

RESOLVED FIXED

People

(Reporter: mayankleoboy1, Unassigned)

References

()

Details

Crash Data

Attachments

(1 file)

Crash Signature: [@ <unknown in xul.pdb> ]

Rough STR is in bug 1970438

See Also: → 1970438
Attached file Better stack trace —

This is a bug in the stackwalker. The crash is happening within JIT'd code but we don't unwind properly from it. We have the fix available upstream so it's just a matter of cutting a new release and updating the version used in Socorro. In the meantime this is the proper stack trace for the crash. Note that the crash address contains the 0x4b pattern which is likely coming from the JS_SWEPT_TENURED_PATTERN. Nicolas, given that this seems reproducible can you have a look?

Flags: needinfo?(nicolas.b.pierron)

Bug 1970164 is probably about the same thing.

See Also: → 1970164

(In reply to Mayank Bansal from comment #3)

Bug 1970164 is probably about the same thing.

Bug 1970164 only landed in Firefox 140, and many of our crashes are coming from Firefox 139:

  • 139.0.1 186 73.81 %
  • 139.0 30 11.90 %
  • 139.0b10 1 0.40 %

(In reply to Gabriele Svelto [:gsvelto] from comment #2)

Nicolas, given that this seems reproducible can you have a look?

Thanks for the pointer, I will follow-up in Bug 1970164.

Flags: needinfo?(nicolas.b.pierron)

This now has symbols, I think.

https://crash-stats.mozilla.org/report/index/1e235022-d3be-419b-b8d5-9fca50250610 is what i get today. Looks like soccoro was updated?

gsvelto, should we close this bug as FIXED, as the crash signature has improved?

Flags: needinfo?(gsvelto)

That was probably just luck. We need bug 1973186 for this to be fixed for good.

Flags: needinfo?(gsvelto)

The bug is linked to a topcrash signature, which matches the following criterion:

  • Top 20 desktop browser crashes on release (startup)

For more information, please visit BugBot documentation.

The severity field is not set for this bug.
:gsvelto, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(gsvelto)

(In reply to Gabriele Svelto [:gsvelto] from comment #8)

That was probably just luck. We need bug 1973186 for this to be fixed for good.

Should we make this bug depend on that one then?

Yes, this will fix this bug in terms of crash reporting even though it won't fix the root cause.

Depends on: 1973186
Flags: needinfo?(gsvelto)

Based on the topcrash criteria, the crash signature linked to this bug is not a topcrash signature anymore.

For more information, please visit BugBot documentation.

The new stackwalker is in production and this particular type of crash is not producing a bad signature anymore (though we have more crashes under this signature, but they're different crashes).

Severity: -- → S3
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: