Remove redundant basic multithreaded benchmark

[Amaury Chamayou (achamayou)]

Summary

• remove the Piccolo-backed pi_basic_mt performance test
• document that pi_basic_blocking_locust already exercises every configured worker thread

Rationale

The benchmark workflows configure WORKER_THREADS=2, which is passed to pi_basic_blocking_locust through the common network test arguments. Its 320 concurrent Locust connections already cover the multi-threaded basic workload, so adding another blocking Locust replacement would duplicate coverage.

Validation

• configured CMake with BUILD_TESTS=ON and WORKER_THREADS=2
• confirmed the generated CTest registration includes pi_basic_blocking_locust with --worker-threads 2 and no pi_basic_mt
• formatted CMakeLists.txt with gersemi 0.27.0
• ran diff and ASCII integrity checks

[Copilot]

Pull request overview

This PR simplifies CCF’s performance/benchmark test registration by removing the redundant Piccolo-backed pi_basic_mt test and documenting that the existing Locust-based basic benchmark already exercises multi-threaded behavior under the benchmark workflow configuration.

Changes:

• Remove the pi_basic_mt Piccolo performance test from the CMake test registrations.
• Update the pi_basic_blocking_locust comment to describe its relationship to multi-threaded coverage.

Custom instructions used

• None (no .github/copilot-instructions.md / .github/instructions/* files were loaded directly during this review)

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.