Closed Bug 1637714 Opened 4 years ago Closed 4 years ago

Delay WMF system DLL loading

Categories

(Core :: Audio/Video, enhancement)

enhancement

Tracking

()

RESOLVED DUPLICATE of bug 1595994

People

(Reporter: emmamalysz, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxperf])

Attachments

(1 file)

4.53 MB, application/x-7z-compressed
Details

Right now, the following DLLs are loaded during startup: https://searchfox.org/mozilla-central/source/dom/media/platforms/wmf/WMFUtils.cpp#171-176. In the attached profile, these reads took over a second during a cold startup on an HDD device. If they are not necessary for startup, let's see if we can delay loading these system DLLs to improve Disk I/O.

Jean-Yves, is it feasible / do you know what it would take to defer running wmf::MFStartup() until after startup, and maybe just lazily initialize it if we're in a startup path that needs it? It feels like something which would not commonly be needed for about:home or a typical page loaded on startup, and its DLLs do seem to represent a nontrivial chunk of startup IO.

Flags: needinfo?(jyavenard)

(In reply to Doug Thayer [:dthayer] from comment #1)

Jean-Yves, is it feasible / do you know what it would take to defer running wmf::MFStartup() until after startup, and maybe just lazily initialize it if we're in a startup path that needs it? It feels like something which would not commonly be needed for about:home or a typical page loaded on startup, and its DLLs do seem to represent a nontrivial chunk of startup IO.

Problem is that on startup we check HW acceleration support ; this is done very early on and require the DLL to be loaded.

However, what you are trying to achieve will be accomplished once bug 1595994 is completed.

It's really the only way to go about it that won't slow down all content processes.

I'll mark this bug as duplicate of that bug, as that's one of the aim (along many others)

Status: NEW → RESOLVED
Closed: 4 years ago
Flags: needinfo?(jyavenard)
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: