Closed
Bug 1378944
Opened 6 years ago
Closed 6 years ago
Intermittent TEST-UNEXPECTED-TIMEOUT | gfx/layers/apz/test/mochitest/test_wheel_scroll.html | application timed out after 330 seconds with no output
Categories
(Core :: Security: Process Sandboxing, defect)
Core
Security: Process Sandboxing
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: kats)
References
(Blocks 1 open bug)
Details
(Keywords: intermittent-failure, Whiteboard: [stockwell fixed:other]sb+)
Attachments
(1 file)
Filed by: wkocher [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=112245990&repo=autoland https://queue.taskcluster.net/v1/task/D9LL18GmTNmsI3gtNMGFgA/runs/0/artifacts/public/logs/live_backing.log
Assignee | ||
Comment 1•6 years ago
|
||
Log contains this: Sandbox: seccomp sandbox violation: pid 1131, tid 1200, syscall 241, args 0 128 3685376576 4293294352 3685376704 3685376712. Killing process. And then the dead process causes the test to time out.
Component: Panning and Zooming → Security: Process Sandboxing
Comment 2•6 years ago
|
||
On x86_64 syscall 241 is |__NR_mq_unlink|, on x86_32 it's |__NR_sched_setaffinity|.
Comment 3•6 years ago
|
||
Those numbers in comment #1 look very 32-bit (also, Treeherder shows it's a 32-bit job). This seems to be used by Chaos Mode: https://searchfox.org/mozilla-central/rev/5e1e8d2f244bd8c210a578ff1f65c3b720efe34e/xpcom/threads/nsThread.cpp#467 What I don't understand is why this would be intermittent, or would have started recently — the code in question seems to have been present (and enabled for this test) for years.
Assignee | ||
Comment 5•6 years ago
|
||
Two bugs filed with this failure today after not seeing it ever before. Something must have changed recently to start triggering this.
Comment hidden (Intermittent Failures Robot) |
Comment 7•6 years ago
|
||
(In reply to Jed Davis [:jld] (⏰UTC-6) from comment #3) > This seems to be used by Chaos Mode: > https://searchfox.org/mozilla-central/rev/ > 5e1e8d2f244bd8c210a578ff1f65c3b720efe34e/xpcom/threads/nsThread.cpp#467 I would propose whitelisting the syscall via our preferences, specifically for tests that rely on Chaos Mode?
![]() |
||
Updated•6 years ago
|
Whiteboard: sblc3
Assignee | ||
Comment 8•6 years ago
|
||
We could also remove chaos mode from the tests. I don't think the tests *rely* on it, they just use it because it flushes out intermittent bugs faster, but I don't know if they really provide that much value.
Comment hidden (Intermittent Failures Robot) |
Comment 10•6 years ago
|
||
This particular chaos mode feature, as it's used here, seems to apply only to threads started while the test is running. That might help explain why it's intermittent and started happening recently. But also, I'm not sure how much it's helping in that case. If it were enabled at process startup it might be more useful, and then we could check for it when creating the sandbox policy so we wouldn't have to change anything for non-test usage.
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
![]() |
||
Updated•6 years ago
|
Whiteboard: sblc3 → sb+
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 15•6 years ago
|
||
So what should we do here? Is somebody going to whitelist/work around this in the code, or should I just remove the chaos-mode annotation?
Flags: needinfo?(jld)
Comment 16•6 years ago
|
||
I'd say we should remove the chaos mode annotations. They don't seem to really be doing anything useful at present; you'd need to enable the thread scheduling changes early in startup so they apply to all threads, at which point it'd be easy to test for it when building the sandbox policy.
Flags: needinfo?(jld)
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → bugmail
Comment hidden (mozreview-request) |
Comment 18•6 years ago
|
||
mozreview-review |
Comment on attachment 8897151 [details] Bug 1378944 - Stop running some APZ mochitests with chaos mode since it causes sandbox failures and doesn't provide much value. https://reviewboard.mozilla.org/r/168452/#review173688
Attachment #8897151 -
Flags: review?(jld) → review+
Comment 19•6 years ago
|
||
Pushed by kgupta@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f374ccb241c7 Stop running some APZ mochitests with chaos mode since it causes sandbox failures and doesn't provide much value. r=jld
Comment 20•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f374ccb241c7
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Comment hidden (Intermittent Failures Robot) |
Updated•6 years ago
|
Whiteboard: sb+ → [stockwell fixed:other]sb+
You need to log in
before you can comment on or make changes to this bug.
Description
•