Commit Graph

2 Commits

Author SHA1 Message Date
taptap 83f129314e Fix: Multiple benchmark script bugs
- benchmark_network.sh:
  - Removed info() call in run_fastsync that contaminated output parsing
  - Reduced excessive wait times (was up to 66s for WAN)
  - More reasonable dynamic wait based on latency
- compare_rsync.sh:
  - Fixed hardcoded -n 4 connections, now accepts connection count parameter
  - Replaced slow md5sum verification with fast file count+size check
  - Added latency-aware wait times for 1 connection
- benchmark.sh:
  - Added sync after client to ensure disk writes complete
  - Added EXIT trap for tc netem cleanup and process cleanup
  - Removed duplicate manual cleanup
- benchmark_comprehensive.sh:
  - Added latency-aware wait times for 1 connection in run_fastsync
  - Added documentation note that rsync/cp bypass tc netem (use benchmark_network.sh for fair TCP comparison)

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-06-24 16:35:31 +02:00
Theo Tappe bacb61f6ec Fix: Add missing checksum field to file_meta_t and file_task_t structs
The file_meta_t and file_task_t structs were missing the checksum field
that was being used in both client and server code, causing a compilation
error. Added uint64_t checksum to both structs and updated wq_push
declaration to match implementation.

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-06-24 12:40:23 +02:00