[dpdk-dev] [PATCH 1/3] ether: Update field name and description for queue start
Stephen Hemminger
stephen at networkplumber.org
Tue Jul 22 19:43:21 CEST 2014
On Tue, 22 Jul 2014 15:47:30 +0800
Ouyang Changchun <changchun.ouyang at intel.com> wrote:
> + /**< If rx_enable_queue is true, rte_eth_dev_rx_queue_start must be
> + invocated after rte_eth_dev_start's invocation to start RX for
> + one queue, and rte_eth_dev_rx_queue_start instead of
> + rte_eth_dev_start is responsible for allocating mbuf from
> + mempool and setup the DMA physical address. It is useful in
> + such scenario: buffer address is not available at the point of
> + rte_eth_dev_start's invocation but available later, e.g. in
> + VHOST zero copy case, the buffer address to be setup DMA
> + address is available only after one VM startup. */
Good documentation about semantics is really valuable as long as it
kept up to date. In this case, the comment stands out which is not
a good thing.
An explanation this long belongs in documentation not in code.
Alternatively, if it is this hard to explain maybe it isn't the right design :-)
More information about the dev
mailing list