Skip to content

Fix CI: update whir to fix batch_size assertion on Linux aarch64#309

Open
Bisht13 wants to merge 1 commit intomainfrom
px/fix-whir-batch-size
Open

Fix CI: update whir to fix batch_size assertion on Linux aarch64#309
Bisht13 wants to merge 1 commit intomainfrom
px/fix-whir-batch-size

Conversation

@Bisht13
Copy link
Collaborator

@Bisht13 Bisht13 commented Feb 26, 2026

Summary

Root Cause

small_sha and complete_age_check tests panic on CI (ubuntu-24.04-arm, Linux aarch64) with:

assertion failed: batch_size >= 1
at whir/src/protocols/matrix_commit.rs:355

On Linux aarch64, workload_size falls through to the 32KB default. With starting_log_inv_rate = 2 (4x codeword expansion), message_size exceeds target = 4096, making target / message_size = 0batch_size = 0.

The whir fix (1) adds .max(1) before next_multiple_of and (2) assigns Linux aarch64 a 64KB cache size matching Graviton/Ampere L1d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant