On Sat, Sep 18, 2021 at 10:34 AM Yunjian Wang <wangyunj...@huawei.com> wrote: > > This series for deleting HW rings when releasing queues for > igb, ixgbe, i40e, ice & em drivers. > > --- > v2: > * Update commit log > > Yunjian Wang (4): > net/e1000: delete HW rings when releasing queues > net/ice: delete HW rings when releasing queues > net/i40e: delete HW rings when releasing queues > net/ixgbe: delete HW rings when releasing queues > > drivers/net/e1000/em_rxtx.c | 8 ++++++-- > drivers/net/e1000/igb_rxtx.c | 9 +++++++-- > drivers/net/i40e/i40e_fdir.c | 3 --- > drivers/net/i40e/i40e_rxtx.c | 8 ++++++-- > drivers/net/i40e/i40e_rxtx.h | 2 ++ > drivers/net/ice/ice_rxtx.c | 6 ++++-- > drivers/net/ice/ice_rxtx.h | 2 ++ > drivers/net/ixgbe/ixgbe_rxtx.c | 6 ++++-- > drivers/net/ixgbe/ixgbe_rxtx.h | 2 ++ > 9 files changed, 33 insertions(+), 13 deletions(-) >
- In net/ice (at least), the fdir rxq/txq memzones can be aligned on the same scheme. Looking at the remaining drivers (net/cnxk, net/cxgbe and net/octeontx2), we could apply the same principle of keeping a reference to mz in internal driver structures. Afterwards, I see no need to keep rte_eth_dma_zone_free() (it's internal, so we can remove and it's easy to re-add if a need arises). Wdyt? - Is this worth backporting to stable branches? -- David Marchand
