Skip to content
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

ARM 32-bit issues (again) #5656

Closed
claudioandre-br opened this issue Feb 3, 2025 · 0 comments · Fixed by #5663
Closed

ARM 32-bit issues (again) #5656

claudioandre-br opened this issue Feb 3, 2025 · 0 comments · Fixed by #5663

Comments

@claudioandre-br
Copy link
Member

We can/should consider the 32-bit world in the past and ignore it completely.

The logic used to configure packages (in this case ARM) is not properly adjusted.

The congure detects that it is 32-bit, but then the SIMD tests (probably inside jtr_arm_logic.m4) force it to use a 64-bit header.

Let's dig into it:

:: -----------------------------------------------------------
:: Build system information
:: --------------------------------
:: Linux
:: armv7l
:: Linux bos03-arm64-023 6.8.0-51-generic #52-Ubuntu SMP PREEMPT_DYNAMIC Thu Dec  5 13:32:09 UTC 2024 armv7l armv7l armv7l GNU/Linux
:: --------------------------------
:: processor  : 0
:: model name  : ARMv8 Processor rev 1 (v8l)
:: BogoMIPS  : 50.00
:: Features  : half thumb fastmult vfp edsp neon vfpv3 tls vfpv4 idiva idivt lpae evtstrm fphp asimdhp asimddp aes pmull sha1 sha2 crc32 ssbs
:: CPU implementer  : 0x41
:: CPU architecture: 8
:: CPU variant  : 0x3
:: CPU part  : 0xd0c
:: CPU revision  : 1
:: --------------------------------
:: DISTRIB_DESCRIPTION="Ubuntu 24.04.1 LTS"
:: PRETTY_NAME="Ubuntu 24.04.1 LTS"

So it's arm 7 running on modern/up-to-date hardware.

But:

:: checking if gcc supports -m64... skipped
:: checking for 32/64 bit... 32-bit
[...]
:: checking special compiler flags... ARM
:: configure: Testing build host's native CPU features
:: checking for NEON... yes
:: checking for ASIMD... yes
:: checking if gcc supports -fno-strict-aliasing... yes
:: checking for extra ASFLAGS... None needed
:: checking for arch.h alternative... arm64le.h
[...]
:: Target CPU ......................................... armv7l ASIMD, 32-bit LE
:: Target OS .......................................... linux-gnueabihf
:: Cross compiling .................................... no
:: Legacy arch header ................................. arm64le.h

See also #5631.

  • If the CPU being used supports something, it doesn't mean that the whole environment is this or that.
  • Other checks are simply ignored.
claudioandre-br added a commit to claudioandre-br/JohnTheRipper that referenced this issue Feb 9, 2025
It doesn't make much sense to run this test if we already know we're
on ARMv7 hardware.

Fix: openwall#5656.

Signed-off-by: Claudio André <dev@claudioandre.slmail.me>
claudioandre-br added a commit to claudioandre-br/JohnTheRipper that referenced this issue Feb 9, 2025
It doesn't make much sense to run this test if we already know we're
on ARMv7 hardware.

Fix: openwall#5656.

Signed-off-by: Claudio André <dev@claudioandre.slmail.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants