Skip to content

perf(dpdk): copy BSD packets directly into transmit mbufs - #12

Merged
thweetkomputer merged 1 commit into
mainfrom
perf/dpdk-direct-frame-copy
Oct 12, 2026
Merged

thweetkomputer merged 1 commit into
mainfrom
perf/dpdk-direct-frame-copy

Conversation

@thweetkomputer

Copy link
Copy Markdown
Contributor

Replace the contiguous-frame callback with a synchronous borrowed packet copier. DPDK fills its final mbuf directly, eliminating the intermediate frame copy while retaining separate BSD and host ownership.

Exercise pre-copy and post-copy rejection through real TCP loopback. Verified nine focused CTests, the 3 GiB TCP sequence check, DPDK poll/adaptive TAP smoke with and without loss, and the BSD TAP smoke with loss.

Replace the contiguous-frame callback with a synchronous borrowed packet copier. DPDK fills its final mbuf directly, eliminating the intermediate frame copy while retaining separate BSD and host ownership.

Exercise pre-copy and post-copy rejection through real TCP loopback. Verified nine focused CTests, the 3 GiB TCP sequence check, DPDK poll/adaptive TAP smoke with and without loss, and the BSD TAP smoke with loss.
@coderabbitai

coderabbitai Bot commented Oct 12, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 881b1a80-d83c-477c-826a-5cfc3a25c75f

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@thweetkomputer
thweetkomputer merged commit d13be7b into main Oct 12, 2026
10 checks passed
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