09/10/2024 14:08, Brandes, Shai: > From: Thomas Monjalon <tho...@monjalon.net> > > > > The function abort() should be forbidden for drivers and libs. > > It is not used in ENA, but the rule is enfocred by removing the macro. > > > > Signed-off-by: Thomas Monjalon <tho...@monjalon.net> > > --- > > drivers/net/ena/base/ena_plat_dpdk.h | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/drivers/net/ena/base/ena_plat_dpdk.h > > b/drivers/net/ena/base/ena_plat_dpdk.h > > index 21b96113c7..a41a4e4506 100644 > > --- a/drivers/net/ena/base/ena_plat_dpdk.h > > +++ b/drivers/net/ena/base/ena_plat_dpdk.h > > @@ -63,8 +63,6 @@ typedef uint64_t dma_addr_t; > > > > #define ENA_CDESC_RING_SIZE_ALIGNMENT (1 << 12) /* 4K */ > > > > -#define ENA_ABORT() abort() > > - > > #define ENA_MSLEEP(x) rte_delay_us_sleep(x * 1000) #define > > ENA_USLEEP(x) rte_delay_us_sleep(x) #define ENA_UDELAY(x) > > rte_delay_us_block(x) > > -- > > 2.46.0 > [Brandes, Shai] Thanks Thomas, we verified this in our CI. > Acked-by: Shai Brandes <shaib...@amazon.com>
Thanks for the quick reply. Applied
