diff mbox series

[v2,1/2] mbuf: remove unneeded atomic header include

Message ID 20210204100520.15414-1-david.marchand@redhat.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series [v2,1/2] mbuf: remove unneeded atomic header include |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-testing warning Testing issues

Commit Message

David Marchand Feb. 4, 2021, 10:05 a.m. UTC 
  There is no need for the direct inclusion of the generic/ header [1]
now that we don't use the rte_atomic API anymore.

1: https://git.dpdk.org/dpdk/commit/?id=3eb860b08eb7

Fixes: e41d27a68df6 ("mbuf: remove atomic reference counters")
Cc: stable@dpdk.org

Signed-off-by: David Marchand <david.marchand@redhat.com>
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
---
 lib/librte_mbuf/rte_mbuf_core.h | 1 -
 1 file changed, 1 deletion(-)

Comments

Andrew Rybchenko Feb. 4, 2021, 1:35 p.m. UTC | #1 
On 2/4/21 1:05 PM, David Marchand wrote:
> There is no need for the direct inclusion of the generic/ header [1]
> now that we don't use the rte_atomic API anymore.
> 
> 1: https://git.dpdk.org/dpdk/commit/?id=3eb860b08eb7
> 
> Fixes: e41d27a68df6 ("mbuf: remove atomic reference counters")
> Cc: stable@dpdk.org
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>

Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
diff mbox series

Patch

diff --git a/lib/librte_mbuf/rte_mbuf_core.h b/lib/librte_mbuf/rte_mbuf_core.h
index a85cabdd18..9d1609336a 100644
--- a/lib/librte_mbuf/rte_mbuf_core.h
+++ b/lib/librte_mbuf/rte_mbuf_core.h
@@ -20,7 +20,6 @@ 
 
 #include <rte_compat.h>
 #include <rte_byteorder.h>
-#include <generic/rte_atomic.h>
 
 #ifdef __cplusplus
 extern "C" {