[dpdk-dev] [PATCH v4 1/3] ethdev: modify rte API for single flow dump
Haifei Luo
haifeil at nvidia.com
Wed Apr 14 11:07:24 CEST 2021
Hi Thomas,
I will remove it from prog_guide/rte_flow.rst. Is it okay ? Thank you.
-----Original Message-----
From: Thomas Monjalon <thomas at monjalon.net>
Sent: Wednesday, April 14, 2021 5:01 PM
To: Ori Kam <orika at nvidia.com>
Cc: dev at dpdk.org; Haifei Luo <haifeil at nvidia.com>; Slava Ovsiienko <viacheslavo at nvidia.com>; Raslan Darawsheh <rasland at nvidia.com>; Xueming(Steven) Li <xuemingl at nvidia.com>; Haifei Luo <haifeil at nvidia.com>; ajit.khaparde at broadcom.com; Xiaoyun Li <xiaoyun.li at intel.com>; Matan Azrad <matan at nvidia.com>; Shahaf Shuler <shahafs at nvidia.com>; Jerin Jacob <jerinj at marvell.com>; Nithin Dabilpuram <ndabilpuram at marvell.com>; Kiran Kumar K <kirankumark at marvell.com>; Ferruh Yigit <ferruh.yigit at intel.com>; Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru>
Subject: Re: [PATCH v4 1/3] ethdev: modify rte API for single flow dump
External email: Use caution opening links or attachments
14/04/2021 10:41, Haifei Luo:
> Previous implementations support dump all the flows. Add new arg
> rte_flow in rte_flow_dev_dump to dump one flow.
>
> Signed-off-by: Haifei Luo <haifeil at nvidia.com>
> Acked-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
> Acked-by: Ori Kam <orika at nvidia.com>
[...]
> +Dump
> +~~~~~
> +
> +Dump information for all or one flows.
> +
> +.. code-block:: c
> +
> + int
> + rte_flow_dev_dump(uint16_t port_id, struct rte_flow *flow,
> + FILE *file,
> + struct rte_flow_error *error);
> +
> +Arguments:
> +
> +- ``port_id``: port identifier of Ethernet device.
> +- ``flow``: flow rule handle to dump. NULL to dump all.
> +- ``file``: a pointer to a file for output
> +- ``error``: perform verbose error reporting if not NULL. PMDs
> +initialize
> + this structure in case of error only.
> +
> +Return values:
> +
> +- 0 on success, a negative errno value otherwise and ``rte_errno`` is set.
I really don't understand why we continue duplicating rte_flow doxygen in the guide. Please stop.
Do one good API description in doxygen,
and describe only the big picture and how-to in the guide.
More information about the dev
mailing list