Closed
Bug 1448703
Opened 6 years ago
Closed 6 years ago
Intermittent /html/semantics/forms/constraints/form-validation-reportValidity.html | application crashed [@ CrashStatsLogForwarder::CrashAction]
Categories
(Core :: Graphics: WebRender, defect, P5)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla61
People
(Reporter: intermittent-bug-filer, Assigned: lsalzman)
References
Details
(Keywords: crash, intermittent-failure)
Crash Data
Attachments
(2 files)
3.51 KB,
patch
|
jrmuizel
:
review+
|
Details | Diff | Splinter Review |
5.34 KB,
patch
|
jrmuizel
:
review+
|
Details | Diff | Splinter Review |
Filed by: nbeleuzu [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=170158111&repo=mozilla-inbound https://queue.taskcluster.net/v1/task/ESHL77N6TGaJTGnm2fmnZw/runs/0/artifacts/public/logs/live_backing.log
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Updated•6 years ago
|
Component: web-platform-tests → Graphics: WebRender
Product: Testing → Core
Version: Version 3 → unspecified
Comment 3•6 years ago
|
||
Lee, here's another crash with this: PID 5609 | [GFX1 38]: Failed to get UnscaledFont entry for FontKey 35 and it happened on April 1st which is after your previous fix (bug 1447910) landed. Can you look into it? There's a slew of these bugs and I'm going to mark them dependent on this one, but I'm not sure how many of them were fixed by the old issue and how many are still occurring (I could check, but based on this one it's not fixed yet, so I'm not going to bother checking).
Comment 4•6 years ago
|
||
> Failed to get UnscaledFont entry for FontKey is the GraphicsCriticalError of recent crashes of bug 1418012.
See Also: → 1418012
Updated•6 years ago
|
Blocks: stage-wr-trains
Assignee | ||
Comment 5•6 years ago
|
||
Bug 1447910 assumed that by fixing the ordering of font key deletions to be bound by a transaction, it would get rid of possible race condition shenanigans. While the incidence of such errors seems to have gone done, they were not eliminated. To figure out why these are still occurring, it would help to know the reason a font doesn't exist at the time it is queried. This can hypothetically be because of 3 reasons: 1) the font was deleted, 2) the namespace was cleared, or 3) we just have no record of the font. This implements a simple ring-buffer log to keep track of that, noting when and why the font was deleted. When the error is encountered, it looks back in the log and checks why. This should only have a small static space overhead (~2KB), and minimal performance aspect, so I just left this on in both debug and opt builds so we can get more useful info from errors in both scenarios.
Attachment #8966091 -
Flags: review?(jmuizelaar)
Updated•6 years ago
|
Attachment #8966091 -
Flags: review?(jmuizelaar) → review+
Pushed by lsalzman@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/e352d286f176 log WR blob image font deletions for debugging. r=jrmuizel
Assignee | ||
Updated•6 years ago
|
Keywords: leave-open
Comment hidden (Intermittent Failures Robot) |
Comment 8•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e352d286f176
Comment 9•6 years ago
|
||
So in https://treeherder.mozilla.org/logviewer.html#?job_id=173764959&repo=try&lineNumber=3343 for example we this output: [GFX1 38]: Failed to get UnscaledFont entry for FontKey 5 because cleared namespace Lee, next steps here?
Assignee | ||
Comment 11•6 years ago
|
||
This will be resolved in part by WR PR https://github.com/servo/webrender/pull/2676
Depends on: 1454623
See Also: → https://github.com/servo/webrender/pull/2676
Assignee | ||
Comment 12•6 years ago
|
||
We need to make sure we don't clear out blob image resources until such time as the ClearNamespace message is actually processed. So this uses the new clear_namespace hook in https://github.com/servo/webrender/pull/2676 to call ClearBlobImageResources at that time.
Attachment #8969379 -
Flags: review?(jmuizelaar)
Updated•6 years ago
|
Attachment #8969379 -
Flags: review?(jmuizelaar) → review+
Comment hidden (Intermittent Failures Robot) |
Comment 14•6 years ago
|
||
Pushed by lsalzman@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/42781c597745 clear blob image resources from clear_namespace hook. r=jrmuizel
Comment 15•6 years ago
|
||
Backed out for leakchecks Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=42781c5977453fc4c1ac9538c9eef240456b3e10 Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=175250791&repo=mozilla-inbound&lineNumber=11872 (tier2) https://treeherder.mozilla.org/logviewer.html#?job_id=175255399&repo=mozilla-inbound&lineNumber=7705 (tier1) Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/4f3e0c71818804f816c5e37f326ec4d9a4446f00
Flags: needinfo?(lsalzman)
Comment 16•6 years ago
|
||
Pushed by lsalzman@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/3ad3cb5e9b65 clear blob image resources from clear_namespace hook. r=jrmuizel
Assignee | ||
Comment 17•6 years ago
|
||
The leak should be fixed now. If this finally gets rid of all of these intermittents too, then we can close out this bug and its related/duplicated ilk.
Flags: needinfo?(lsalzman)
Comment 18•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3ad3cb5e9b65
Updated•6 years ago
|
Comment 19•6 years ago
|
||
Pushed by kgupta@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/c1b58d542d97 Follow-up to regenerate FFI bindings. r=me
Comment 20•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c1b58d542d97
Comment 21•6 years ago
|
||
Pushed by lsalzman@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/81601f1ab879 follow-up - disambiguate unknown font from cleared all. r=me
Comment 22•6 years ago
|
||
Backed out for build bustages at builds/worker/workspace/build/src/gfx/webrender_bindings/Moz2DImageRenderer.cpp:108 Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=81601f1ab8799a121d82d5b560888aa7296fc776 Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=175760347&repo=mozilla-inbound&lineNumber=18324 Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/7a50e32d708acd69e5fc41dbc3eb6815d93cfbde
Updated•6 years ago
|
Flags: needinfo?(lsalzman)
Assignee | ||
Comment 23•6 years ago
|
||
(In reply to Andreea Pavel [:apavel] from comment #22) > Backed out for build bustages at > builds/worker/workspace/build/src/gfx/webrender_bindings/Moz2DImageRenderer. > cpp:108 > > Push that started the failures: > https://treeherder.mozilla.org/#/jobs?repo=mozilla- > inbound&revision=81601f1ab8799a121d82d5b560888aa7296fc776 > > Failure log: > https://treeherder.mozilla.org/logviewer.html#?job_id=175760347&repo=mozilla- > inbound&lineNumber=18324 > > Backout: > https://hg.mozilla.org/integration/mozilla-inbound/rev/ > 7a50e32d708acd69e5fc41dbc3eb6815d93cfbde Fixed
Flags: needinfo?(lsalzman)
Comment 24•6 years ago
|
||
Pushed by lsalzman@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/28c2803c243a follow-up - disambiguate unknown font from cleared all. r=me
Comment 25•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/28c2803c243a
Comment 26•6 years ago
|
||
I'm going to claim and all the intermittent failure bugs depending on this one are fixed. I went through the intermittent-failures database and there's nothing newer than Apr 24, but the frequency wasn't that high to being with so I can't be sure. But we can close this now and file a new bug if there's still stuff that needs fixing.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Updated•6 years ago
|
Keywords: leave-open
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•