1686974 - GPU memory reporter should run in the process where we use the GPU

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Jeff Muizelaar [:jrmuizel]]

It looks like it's currently only running in the parent process and not in the GPU process.

Comment 1

[Jeff Muizelaar [:jrmuizel]]

Sotaro, are you able to take a look at fixing this?

Comment 2

[Sotaro Ikeda [:sotaro]]

(In reply to Jeff Muizelaar [:jrmuizel] from comment #1)

Sotaro, are you able to take a look at fixing this?

OK, I am going to look into it. :jrmuizel, how did you notice it?

Comment 3

[Jeff Muizelaar [:jrmuizel]]

Was looking for these numbers in the about:memory:
https://searchfox.org/mozilla-central/source/gfx/thebes/gfxWindowsPlatform.cpp#240
I noticed that they were zero and only reported for the parent process.

Comment 4

[Sotaro Ikeda [:sotaro]]

Thank you!

Comment 5

[Sotaro Ikeda [:sotaro]]

On GPU Process gfxWindowsPlatform() is not created. It creates GPUAdapterReporter. Then on GPU process, GPUAdapterReporter was not created.

Comment 6

[Sotaro Ikeda [:sotaro]]

Attachment: Bug 1686974 - Add memory reporters to GPU process

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/91a03ce2391f
Add memory reporters to GPU process r=jrmuizel

Comment 8

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/91a03ce2391f