1520929 - gather information on x86-on-aarch64 clients

Status: RESOLVED FIXED

(Core :: XPCOM, enhancement)

Description

[Nathan Froyd [:froydnj]]

This would be useful information to know, to encourage people there's a better
firefox experience out there.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: part 1 - add an isWowARM64 property to nsSystemInfo

In addition to knowing whether we're running x86-on-x86-64, we'd also
like to know about the x86-on-arm64 case.  The current code doesn't
provide enough information to determine that, so we need to query a
little bit harder.

Comment 2

[Nathan Froyd [:froydnj]]

Attachment: part 2 - send isWowARM64 through telemetry

Please let me know if there's something more detailed I need to do here.

Comment 3

[Chris H-C :chutten]

Comment on attachment 9037390 [details] [diff] [review]
part 2 - send isWowARM64 through telemetry

Review of attachment 9037390 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good and complete to me. It'll need [Data Collection Review](https://wiki.mozilla.org/Firefox/Data_Collection), of course, but technically speaking this covers code, docs, and test.

::: toolkit/components/telemetry/docs/data/environment.rst
@@ +89,5 @@
>        system: {
>          memoryMB: <number>,
>          virtualMaxMB: <number>, // windows-only
>          isWow64: <bool>, // windows-only
> +	isWowARM64: <bool>, // windows-only

I'd love a one-liner about what isWowARM64 means. Something like "true if running a non-arm64 build of Firefox on an arm64 build of Windows" (...is that even true?)

Comment 4

[Chris H-C :chutten]

Comment on attachment 9037390 [details] [diff] [review]
part 2 - send isWowARM64 through telemetry

Review of attachment 9037390 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/components/telemetry/docs/data/environment.rst
@@ +89,5 @@
>        system: {
>          memoryMB: <number>,
>          virtualMaxMB: <number>, // windows-only
>          isWow64: <bool>, // windows-only
> +	isWowARM64: <bool>, // windows-only

Wait a minute, is that a tab character? Tabs should be spaces in this file.

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

There are some r+ patches which didn't land and no activity in this bug for 2 weeks.
:froydnj, could you have a look please?

Comment 6

[Nathan Froyd [:froydnj]]

Attachment: data-review-answers.txt

Data review filled out. Be gentle, please! :)

Comment 7

[Chris H-C :chutten]

Comment on attachment 9054323 [details]
data-review-answers.txt

Preliminary note:

Please file a follow-up bug for removing or renewing this collection before it expires.

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. This collection is part of the Telemetry Environment so is documented in its documentation here: https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/data/environment.html

    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?

No. This collection will expire in 6 months.

    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?

Yes. Nathan Froyd is responsible for renewing or removing the collection before September 29, 2019.

---
Result: datareview+

Comment 8

[Eric Rahm [:erahm]]

Nathan, is this ready to land?

Comment 9

[Nathan Froyd [:froydnj]]

(In reply to Eric Rahm [:erahm] from comment #8)

Nathan, is this ready to land?

Yes, currently waiting for inbound to reopen since it's be closed...all afternoon and evening.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d49485b4f422
part 1 - add an isWowARM64 property to nsSystemInfo; r=aklotz
https://hg.mozilla.org/integration/mozilla-inbound/rev/67a15c8abdbc
part 2 - send isWowARM64 through telemetry; r=chutten,data-review=chutten

Comment 11

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/d49485b4f422
https://hg.mozilla.org/mozilla-central/rev/67a15c8abdbc

Comment 12

[Eric Rahm [:erahm]]

Nathan, should we uplift this?

Comment 13

[Nathan Froyd [:froydnj]]

Comment on attachment 9037389 [details] [diff] [review]
part 1 - add an isWowARM64 property to nsSystemInfo

Beta/Release Uplift Approval Request

• Feature/Bug causing the regression: None
• User impact if declined: Minimal impact on users. Mozilla won't be able to determine whether some users are running x86 builds on aarch64 windows, and therefore could benefit from a faster native experience.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Low risk: adding a little bit of well-understood code and some telemetry on top.
• String changes made/needed: None

Comment 14

[Pascal Chevrel:pascalc]

Comment on attachment 9037389 [details] [diff] [review]
part 1 - add an isWowARM64 property to nsSystemInfo

Additional telemetry for ARM64, low risk, uplift approved for 67 beta 10, thanks.

Comment 15

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/releases/mozilla-beta/rev/063a0c6b1078
https://hg.mozilla.org/releases/mozilla-beta/rev/4a094d23761c