Hi Kevin, On Fri, Jan 12, 2018 at 10:48:43AM +0000, Kevin Laatz wrote: > Increasing the RX/TX default ring size to 1024/1024 to accommodate for > faster NICs. With the increase of number of PPS, a larger RX buffer is > required in order to avoid packet loss. While a ring size of 128 may be > large enough for 1G and possibly 10G NICs, this is not going to scale to > small packet sizes at 25G and 40G line rates. As we are increasing the RX > buffer size to 1024, we also need to increase the TX buffer size to ensure > that the TX side does not become the bottleneck. > > v2 > - fixed typos in commit messages > - fixed typo in Cc email address
I agree with the above and this series contents but would like to comment anyway. Since typical TX/RX bursts are usually somewhere between 16 to 64 packets depending on the application, increasing ring size instead of burst size to keep up with packet rate may mean that software (PMD/application) is too slow on the RX side or hardware is too slow on the TX side (rings always full basically), and this is worked around by introducing latency to absorb packet loss. This is not necessarily a good trade-off. Granted the most appropriate burst/ring/threshold values always depend on the application and underlying hardware, and each vendor is responsible for documenting ideal values for typical applications by providing performance results. My concern is that modifying defaults makes performance comparison with past DPDK releases more difficult for existing automated tests that do not provide ring size and other parameters. There should an impact given that larger rings require more buffers, use more cache, and access more memory in general. -- Adrien Mazarguil 6WIND
