[PATCH v8 0/7] Stage-Ordered API and other extensions for ring library

Thomas Monjalon thomas at monjalon.net
Fri Nov 8 22:56:12 CET 2024


> Konstantin Ananyev (6):
>   test/ring: fix failure with custom number of lcores
>   ring: common functions for 'move head' ops
>   ring: make copying functions generic
>   ring/soring: introduce Staged Ordered Ring
>   app/test: add unit tests for soring API
>   test: add stress test suite

This looks too risky to change something in the ring library
at this stage of the release cycle.
Sorry we got too many issues with other features merged recently
to allow more big changes in 24.11.

For next version, please could you fixe those typos in comments?
	avaialble -> available
	calcuated -> calculated
	direclty -> directly
	elemnts -> elements
	errorneous -> erroneous
	paramer -> parameter
	relase -> release
	strctures -> structures

Instead of "RTE soring", please write "DPDK soring",
because RTE is nothing more than a namespace prefix.

One more thing:
Warning in app/test/test_soring_stress_impl.h:
Using __atomic_xxx/__ATOMIC_XXX built-ins, prefer rte_atomic_xxx/rte_memory_order_xxx

Thanks




More information about the dev mailing list