[dpdk-dev] [PATCH v2] doc: propose correction rte_{bsf, fls} inline functions type use
Tyler Retzlaff
roretzla at linux.microsoft.com
Mon Mar 15 20:34:38 CET 2021
The proposal has resulted from request to review [1] the following
functions where there appeared to be inconsistency in return type
or parameter type selections for the following inline functions.
rte_bsf32()
rte_bsf32_safe()
rte_bsf64()
rte_bsf64_safe()
rte_fls_u32()
rte_fls_u64()
rte_log2_u32()
rte_log2_u64()
[1] http://mails.dpdk.org/archives/dev/2021-March/201590.html
Signed-off-by: Tyler Retzlaff <roretzla at linux.microsoft.com>
---
doc/guides/rel_notes/deprecation.rst | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 64629e064..4934f4da4 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -17,6 +17,13 @@ Deprecation Notices
* eal: The function ``rte_eal_remote_launch`` will return new error codes
after read or write error on the pipe, instead of calling ``rte_panic``.
+* eal: Fix inline function return and parameter types for rte_{bsf,fls}
+ inline functions to be consistent.
+ Change ``rte_bsf32_safe`` parameter ``v`` from ``uint64_t`` to ``uint32_t``.
+ Change ``rte_bsf64`` return type to ``uint32_t`` instead of ``int``.
+ Change ``rte_fls_u32`` return type to ``uint32_t`` instead of ``int``.
+ Change ``rte_fls_u64`` return type to ``uint32_t`` instead of ``int``.
+
* rte_atomicNN_xxx: These APIs do not take memory order parameter. This does
not allow for writing optimized code for all the CPU architectures supported
in DPDK. DPDK will adopt C11 atomic operations semantics and provide wrappers
--
2.30.0.vfs.0.2
More information about the dev
mailing list