Hello, On Fri, Feb 13, 2015 at 9:19 AM, Chen Jing D(Mark) <jing.d.chen at intel.com> wrote:
[snip] +static int > +fm10k_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_id, > + uint16_t nb_desc, unsigned int socket_id, > + const struct rte_eth_rxconf *conf, struct rte_mempool *mp) > +{ > + struct fm10k_hw *hw = > FM10K_DEV_PRIVATE_TO_HW(dev->data->dev_private); > + struct fm10k_rx_queue *q; > + const struct rte_memzone *mz; > + > + PMD_INIT_FUNC_TRACE(); > + > + /* make sure the mempool element size can account for alignment. > Use > + * RTE_LOG directly to make sure this error is seen. */ > Comment is not valid anymore since you call PMD_INIT_LOG. > + if (!mempool_element_size_valid(mp)) { > + PMD_INIT_LOG(ERR, "Error : Mempool element size is too > small"); > + return (-EINVAL); > + } > + > -- David Marchand