> -----Original Message----- > From: wangyunjian <[email protected]> > Sent: Saturday, September 18, 2021 11:11 > To: Wang, Haiyue <[email protected]>; [email protected] > Cc: Xing, Beilei <[email protected]>; Yang, Qiming > <[email protected]>; Zhang, Qi Z > <[email protected]>; dingxiaoxiong <[email protected]> > Subject: RE: [dpdk-dev] [PATCH 4/4] net/ixgbe: delete HW rings when releasing > queues > > > -----Original Message----- > > From: Wang, Haiyue [mailto:[email protected]] > > Sent: Friday, September 17, 2021 11:47 PM > > To: wangyunjian <[email protected]>; [email protected] > > Cc: Xing, Beilei <[email protected]>; Yang, Qiming > > <[email protected]>; Zhang, Qi Z <[email protected]>; dingxiaoxiong > > <[email protected]> > > Subject: RE: [dpdk-dev] [PATCH 4/4] net/ixgbe: delete HW rings when > > releasing > > queues > > > > > -----Original Message----- > > > From: Yunjian Wang <[email protected]> > > > Sent: Friday, September 17, 2021 19:25 > > > To: [email protected] > > > Cc: Wang, Haiyue <[email protected]>; Xing, Beilei > > > <[email protected]>; Yang, Qiming <[email protected]>; Zhang, > > > Qi Z <[email protected]>; [email protected]; Yunjian Wang > > > <[email protected]> > > > Subject: [dpdk-dev] [PATCH 4/4] net/ixgbe: delete HW rings when > > > releasing queues > > > > > > Normally when closing the device the queue memzone should be freed. > > > But the memzone will be not freed, when device setup ops like: > > > - rte_eth_bond_slave_remove > > > - rte_eth_dev_internal_reset > > > - eth_dev_rx_queue_config > > > - dev_rx_queue_release > > > - dev_close > > > - dev_free_queues > > > > > > > What's these ops ? function call flow ? And where these names from ? > > The above is simplified, and the original is: > rte_eth_bond_slave_remove > -->__eth_bond_slave_remove_lock_free > ---->slave_remove > ------> rte_eth_dev_internal_reset > --------> rte_eth_dev_rx_queue_config > ----------> eth_dev_rx_queue_config > ------------>i40e_dev_rx_queue_release > rte_eth_dev_close > -->i40e_dev_close > ---->i40e_dev_free_queues > ------>i40e_dev_rx_queue_release(not been called due to nb_rx_queues and > nb_tx_queues are 0) >
Yeah, this is better. The patch LGTM, and make sense. Please update it as new commit message. Thanks! > > > > > In order to free the memzone, we can release the memzone > > > when releasing queues. > > > > > > Signed-off-by: Yunjian Wang <[email protected]> > > > --- > > > > > > > -- > > > 2.23.0
