matthew
ae031b8505
Drive::read is now single-shot. Phase 1/2/3 retries + scsi::reset+reopen
removed (~80 lines). recovery=true bumps timeout to 30s; recovery=false
stays at 1.5s. On any failure returns Err(DiscRead) immediately — caller
(Disc::patch outer loop, DiscStream batch halver) handles retries.
Inline reset+reopen WAS the wedge primitive on the LG BU40N. Per prior
post-mortem, every USB/SCSI reset path tested fails to recover the
wedged Initio bridge — the inline retry was pure cost.
SgIoTransport::reset (Linux) trimmed to kernel SG_IO state flush +
ALLOW MEDIUM REMOVAL. SG_SCSI_RESET ioctl + STOP/START UNIT escalation
removed. macOS reset removed (no-op). scsi::reset() top-level family
removed (no callers).
EventKind::BytesRead { bytes, total } now actually emitted from
DiscStream::fill_extents after each successful sector read. Was
declared in 0.13.0, never fired. Drives autorip per-device progress
in direct mode.
EventKind::Retry / SectorRecovered no longer emitted (variants kept
for forward compat). SpeedChange still emitted via Drive::set_speed
public path.
Tests: new tests/integration_progress_and_halt.rs (5 tests). 233 unit
tests + 5 integration green.
21 lines
349 B
Makefile
21 lines
349 B
Makefile
# libfreemkv — local dev helper.
|
|
# Mirrors the cross-crate scripts in freemkv-private/scripts/test-all.sh
|
|
# but scoped to this single crate.
|
|
|
|
.PHONY: test build check ci clean
|
|
|
|
test:
|
|
cargo test --tests
|
|
|
|
build:
|
|
cargo build --release
|
|
|
|
check:
|
|
cargo fmt --check
|
|
cargo clippy --all-targets -- -D warnings
|
|
|
|
ci: check build test
|
|
|
|
clean:
|
|
cargo clean
|