Configurable XDP Ring Sizes and Enhanced Concurrent Buffer Support on Windows#1621
Configurable XDP Ring Sizes and Enhanced Concurrent Buffer Support on Windows#1621mvp-2003 wants to merge 3 commits intomicrosoft:con-xdp-1437from mvp-2003:con-xdp-1437
Conversation
…nce metrics - Added new configuration parameters for TX and RX fill and completion ring sizes. - Introduced enhanced concurrency options for XDP operations, including batch sizes and adaptive batching. - Implemented a performance testing example to demonstrate the benefits of the new XDP backend. - Created a batch processing utility for efficient transmission of packets. - Developed functional and unit tests to validate backward compatibility and new features. - Updated configuration parsing to support new parameters and ensure proper validation.
- Added `DynamicRingManager` to manage dynamic resizing of RX/TX rings based on workload and system metrics. - Introduced `DynamicSizingConfig` and `DynamicRingSizer` for configuring and calculating optimal ring sizes. - Implemented metrics collection through `MetricsCollector` to gather system and workload metrics. - Created `RingResizeCallback` trait for handling resize events with a default logging implementation. - Added functionality to evaluate and apply sizing recommendations based on collected metrics. - Enhanced error handling and validation for configuration parameters. - Established a background task for periodic evaluation of ring sizes. - Included detailed logging for resizing operations and metrics collection.
|
(Documentation)demikernel-issue-1437-complete-analysis.txt @anandbonde I have created this new branch and re-raising the PR along with the documentation. Please let me know if I missed anything or something has to be done from my side. Thank you! |
|
@anandbonde Can you please have a look at this PR and let me know about any issues you see? |
@mvp-2003, could you please write a brief description of the issue in the PR and also name the PR appropriately (1 liner explanation of the actual issue that you are fixing in this PR) - you can refer to the issue number in the description. Also, I've added other reviewers (@iyzhang and @kyleholohan) to provide their expertise in this area. |
|
Sure on it. |
|
Issue Summary: Goal Achieved: |
Fixes issue #1437