Skip to content

Fix: fix eval hang#41

Merged
GregoryLi360 merged 4 commits into
mainfrom
fix/fix-eval-hang
Jul 9, 2026
Merged

Fix: fix eval hang#41
GregoryLi360 merged 4 commits into
mainfrom
fix/fix-eval-hang

Conversation

@GregoryLi360

@GregoryLi360 GregoryLi360 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

fix eval <file> computed correctly but hung on shutdown (closes #40, regression after #39). Drop ran before kernel::shutdown(), which called the Close path for the first time which had 2 bugs:

  • Host never acked Close, but caller expects it and blocks waiting for it
  • After the host sends the ack response, arca panics with Err(Closed) because Reader::read reported the pipe being closed before reading the ack

Also updated README just fix eval addblob.fix

@GregoryLi360 GregoryLi360 merged commit 3235a30 into main Jul 9, 2026
6 checks passed
@GregoryLi360 GregoryLi360 deleted the fix/fix-eval-hang branch July 9, 2026 20:28
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.

fix eval <file> is not shutting down properly

1 participant