1645732 - Collect Linux back end telemetry data

Status: RESOLVED FIXED

(Core :: Graphics, enhancement)

Description

[Martin Stránský [:stransky] (ni? me)]

Collect Linux back end telemetry data (Wayland/X11) to better plan Linux development.
It's at about:support, "Window Protocol" and can be "Wayland", "Wayland/drm", "x11" for now.

Comment 1

[Jan-Erik Rediger [:janerik]]

Data collections are usually implemented by the components doing the collection. Looks like Core:Graphics is the right place to do that. In bug 1554540 it was added to about:support.

Comment 2

[Darkspirit]

https://searchfox.org/mozilla-central/rev/25d491b7924493b5d4bedc07841a1cd92f271100/toolkit/content/aboutSupport.js#603-604
Please include windowProtocol and, if possible, desktopEnvironment in crash reports as well.

Comment 3

[Mike Kaply [:mkaply] (Out until 9/29)]

Here's the process for this.

https://wiki.mozilla.org/Firefox/Data_Collection

We'll need to do a data collection request. I'll own working on that.

Comment 4

[Mike Kaply [:mkaply] (Out until 9/29)]

Attachment: Data collection form

Comment 5

[Chris H-C :chutten]

Comment on attachment 9157336 [details]
Data collection form

There is a new version of the Data Review Request form: https://github.com/mozilla/data-review/blob/master/request.md

Of specific notice is that there is a new question about where documentation for this collection will live.

(( Otherwise the request looks fine. This'll be an easy r+ on the new form ))

Comment 6

[Mike Kaply [:mkaply] (Out until 9/29)]

Attachment: Data collection form

I think this is right...

Comment 7

[Chris H-C :chutten]

Comment on attachment 9165777 [details]
Data collection form

So sorry for the delay, somehow I missed your request.

DATA COLLECTION REVIEW RESPONSE:

    Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate?

Yes. 

    Is there a control mechanism that allows the user to turn the data collection on and off?

Yes. This collection is Telemetry so can be controlled through Firefox's Preferences.

    If the request is for permanent data collection, is there someone who will monitor the data over time?

Yes, :mkaply is responsible.

    Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 1, Technical.

    Is the data collection request for default-on or default-off?

Default on for all channels.

    Does the instrumentation include the addition of any new identifiers?

No.

    Is the data collection covered by the existing Firefox privacy notice?

Yes.

    Does there need to be a check-in in the future to determine whether to renew the data?

No. This collection is permanent.

---
Result: datareview+

Comment 8

[Mike Kaply [:mkaply] (Out until 9/29)]

crash report annotation is here:

https://searchfox.org/mozilla-central/source/widget/GfxInfoX11.cpp#55

I think I can add telemetry here:

https://searchfox.org/mozilla-central/source/widget/GfxInfoX11.cpp#693

any advice on where to put this chutten? I guess I'm looking for something that only happens once.

Most of this is called from FindBlocklistedDeviceInList. I have no idea how often that gets called.

Martin, is this something one of your team can take on? We have permission to do it now and I can help with getting the code together, just don't know where to put it.

Comment 9

[Chris H-C :chutten]

I'm afraid I don't know gfx code all that well, so I can't point to any place that's called once. (and then to find one that also has easy access to the backend name). But when you do, I presume you're planning on stuffing it in a String Scalar? You won't have any problems accessing that from wherever you are (even if you have to do it from inside the gpu process)

Comment 10

[Martin Stránský [:stransky] (ni? me)]

(In reply to Mike Kaply [:mkaply] from comment #8)

https://searchfox.org/mozilla-central/source/widget/GfxInfoX11.cpp#693

any advice on where to put this chutten? I guess I'm looking for something that only happens once.

Most of this is called from FindBlocklistedDeviceInList. I have no idea how often that gets called.

Martin, is this something one of your team can take on? We have permission to do it now and I can help with getting the code together, just don't know where to put it.

https://searchfox.org/mozilla-central/source/widget/GfxInfoX11.cpp#693 is a good place, it's called only once after Firefox start.
Yes, I can review/create the patch if you give me some hints how to export the data.

Thanks.

Comment 11

[Mike Kaply [:mkaply] (Out until 9/29)]

Attachment: Bug 1645732 - Add telemetry for Linux window protocol.

Comment 12

[Mike Kaply [:mkaply] (Out until 9/29)]

Martin: I think this patch will do it.

The way to test is to build and then start Firefox and go to about:telemetry.

Search on LINUX_WINDOW_PROTOCOL or Wayland and hopefully you'll see something.

Comment 13

[Martin Stránský [:stransky] (ni? me)]

(In reply to Mike Kaply [:mkaply] from comment #12)

Martin: I think this patch will do it.

The way to test is to build and then start Firefox and go to about:telemetry.

Search on LINUX_WINDOW_PROTOCOL or Wayland and hopefully you'll see something.

Yes, I do see 'gfx.linux_window_protocol' is 'wayland/drm' in my case.

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e55422de3781
Add telemetry for Linux window protocol. r=chutten,aosmond

Comment 15

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/e55422de3781