Open mmilata opened 1 year ago
I will be back 10.7. I would build latest monero, simulator works fine, huge PR waits to be merged.
Tests are broken on real device, right?
Yep, real device.
Btw if the testsuite is basically a patch on top of the monero repo we might be able to easily build it ourselves using nix.
I will create a new testing binary from this PR https://github.com/monero-project/monero/pull/8752 it works fine, tested
a bit late to the party, but finally: https://github.com/ph4r05/monero/releases/tag/v0.18.3.1-dev-tests-u20.04-01 @mmilata can you pls try this one? Thanks!
Thank you! I've updated the URL in #3463 and verified it works with real device, however gen_trezor_wallet_passphrase
is failing (both device and emulator). Any idea what to look for?
2023-12-14 17:01:26.307 E cache_key != m_cache_key. THROW EXCEPTION: error::invalid_password
2023-12-14 17:01:26.307 W /root/monero/src/wallet/wallet2.cpp:4371:N5tools5error16invalid_passwordE: invalid password
2023-12-14 17:01:26.308 E gen_trezor_wallet_passphrase generation failed: what=invalid password
2023-12-14 17:01:26.308 E #TEST# Failed gen_trezor_wallet_passphrase
Interesting! I will take a look. There is probably a change causing this. When I created the binary, it worked with master emulator.
I found the problem - just a technicality with setting up a test wallet in the test code, nothing affecting production code. I will release new testing binaries soon / in a few days
Works now, thank you! Though on our self-hosted CI runners I'm getting Illegal instruction (core dumped)
, any clue? I'll try to build the test from source to see if it changes anything.
Works now, thank you! Though on our self-hosted CI runners I'm getting
Illegal instruction (core dumped)
, any clue? I'll try to build the test from source to see if it changes anything.
Sounds like SSE3, SSE4 or similar was available on the host that compiled the binary and compiler was told to use it. And this is not available on the runner.
We need to fix the compilation via providing the correct march/mcpu flags.
Thank you, but still the same result. Perhaps it's something to do with our runners, I'll investigate further.
Hmm thats weird. But it can be anything, e.g., some new optimizations in monero build, instructions sets as @prusnak indicated. Can you pls provide cat /proc/cpuinfo
? Builder machine
cat /proc/cpuinfo
processor : 0
vendor_id : GenuineIntel
cpu family : 6
model : 63
model name : Intel(R) Xeon(R) CPU E5-2620 v3 @ 2.40GHz
stepping : 2
microcode : 0x1
cpu MHz : 2399.996
cache size : 16384 KB
physical id : 0
siblings : 4
core id : 0
cpu cores : 4
apicid : 0
initial apicid : 0
fpu : yes
fpu_exception : yes
cpuid level : 13
wp : yes
flags : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht syscall nx pdpe1gb rdtscp lm constant_tsc arch_perfmon rep_good nopl xtopology cpuid tsc_known_freq pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm abm cpuid_fault invpcid_single pti ssbd ibrs ibpb fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid xsaveopt arat umip
bugs : cpu_meltdown spectre_v1 spectre_v2 spec_store_bypass l1tf mds swapgs itlb_multihit
bogomips : 4799.99
clflush size : 64
cache_alignment : 64
address sizes : 40 bits physical, 48 bits virtual
power management:
Also, can you pls provide more info on the runners?
Btw are you running tests with MONERO_RANDOMX_UMASK=8
env var? It was related to app crashes (not using it caused crash)
We don't. Will try that.
any update @mmilata ?
nope, sorry, got stuck working on something else
@ph4r05 Hey, sorry, but we won't touch this anytime soon as we have higher priorities right now…
Since eacd15383b6801fc09db9d51b31bdfb83da09097 hardware tests in CI are failing. Seems to be locally reproducible:
The test binary can see the device through Trezor Bridge after I start it but it doesn't support debuglink. Emulator seems to also work fine. Any clues @ph4r05?