All users were logged out of Bugzilla on October 13th, 2018

Report support for AVX and AVX2 in telemetry

RESOLVED FIXED in Firefox 49

Status

()

RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: jrmuizel, Assigned: jrmuizel)

Tracking

(Blocks: 1 bug)

unspecified
mozilla49
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox49 fixed)

Details

(Whiteboard: [measurement:client])

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Updated

3 years ago
Depends on: 1270591
Blocks: 1121467
Whiteboard: [measurement:client]
(Assignee)

Comment 1

2 years ago
Created attachment 8753561 [details] [diff] [review]
Report AVX and AVX2
Attachment #8753561 - Flags: review?(gfritzsche)
Assignee: nobody → jmuizelaar
Comment on attachment 8753561 [details] [diff] [review]
Report AVX and AVX2

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

This looks fine to me, flagging bsmedberg for data collection review:
https://wiki.mozilla.org/Firefox/Data_Collection

::: toolkit/components/telemetry/TelemetryEnvironment.jsm
@@ -1201,5 @@
>      };
>  
>      const CPU_EXTENSIONS = ["hasMMX", "hasSSE", "hasSSE2", "hasSSE3", "hasSSSE3",
> -                            "hasSSE4A", "hasSSE4_1", "hasSSE4_2", "hasEDSP", "hasARMv6",
> -                            "hasARMv7", "hasNEON"];

Please also update the documentation in toolkit/components/telemetry/docs/environment.rst
You can view it using |mach doc| and is used to generate this:
https://gecko.readthedocs.org/en/latest/toolkit/components/telemetry/telemetry/environment.html
Attachment #8753561 - Flags: review?(gfritzsche)
Attachment #8753561 - Flags: review+
Attachment #8753561 - Flags: feedback?(benjamin)

Comment 3

2 years ago
Comment on attachment 8753561 [details] [diff] [review]
Report AVX and AVX2

With the docs updated, data-review=me
Attachment #8753561 - Flags: feedback?(benjamin) → feedback+

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/2fc681d07cc0
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.