Closed
Bug 828632
Opened 12 years ago
Closed 12 years ago
running proxy config code off the main thread on OS X causes memory leak, syslogd load
Categories
(Core :: Networking, defect)
Tracking
()
People
(Reporter: s.t.smith, Assigned: mcmanus)
References
Details
(Keywords: regression)
Attachments
(1 file)
1.07 KB,
patch
|
jaas
:
review+
akeybl
:
approval-mozilla-aurora+
akeybl
:
approval-mozilla-beta+
bajaj
:
approval-mozilla-release+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_7_2) AppleWebKit/534.52.7 (KHTML, like Gecko) Version/5.1.2 Safari/534.52.7
Steps to reproduce:
New upgrade to FF 18.0 on OS X Lion 10.7.2.
System Preferences>Network>Advanced...>Proxies>Automatic Proxy Configuration: file://localhost/Library/Internet Plug-Ins/proxy.pac
Firefox>Preferences>Advanced>Network>Settings...>Use system proxy settings
As this bug already has a RESOLVED status, I'll start a new ticket.
Actual results:
FF hangs, system log fills at huge rate with messages that look like:
org.mozilla.firefox objc Object class __NSCFData autoreleased with no pool in place - just leaking - break on objc_autoreleaseNoPool() to debug
syslogd cant's keep up with the error load and the Console app pizza's until I tell FF to stop using my system proxy settings, which use a local proxy.pac file.
Expected results:
No memory leak, no syslogd errors when FF is set to use system proxy settings, which are set to use a local proxy.pac file.
Updated•12 years ago
|
Component: Untriaged → Networking
Product: Firefox → Core
Updated•12 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Any ideas Patrick? Since this seems to be fixed on m-c, perhaps this is a patch that just didn't get approved for the FF18 branch?
Comment 4•12 years ago
|
||
Cc'ing Asa to help expedite through rapid release.
So is this a dup too? What's the bug with the fix-patch?
/be
Assignee | ||
Comment 5•12 years ago
|
||
(In reply to Josh Aas (Mozilla Corporation) from comment #3)
> Any ideas Patrick? Since this seems to be fixed on m-c, perhaps this is a
> patch that just didn't get approved for the FF18 branch?
This is in my queue to look at.
I'm not 100% convinced this is fixed in nightly - would be worth testing that. If that's true its almost certainly for the lack of the same patches that weren't approved for 18 that are causing the problem in 828202. (which I don't know off the top of my head, but I can figure out.)
If you've got a mac handy can you confirm the 18/nightly (or 19) behavior? The only one I have is old enough to be a real pain to work with.
Comment 6•12 years ago
|
||
Need automated test coverage.
/be
Assignee | ||
Comment 7•12 years ago
|
||
(In reply to Brendan Eich [:brendan] from comment #4)
> Cc'ing Asa to help expedite through rapid release.
>
> So is this a dup too? What's the bug with the fix-patch?
>
If it is a dup of 828202 then the patches in question are 815783 and 819902
Updated•12 years ago
|
tracking-firefox18:
--- → ?
Assignee | ||
Comment 8•12 years ago
|
||
If you can reproduce this bug I'd appreciate you testing another build. I have not yet reproduced myself (still working on it), but there are a couple possible related patches that I have included in a test build (which is still compiling)
Builds will appear here based on FF18 with the prospective fixes included
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/mcmanus@ducksong.com-4bd823c0f036
The builds could take a while (anywhere from 30 mins to a few hours) to appear.
Updated•12 years ago
|
status-firefox18:
--- → affected
Assignee | ||
Comment 9•12 years ago
|
||
I think there are two issues here.. one is a dup of 828202 - if you hit that (depends on your pac uri) then you get a crazy spin.. this is already fixed >= ff19. the other issue is a leak/syslog that seems to come from running the os x system interaction stuff off the main thread (I'm having trouble quantifying its size).. if you're not getting the spin then this log is obviously happening a lot less. That problem exists on 18->21.
to fix the latter, fortunately we've already got a mechanism to ping/pong this stuff between threads so it runs on the main thread. (linux uses it). so initial tests indicate this can be fixed just by flipping a bool.
I'll use this bug for the leak/log issue.
Assignee | ||
Comment 10•12 years ago
|
||
STS can you try the build below and report back if that resolves the problem for you?
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/mcmanus@ducksong.com-2102aa36db35/try-macosx64/
Flags: needinfo?(s.t.smith)
Assignee | ||
Comment 11•12 years ago
|
||
so 769764 moved the code that interacts with the system proxy settings off the main thread.. that code invariably involves os specific calls which are kind of opaque and they could block (or be changed to block) as they have been observed to do on windows.
however, executing that code on the pac thread on OSX seems to result in a syslog (and maybe a memleak if the log is to be believed) which can pile up at a high rate of speed.
fortunately there is already a mechanism for running this stuff on the main thread (linux glib requires it) so that can easily be fixed - it fixes the syslog about the leak for me. The only risk is that the system call could block but I haven't observed OS X to ever do that.. and if it were to do so that would still be a smaller problem imo.
given the osx nature of this I think josh is probably the best r?
Attachment #701144 -
Flags: review?(joshmoz)
Attachment #701144 -
Flags: review?(joshmoz) → review+
Updated•12 years ago
|
Assignee: nobody → mcmanus
Blocks: 769764
status-firefox19:
--- → affected
status-firefox20:
--- → affected
status-firefox21:
--- → affected
tracking-firefox19:
--- → ?
tracking-firefox20:
--- → ?
Keywords: regression
Hardware: x86_64 → All
Summary: System Proxy Configuration Causes Memory Leak, syslogd load → running proxy config code off the main thread on OS X causes memory leak, syslogd load
Assignee | ||
Comment 12•12 years ago
|
||
Updated•12 years ago
|
Reporter | ||
Comment 13•12 years ago
|
||
(In reply to Patrick McManus [:mcmanus] from comment #10)
> STS can you try the build below and report back if that resolves the problem
> for you?
>
> http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/mcmanus@ducksong.
> com-2102aa36db35/try-macosx64/
Thanks -- that build works for me!
Flags: needinfo?(s.t.smith)
Assignee | ||
Comment 14•12 years ago
|
||
Comment on attachment 701144 [details] [diff] [review]
patch 0
[Approval Request Comment]
Bug caused by (feature/regressing bug #): 769764
User impact if declined: users with proxies defined on OSX via System Configuration will see a serious CPU tax and maybe a severe memory leak. This involves repeated syslog warnings which will slow the system.
Testing completed (on m-c, etc.): on m-i, and reporter confirms fix in comment 13
Risk to taking this patch (and alternatives if risky): The patch is simple (1 line) and while it does change the runtime logic considerably by using a different data flow that flow is already used in linux.
String or UUID changes made by this patch: none
Attachment #701144 -
Flags: approval-mozilla-release?
Attachment #701144 -
Flags: approval-mozilla-beta?
Attachment #701144 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 15•12 years ago
|
||
Thanks STS!
Comment 16•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Comment 17•12 years ago
|
||
Comment on attachment 701144 [details] [diff] [review]
patch 0
Please land on Aurora/Beta asap.
Attachment #701144 -
Flags: approval-mozilla-beta?
Attachment #701144 -
Flags: approval-mozilla-beta+
Attachment #701144 -
Flags: approval-mozilla-aurora?
Attachment #701144 -
Flags: approval-mozilla-aurora+
Comment 18•12 years ago
|
||
Updated•12 years ago
|
Attachment #701144 -
Flags: approval-mozilla-release? → approval-mozilla-release+
Comment 19•12 years ago
|
||
Comment 20•12 years ago
|
||
Did some basic verification of this one:
Setup:
-- OS X 10.8
-- Squid proxy server running locally
-- Fx using System Settings
-- Mac Sys Prefs set to file://localhost/Users/steveworkman/proxySquid.pac
Fx18:
-- Problem reproduced: Memory leak messages observed in syslog with Squid stopped (2 errors) and started (~20 errors trying google.com).
Fx 18.0.1:
-- No memory leak messages in syslog with Squid stopped or started.
Reporter | ||
Comment 21•12 years ago
|
||
(In reply to Steve Workman [:sworkman] from comment #20)
> Did some basic verification of this one:
>
> Setup:
> -- OS X 10.8
> -- Squid proxy server running locally
> -- Fx using System Settings
> -- Mac Sys Prefs set to file://localhost/Users/steveworkman/proxySquid.pac
>
> Fx18:
> -- Problem reproduced: Memory leak messages observed in syslog with Squid
> stopped (2 errors) and started (~20 errors trying google.com).
>
> Fx 18.0.1:
> -- No memory leak messages in syslog with Squid stopped or started.
FWIW, I'm also running a squid proxy when I see this bug.
Comment 22•12 years ago
|
||
verified fixed running 18.0.1 on OS X 10.6.8
No memory leak messages in syslog with pac file
Comment 23•12 years ago
|
||
Verified as fixed on Firefox 19 beta 2 - no memory leak messages in syslog after configuring a system proxy using a file:// pac file.
Verified on Mac OS X 10.7.5 and on Mac OS X 10.8.2:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:19.0) Gecko/20100101 Firefox/19.0 Build ID: 20130116072953
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:19.0) Gecko/20100101 Firefox/19.0 Build ID: 20130116072953
QA Contact: simona.marcu
Comment 24•12 years ago
|
||
This issue is also verified on Firefox 19 beta 6 (20130212082553), on the next machines:
1) Ubuntu 12.04 64-bit
2) Windows 7 64-bit
3) Windows 8 32-bit
4) Mac OSX 10.6.8
Comment 25•12 years ago
|
||
Batch edit: Bugs marked status-b2g18: affected after 2/13 branching of v1.0.1 are now also status-b2g18-v1.0.1: affected
status-b2g18-v1.0.1:
--- → affected
You need to log in
before you can comment on or make changes to this bug.
Description
•