Hi Andrew,
> -----Original Message-----
> From: Andrew Rybchenko <[email protected]>
> Sent: Tuesday, October 4, 2022 3:53 PM
> To: Wang, YuanX <[email protected]>; [email protected]; Thomas
> Monjalon <[email protected]>; Ferruh Yigit <[email protected]>;
> Ray Kinsella <[email protected]>
> Cc: [email protected]; Li, Xiaoyun <[email protected]>; Singh, Aman
> Deep <[email protected]>; Zhang, Yuying
> <[email protected]>; Zhang, Qi Z <[email protected]>; Yang,
> Qiming <[email protected]>; [email protected];
> [email protected]; [email protected]; Ding, Xuan
> <[email protected]>; [email protected]; Tang, Yaqi
> <[email protected]>; Wenxuan Wu <[email protected]>
> Subject: Re: [PATCH v7 1/4] ethdev: introduce protocol header API
>
> On 10/4/22 05:21, Wang, YuanX wrote:
> > Hi Andrew,
> >
> >> -----Original Message-----
> >> From: Andrew Rybchenko <[email protected]>
> >> Sent: Monday, October 3, 2022 3:04 PM
> >> To: Wang, YuanX <[email protected]>; [email protected]; Thomas
> Monjalon
> >> <[email protected]>; Ferruh Yigit <[email protected]>; Ray
> >> Kinsella <[email protected]>
> >> Cc: [email protected]; Li, Xiaoyun <[email protected]>;
> >> Singh, Aman Deep <[email protected]>; Zhang, Yuying
> >> <[email protected]>; Zhang, Qi Z <[email protected]>; Yang,
> >> Qiming <[email protected]>; [email protected];
> >> [email protected]; [email protected]; Ding, Xuan
> >> <[email protected]>; [email protected]; Tang, Yaqi
> >> <[email protected]>; Wenxuan Wu <[email protected]>
> >> Subject: Re: [PATCH v7 1/4] ethdev: introduce protocol header API
> >>
> >> On 10/2/22 00:05, Yuan Wang wrote:
> >>> Add a new ethdev API to retrieve supported protocol headers of a
> >>> PMD, which helps to configure protocol header based buffer split.
> >>>
> >>> Signed-off-by: Yuan Wang <[email protected]>
> >>> Signed-off-by: Xuan Ding <[email protected]>
> >>> Signed-off-by: Wenxuan Wu <[email protected]>
> >>> Reviewed-by: Andrew Rybchenko <[email protected]>
> >>> ---
> >>> doc/guides/rel_notes/release_22_11.rst | 5 ++++
> >>> lib/ethdev/ethdev_driver.h | 15 ++++++++++++
> >>> lib/ethdev/rte_ethdev.c | 33 ++++++++++++++++++++++++++
> >>> lib/ethdev/rte_ethdev.h | 30 +++++++++++++++++++++++
> >>> lib/ethdev/version.map | 3 +++
> >>> 5 files changed, 86 insertions(+)
> >>>
> >>> diff --git a/doc/guides/rel_notes/release_22_11.rst
> >>> b/doc/guides/rel_notes/release_22_11.rst
> >>> index 0231959874..6a7474a3d6 100644
> >>> --- a/doc/guides/rel_notes/release_22_11.rst
> >>> +++ b/doc/guides/rel_notes/release_22_11.rst
> >>> @@ -96,6 +96,11 @@ New Features
> >>> * Added ``rte_event_eth_tx_adapter_queue_stop`` to stop the Tx
> >> Adapter
> >>> from enqueueing any packets to the Tx queue.
> >>>
> >>> +* **Added new ethdev API for PMD to get buffer split supported
> >>> +protocol types.**
> >>> +
> >>> + * Added ``rte_eth_buffer_split_get_supported_hdr_ptypes()``, to
> >>> + get
> >> supported
> >>> + header protocols of a PMD to split.
> >>> +
> >>
> >> ethdev features should be grouped together in release notes.
> >> I'll fix it on applying if a new version is not required.
> >
> > We will send a new version. For the doc changes, I don't understand your
> point very well.
> > Since will be no new changes to the code within this patch, could you help
> to adjust the doc?
> > Thanks very much.
>
> Please, read a comment just after 'New Features' section start.
> Hopefully it will make my note clearer.
> Anyway, don't worry about it a lot. I can easily fix it on applying.
Is it written like the following, if it is not correct please help to fix.
* **Added protocol header based buffer split.**
* Added ``rte_eth_buffer_split_get_supported_hdr_ptypes()``, to get supported
header protocols of a PMD to split.
* Ethdev: The ``reserved`` field in the ``rte_eth_rxseg_split`` structure is
replaced with ``proto_hdr`` to support protocol header based buffer split.
User can choose length or protocol header to configure buffer split
according to NIC's capability.
Thanks,
Yuan
[snip]
