Closed Bug 558557 Opened 15 years ago Closed 15 years ago

Memory leak playing Flash videos

Categories

(Core :: Graphics, defect)

x86_64
Windows 7
defect
Not set
major

Tracking

()

RESOLVED FIXED

People

(Reporter: streetwolf52, Assigned: bas.schouten)

References

(Blocks 1 open bug)

Details

(Keywords: regression)

Attachments

(1 file)

User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a5pre) Gecko/20100410 Minefield/3.7a5pre Firefox/3.6 Build Identifier: 20100410035858 Playing a Flash video very quickly starts depleting available physical memory. Caused by having d2d/dw enabled. OOPP doesn't seemed involved as it can be on or off. Reproducible: Always Steps to Reproduce: 1.Enable d2d/dw 2.Play a Flash video 3. Actual Results: Available memory starts to run out. Expected Results: Very little memory should be used.
Blocks: dwrite
Severity: critical → blocker
Version: unspecified → Trunk
http://hg.mozilla.org/mozilla-central/rev/ba7e4f65c2a8 has no leak. A few d2d patches came right after this build. Might be the cause?
Keywords: regression
Priority: -- → P1
what flash version?
Severity: blocker → major
Priority: P1 → --
10.1 RC (latest one) http://hg.mozilla.org/mozilla-central/rev/e7a05d0e9fc5 has the leak. The 2d2 patches by BAS were right between the build in Comment 1 and this one.
Can't reproduce. Do you have a URL where you can reproduce?
I tried flash on msnbc and cnn. Try a long playing video. Also make sure d2d/dw is enabled and is functioning.
Can you give me a specific url? I have D2D on and it's definitely working.
OK. Video on front page of CNN leaks about 20mb/sec.
It takes something like 15 seconds before the leak is noticeable on my system. I have 9GB of RAM.
Bas is guilty until proven innocent :)
Assignee: nobody → bas.schouten
Status: UNCONFIRMED → NEW
Ever confirmed: true
(In reply to comment #9) > Bas is guilty until proven innocent :) IDK all fingers point to him :-)
This had rather dire symptoms for me: I started watching a video on Hulu and the system began writing to the page file like crazy. The entire system started become unresponsive - I couldn't switch between windows more than a few times per minute. The window playing the video lost its glass effect. Eventually the system switched to a lower resolution and stopped paging. It then popped up a warning that Minefield was using a lot of memory and asked me if it should close it. When I looked with the task manager and process explorer, Minefield's (and all the other processes') memory usage was with normal ranges (everything < 400 MB) but my pagefile had very clearly grown to 11 GB. The kernel paged and nonpaged pools were also at normal levels. There must be some memory metric that's not available to us. I have 128MB of VRAM and a DX9 card.
I can confirm this as well, and I had turned off all addons. By 15 seconds into any Youtube video, RAM usage would start increasing at a really fast rate. However, neither firefox nor plugin-container are showing nearly that drastic an increase in memory usage (in fact, Resource Monitor still says I have low memory usage).
That is also the case with me. If it's a leak then possibly Windows doesn't know it's FF hence we don't see it's memory usage rise as fast as the system in general?
CNN video shot up to 6gb real quick, I have 8gig on system. However, for me YouTube seems to play ok without any memory spikes. Hulu, did however take off within a few seconds of starting up a video. AMD Althlon Phenom II Quad 8gig RAM hd3200 on-board video Today's nightly: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a5pre) Gecko/20100410 Minefield/3.7a5pre ID:20100410035858
The memory is being charged to the Modified Page List - likely from shared memory not backed by a file - and subsequently written out to the page file as physical memory is filled. It only seems to happen for me with both D2D and DW enabled.
I suspect I'm guilty, I've not figured out why yet. This probably happens for windowless flash players only (they use ImageSurfaces to draw). Then this probably starts happening once you run out of VRAM and the D3D starts swapping textures out to RAM. This is probably also why we don't see the process consuming that memory. I don't see yet why I'm leaking here, but I'm looking into it.
Attached patch Make sourceBitmap a RefPtr (deleted) — Splinter Review
This needs to be a RefPtr so that the initial reference acquired by the creation functions is actually released!
Attachment #438289 - Flags: review?(jmuizelaar)
Attachment #438289 - Flags: review?(jmuizelaar) → review+
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
What a guy :-)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: