Skip to content

[feat] hical: add echo-ws-pipeline and echo-ws-limited WebSocket tests - #1483

Open
Hical61 wants to merge 2 commits into
MDA2AV:mainfrom
Hical61:feat/hical-echo-ws-pipeline
Open

Hical61 wants to merge 2 commits into
MDA2AV:mainfrom
Hical61:feat/hical-echo-ws-pipeline

Conversation

@Hical61

@Hical61 Hical61 commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Changes

Add two WebSocket profiles for Hical: echo-ws-pipeline (16-message back-to-back echo throughput) and echo-ws-limited (short-lived connection churn).

Changes to frameworks/hical/meta.json:

  • Add mode: "standard" (framework-standard WS API with default buffer sizes, matching the Standard constraints)
  • Add echo-ws-pipeline and echo-ws-limited to tests
  • enabled stays true (kept since the last re-enable)

Notes

  • All three WS profiles share the same /ws endpoint; no new code is required.
  • Hical's WS core receiveInternal reads the kernel buffer with a single async_read_some, drains multiple frames via a for(;;) loop, and consumes them with memmove — zero extra framework change for the pipelined echo scenario.
  • The handshake stack computes Sec-WebSocket-Accept per connection from the client key per RFC 6455, satisfying the echo-limited constraint against pre-computed 101 responses.
  • A re-benchmark run is needed for these profiles' numbers to appear on the board.

@Hical61 Hical61 changed the title [feat] hical: 报名 echo-ws-pipeline WebSocket 回显测试 [feat] hical: add echo-ws-pipeline WebSocket echo test Sep 18, 2026
@Hical61 Hical61 changed the title [feat] hical: add echo-ws-pipeline WebSocket echo test [feat] hical: add echo-ws-pipeline and echo-ws-limited WebSocket tests Sep 18, 2026
meta.json 补 mode: standard、tests 加 echo-ws-pipeline(16 帧背靠背
回显)。WS 内核 receiveInternal 天然支持单 read 多帧 drain,/ws 端点
已存在,零框架改动,报名后等官方重跑一轮即可出该 profile 数据。
echo-ws-limited 与 echo-ws 共用 /ws 端点,测短连接 churn 下的握手/建连
开销。握手栈逐连接实时算 Sec-WebSocket-Accept,零框架改动。
@MDA2AV
MDA2AV force-pushed the feat/hical-echo-ws-pipeline branch from 030f69b to a64bcd3 Compare September 18, 2026 11:07
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