Closed
Bug 838279
Opened 11 years ago
Closed 11 years ago
Report stats on whether Windows users have a 64-bit OS
Categories
(Toolkit :: Telemetry, defect)
Tracking
()
VERIFIED
FIXED
mozilla21
People
(Reporter: benjamin, Assigned: bugzilla)
Details
Attachments
(1 file)
2.69 KB,
patch
|
benjamin
:
review+
lsblakk
:
approval-mozilla-aurora+
lsblakk
:
approval-mozilla-beta-
|
Details | Diff | Splinter Review |
Some people are again talking about doing Windows 64-bit builds to solve "the PGO issue". But as far as I know, we don't actually know how many of our current users are using 64-bit versus 32-bit versions of Windows. We should at least solve that problem by reporting via telemetry what kind of OS people have. bbondy, can you paste what Windows API we'd use (from a 32-bit build) to figure this out?
Reporter | ||
Updated•11 years ago
|
OS: Mac OS X → Windows 7
Reporter | ||
Updated•11 years ago
|
Flags: needinfo?(netzen)
Comment 1•11 years ago
|
||
btw: we already have this info in the AUS pings. We also have windows service pack. http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/update/nsUpdateService.js#345 http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/update/nsUpdateService.js#3013
Comment 2•11 years ago
|
||
There is also some data available here https://metrics.mozilla.com/stats/firefox.shtml
Comment 3•11 years ago
|
||
I think we might already have this info. I've noticed the 32-bit nightly reports "x86" in the Telemetry info.arch field while the 64-bit nightly reports "x86-64" on the same machine (64-bit CPU, 64-bit OS). We should add this info to FHR as well, as FHR covers a much wider population of Firefox users (unlike Telemetry, FHR is enabled by default on all channels).
Comment 4•11 years ago
|
||
^^ Actually, ignore that.. I thought the question was asking about the number of 64-bit Nightly users
Reporter | ||
Comment 5•11 years ago
|
||
nsUpdateService.js#345 doesn't actually tell me the information I exactly want to know, because it gives me the processor architecture but not whether the OS is win64. I really don't know whether it's likely that there are 64-bit machines out there running win32, but nothing would surprise me. Apparently what I really want is IsWow64Process: http://msdn.microsoft.com/en-us/library/windows/desktop/ms684139%28v=vs.85%29.aspx
Comment 6•11 years ago
|
||
That will only return true if you are using win32 on win64 emulation. So only true when x86 process on x64 OS. I think you want IsWow64Process OR size of int pointer is 8
Flags: needinfo?(netzen)
Reporter | ||
Comment 7•11 years ago
|
||
Since we don't actually ship 64-bit anything except nightlies, I'm not that worried about that case...
Comment 8•11 years ago
|
||
Sure if this is only for non-Nightly but one day we probably will ship x64 so if you are checking wow64, name that wow64 and not x64 windows.
Comment 9•11 years ago
|
||
(In reply to Benjamin Smedberg [:bsmedberg] from comment #5) > nsUpdateService.js#345 doesn't actually tell me the information I exactly > want to know, because it gives me the processor architecture but not whether > the OS is win64. I really don't know whether it's likely that there are > 64-bit machines out there running win32, but nothing would surprise me. > > Apparently what I really want is IsWow64Process: > http://msdn.microsoft.com/en-us/library/windows/desktop/ms684139%28v=vs. > 85%29.aspx We need to stick this somewhere in mozilla for telemetry js. Adding a 'isWow64' to @mozilla.org/system-info;1 sound ok?
Reporter | ||
Comment 10•11 years ago
|
||
Yeah, that's fine.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → aklotz
Assignee | ||
Comment 11•11 years ago
|
||
Attachment #711516 -
Flags: review?(benjamin)
Comment 12•11 years ago
|
||
Comment on attachment 711516 [details] [diff] [review] Adds isWow64 to system-info and to the telemetry ping Review of attachment 711516 [details] [diff] [review]: ----------------------------------------------------------------- ::: xpcom/base/nsSystemInfo.cpp @@ +107,5 @@ > } > > +#ifdef XP_WIN > + BOOL isWow64; > + BOOL gotWow64Value = IsWow64Process(GetCurrentProcess(), &isWow64); Just wanted to mention that this API meets our minimum requirement for client OS but not for server OS. At least this page (http://www.mozilla.org/en-US/firefox/system-requirements.html) lists Windows 2003 but this API requires Windows Server 2003 with SP1. We should either update that page or load this API dynamically similar to the code example given in http://msdn.microsoft.com/en-us/library/windows/desktop/ms684139%28v=vs.85%29.aspx
Comment 13•11 years ago
|
||
Actually we already use it in netwerk so that support page is already invalid.
Assignee | ||
Comment 14•11 years ago
|
||
Looks like that has already been filed under bug 790228 but nobody's updated it.
Reporter | ||
Updated•11 years ago
|
Attachment #711516 -
Flags: review?(benjamin)
Attachment #711516 -
Flags: review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 15•11 years ago
|
||
Try build: https://tbpl.mozilla.org/?tree=Try&rev=0f7203fc17b5
Comment 16•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/63acc1f62049
Keywords: checkin-needed
Comment 17•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/63acc1f62049
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Assignee | ||
Comment 18•11 years ago
|
||
Comment on attachment 711516 [details] [diff] [review] Adds isWow64 to system-info and to the telemetry ping [Approval Request Comment] Bug caused by (feature/regressing bug #): Intention is to gather telemetry data about WOW64 to make better informed decisions about PGO issues and 64-bit builds User impact if declined: Telemetry won't be able to tell us about the prevalence of WOW64 Testing completed (on m-c, etc.): On m-c, manually queried isWow64 value from system-info Risk to taking this patch (and alternatives if risky): Minimal, very simple and easy to test String or UUID changes made by this patch: None
Attachment #711516 -
Flags: approval-mozilla-beta?
Attachment #711516 -
Flags: approval-mozilla-aurora?
Comment 19•11 years ago
|
||
Comment on attachment 711516 [details] [diff] [review] Adds isWow64 to system-info and to the telemetry ping This is not a release-blocker, but it low risk enough that we can take it on Aurora. We're already past the point of taking any more landings that are not critical on Beta.
Attachment #711516 -
Flags: approval-mozilla-beta?
Attachment #711516 -
Flags: approval-mozilla-beta-
Attachment #711516 -
Flags: approval-mozilla-aurora?
Attachment #711516 -
Flags: approval-mozilla-aurora+
Assignee | ||
Comment 20•11 years ago
|
||
Please checkin to Aurora.
Keywords: checkin-needed
Whiteboard: [needs-checkin-aurora]
Comment 21•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/fcd1d2c9dd8b
status-firefox20:
--- → fixed
status-firefox21:
--- → fixed
Keywords: checkin-needed
Whiteboard: [needs-checkin-aurora]
Comment 22•11 years ago
|
||
I confirm the fix is verified on Windows 7 x64 using FF 20.0.1 and FF 21 RC2. 21RC User Agent+Build ID: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:21.0) Gecko/20100101 Firefox/21.0 (20130507220413) All the following fields are implemented and exist in about:telemetry, showing info as expected: > let fields = ["cpucount", "memsize", "arch", "version", "kernel_version", > "device", "manufacturer", "hardware", > "hasMMX", "hasSSE", "hasSSE2", "hasSSE3", > "hasSSSE3", "hasSSE4A", "hasSSE4_1", "hasSSE4_2", > "hasEDSP", "hasARMv6", "hasARMv7", "hasNEON"]; > "hasEDSP", "hasARMv6", "hasARMv7", "hasNEON", "isWow64"];
You need to log in
before you can comment on or make changes to this bug.
Description
•