rust-brotli icon indicating copy to clipboard operation
rust-brotli copied to clipboard

Few tests fail with OOM on 32bit

Open igor-raits opened this issue 5 years ago • 3 comments

failures:
---- enc::test::test_roundtrip_compressed_repeated stdout ----
thread 'enc::test::test_roundtrip_compressed_repeated' panicked at 'OOM', /usr/share/cargo/registry/alloc-no-stdlib-2.0.1/src/stack_allocator.rs:52:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
---- enc::test::test_roundtrip_ukkonooa stdout ----
thread 'enc::test::test_roundtrip_ukkonooa' panicked at 'OOM', /usr/share/cargo/registry/alloc-no-stdlib-2.0.1/src/stack_allocator.rs:52:13
failures:
    enc::test::test_roundtrip_compressed_repeated
    enc::test::test_roundtrip_ukkonooa

igor-raits avatar Dec 14 '19 18:12 igor-raits

I see taht on armv7hl there are more tests which OOM:

failures:
---- enc::test::test_roundtrip_monkey stdout ----
thread 'enc::test::test_roundtrip_monkey' panicked at 'OOM', /usr/share/cargo/registry/alloc-no-stdlib-2.0.1/src/stack_allocator.rs:52:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
---- enc::test::test_roundtrip_quickfox stdout ----
thread 'enc::test::test_roundtrip_quickfox' panicked at 'OOM', /usr/share/cargo/registry/alloc-no-stdlib-2.0.1/src/stack_allocator.rs:52:13
---- enc::test::test_roundtrip_quickfox_repeated stdout ----
thread 'enc::test::test_roundtrip_quickfox_repeated' panicked at 'OOM', /usr/share/cargo/registry/alloc-no-stdlib-2.0.1/src/stack_allocator.rs:52:13
failures:
    enc::test::test_roundtrip_monkey
    enc::test::test_roundtrip_quickfox
    enc::test::test_roundtrip_quickfox_repeated

igor-raits avatar Dec 14 '19 19:12 igor-raits

Interesting--all these tests allocate the working-set data on the stack. Is it possible to make them to skip on arm? I doubt you'd choose a stack allocator on Arm, but it was a nice cross platform way to test the nostd 3rd party allocators.

danielrh avatar Dec 16 '19 18:12 danielrh

These test errors also affects the debian builds: https://ci.debian.net/data/autopkgtest/testing/armel/r/rust-brotli/38555788/log.gz

alexanderkjall avatar Oct 20 '23 19:10 alexanderkjall