New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
32-bit Python on 64-bit Windows reports incorrect architecture #52108
Comments
When running 32-bit Python on a 64-bit version of Windows, therefore running the process in WOW64 mode, platform.machine returns a misleading value. When running in WOW64, the processor architecture is masked to appear as "x86" when the machine is actually "AMD64" (which you see on a 64-bit app on 64-bit OS). The change involves looking at an environment variable only set on WOW64 processes to see the native architecture. See http://blogs.msdn.com/david.wang/archive/2006/03/26/HOWTO-Detect-Process-Bitness.aspx The patch does not include any test, as I'm not really sure how you could test this without using the change itself to figure out when it should be tested. Suggestions would be appreciated. |
It's also inconsistent with Linux behaviour: >>> import sys
>>> sys.maxsize
2147483647
>>> import platform
>>> platform.machine()
'x86_64'
>>> platform.architecture()
('32bit', 'ELF') (on a Python compiled with "-m32" on a 64-bit Linux) |
The test could simply check that platform.machine() never returns "x86" when sys.maxsize is greater than 2**32. |
The patch looks ok. Since we are emulating the Unix uname() function here, we might as well mimic its inconsistencies. Note that platform.machine() and platform.processor() are not really very reliable ways of determining the machine type or processor. |
Antoine Pitrou wrote:
I'm not sure what such a test would prove, e.g. on 64-bit Mac OS X, Note that 'x86_64' is just a 'x86' compatible processor with |
You're right. There doesn't seem to be much consistency accross
Oops, indeed. |
bpo-7347 depends on this for proper testing, and arch_misrepresented.diff seems to have been labeled as acceptable. Would anyone be willing to check it in? |
I think it is actually pretty straightforward to write a *unit* test for this. We just need to check that the logic works correctly given the expected presence or absence of the environment variables. That doesn't test whether or not the right thing happens in the environment when you actually run a WOW64, but I don't think it is Python's responsibility to test that. If Microsoft changes the API, platform will break and the tests won't notice, but I don't think there's anything we can do about that, since as you say the API is the only way to find out what to expect for results. Test patch attached. Brian, if you can confirm that this test fails before your patch and succeeds afterward, I will commit both patches. |
Ah, that's simple enough :) Minor changes to the test patch: uname caches it's results, so I added a few lines to clear the cache before the uname calls. In order to not affect other tests, the whole thing is in a try/finally so we don't leave anything behind in cache. The test passes on both a 32 and 64 bit Python. |
Patches applied to trunk in r79294, py3k in r79298. Should this be backported? |
R. David Murray wrote:
I don't think so: applications relying on the previous behavior |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: