On 2015-02-25 04:24, Ouyang, Changchun wrote: > > >> -----Original Message----- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pawel Wodkowski >> Sent: Thursday, February 19, 2015 11:55 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH v4 3/7] pmd: igb/ixgbe split nb_q_per_pool to rx >> and tx nb_q_per_pool >> [...] >> >> /* check valid queue number */ >> - if ((nb_rx_q > RTE_ETH_DEV_SRIOV(dev).nb_q_per_pool) || >> - (nb_tx_q > RTE_ETH_DEV_SRIOV(dev).nb_q_per_pool)) { >> + if ((nb_rx_q > RTE_ETH_DEV_SRIOV(dev).nb_tx_q_per_pool) > > Here, how about use nb_rx_q_per_pool to replace nb_tx_q_per_pool ? > so it will be more clear to check rx queue number.
Yes, this should be nb_rx_q_per_pool. I missed this, because in next patch I moved this and corrected "on the fly" :). I will correct this in next version. -- Pawel
