[v2,1/6] mempool: avoid flags documentation in the next line

Message ID 20211019100845.1632332-2-andrew.rybchenko@oktetlabs.ru (mailing list archive)
State Superseded, archived
Delegated to: David Marchand
Headers
Series mempool: cleanup namespace |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Andrew Rybchenko Oct. 19, 2021, 10:08 a.m. UTC
  Move documentation into a separate line just before define.
Prepare to have a bit longer flag name because of namespace prefix.

Signed-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
---
 lib/mempool/rte_mempool.h | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)
  

Comments

Olivier Matz Oct. 19, 2021, 4:13 p.m. UTC | #1
On Tue, Oct 19, 2021 at 01:08:40PM +0300, Andrew Rybchenko wrote:
> Move documentation into a separate line just before define.
> Prepare to have a bit longer flag name because of namespace prefix.
> 
> Signed-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>

Acked-by: Olivier Matz <olivier.matz@6wind.com>
  

Patch

diff --git a/lib/mempool/rte_mempool.h b/lib/mempool/rte_mempool.h
index 88bcbc51ef..8ef4c8ed1e 100644
--- a/lib/mempool/rte_mempool.h
+++ b/lib/mempool/rte_mempool.h
@@ -250,13 +250,18 @@  struct rte_mempool {
 #endif
 }  __rte_cache_aligned;
 
+/** Spreading among memory channels not required. */
 #define MEMPOOL_F_NO_SPREAD      0x0001
-		/**< Spreading among memory channels not required. */
-#define MEMPOOL_F_NO_CACHE_ALIGN 0x0002 /**< Do not align objs on cache lines.*/
-#define MEMPOOL_F_SP_PUT         0x0004 /**< Default put is "single-producer".*/
-#define MEMPOOL_F_SC_GET         0x0008 /**< Default get is "single-consumer".*/
-#define MEMPOOL_F_POOL_CREATED   0x0010 /**< Internal: pool is created. */
-#define MEMPOOL_F_NO_IOVA_CONTIG 0x0020 /**< Don't need IOVA contiguous objs. */
+/** Do not align objects on cache lines. */
+#define MEMPOOL_F_NO_CACHE_ALIGN 0x0002
+/** Default put is "single-producer". */
+#define MEMPOOL_F_SP_PUT         0x0004
+/** Default get is "single-consumer". */
+#define MEMPOOL_F_SC_GET         0x0008
+/** Internal: pool is created. */
+#define MEMPOOL_F_POOL_CREATED   0x0010
+/** Don't need IOVA contiguous objects. */
+#define MEMPOOL_F_NO_IOVA_CONTIG 0x0020
 
 /**
  * @internal When debug is enabled, store some statistics.