Closed
Bug 1243950
Opened 9 years ago
Closed 9 years ago
MOZ_PROCESS_LOG is not properly written to for Windows content processes on mozilla-central
Categories
(Core :: IPC, defect)
Core
IPC
Tracking
()
RESOLVED
DUPLICATE
of bug 1193861
People
(Reporter: mccr8, Unassigned)
References
Details
(Whiteboard: [MemShrink])
In bug 1219369, I found that we weren't creating a bloat log on Windows. I thought this was weird, because we actually check that every process in the MOZ_PROCESS_LOG has a bloat log, if appropriate. It turns out that as far as I can see from looking at a mochitest log, a content process on Windows m-c (but not Linux m-c or Windows Aurora) fails to write to MOZ_PROCESS_LOG, presumably for the same reason (like sandboxing) as bug 1219369.
Two things need to happen:
1) This file opening or whatever needs to be fixed so that it works in Windows content processes.
2) EnvironmentLog::print() needs to MOZ_ASSERT or otherwise error out if it fails to create the file, so that we will notice this issue in debug builds.
Reporter | ||
Comment 1•9 years ago
|
||
Bob, is this something you could look into?
Flags: needinfo?(bobowen.code)
Reporter | ||
Comment 2•9 years ago
|
||
Failing to write to the process log is bad for reasons beyond leak checking. I think the test harness uses it to do things like detect and kill zombie processes, so there could be weird Mochitest failures on Windows being caused by this.
tracking-e10s:
--- → ?
Comment 3•9 years ago
|
||
I looked into this in bug 1193861 and even have a patch there, but ultimately decided to use psutil to manage child processes from the harness (for bug 1143547).
This doesn't work on OS X either (bug 950401).
Reporter | ||
Comment 4•9 years ago
|
||
Great.
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(bobowen.code)
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•