ricardgb commented on PR #19469: URL: https://github.com/apache/nuttx/pull/19469#issuecomment-5067682919
@linguini1 on-hardware before/after from a Raspberry Pi Pico 2 W (RP2350), NuttX 13.0.0-RC0, USB CDC-NCM link (board `eth0` 192.168.7.1 ⇆ host `usb0` 192.168.7.2), system tick 100 Hz. Same board, same config; only `cdcncm_send()`'s else-branch differs. - **BEFORE** = `work_queue(..., MSEC2TICK(CDCNCM_DGRAM_COMBINE_PERIOD))` (1 ms → rounds up to a full 10 ms tick) - **AFTER** = `work_queue(..., 0)` (this PR) **BEFORE — host → board, 20 pings (host µs clock):** ``` 20 packets transmitted, 20 received, 0% packet loss rtt min/avg/max/mdev = 12.668/16.858/21.662/3.009 ms ``` **BEFORE — board → host, 10 pings (board 10 ms tick clock):** ``` rtt min/avg/max/mdev = 20.000/20.000/20.000/0.000 ms # flat 20 ms = 2 ticks ``` **AFTER — host → board, 30 pings:** ``` 30 packets transmitted, 30 received, 0% packet loss rtt min/avg/max/mdev = 2.169/2.282/2.405/0.077 ms ``` **AFTER — board → host, 10 pings:** ``` icmp_seq=0 time=10.0 ms icmp_seq=1..9 time=0.0 ms # sub-tick rtt min/avg/max/mdev = 0.000/1.000/10.000/3.000 ms ``` Round-trip drops from **16.9 ms → 2.28 ms** (host clock); the board-clock figure going from a flat 20 ms to sub-tick shows the tick-quantization the patch removes. Consistent with the 21.7 → 2.8 ms in the commit message. *Disclosure: these tests were run and captured with the help of an AI agent (Claude Code, Anthropic); results are from real hardware and human-reviewed.* -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
