Message ID | 20210802104218.15369-1-getelson@nvidia.com (mailing list archive) |
---|---|
State | New |
Delegated to: | Thomas Monjalon |
Headers | show |
Series | net: announce changes in TCP header | expand |
Context | Check | Description |
---|---|---|
ci/checkpatch | success | coding style OK |
ci/github-robot | success | github build: passed |
ci/iol-intel-Functional | success | Functional Testing PASS |
ci/Intel-compilation | success | Compilation OK |
ci/iol-intel-Performance | success | Performance Testing PASS |
ci/intel-Testing | success | Testing PASS |
ci/iol-mellanox-Performance | success | Performance Testing PASS |
ci/iol-testing | success | Testing PASS |
ci/iol-abi-testing | success | Testing PASS |
ci/iol-spell-check-testing | warning | Testing issues |
ci/iol-aarch64-unit-testing-testing | success | Testing PASS |
ci/iol-x86_64-unit-testing-testing | success | Testing PASS |
ci/iol-aarch64-compile-testing | success | Testing PASS |
ci/iol-x86_64-compile-testing | success | Testing PASS |
ci/iol-aarch64-unit-testing | success | Testing PASS |
ci/iol-x86_64-unit-testing | fail | Testing issues |
> -----Original Message----- > From: dev <dev-bounces@dpdk.org> On Behalf Of Gregory Etelson > Sent: Monday, August 2, 2021 13:42 > To: dev@dpdk.org > Cc: Gregory Etelson <getelson@nvidia.com>; Matan Azrad > <matan@nvidia.com>; Raslan Darawsheh <rasland@nvidia.com>; Ori Kam > <orika@nvidia.com>; Ray Kinsella <mdr@ashroe.eu> > Subject: [dpdk-dev] [PATCH] net: announce changes in TCP header > > Announce change to add a union that will provide byte and bitfield access to > TCP flags. > > Signed-off-by: Gregory Etelson <getelson@nvidia.com> Acked-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>
Nobody rejected this change but there is not enough ack to make it an accepted announce of change. 02/08/2021 12:42, Gregory Etelson: > Announce change to add a union that will provide byte and bitfield > access to TCP flags. > > Signed-off-by: Gregory Etelson <getelson@nvidia.com> > --- > +* net: structure ``rte_tcp_hdr`` will have a union that will provide > + byte access to existing ``tcp_flags`` and add a bitfield for TCP flags.
The patch lacks motivation. Why is the change required? Why is it useful? On August 5, 2021 18:28:31 Thomas Monjalon <thomas@monjalon.net> wrote: > Nobody rejected this change but there is not enough ack > to make it an accepted announce of change. > > 02/08/2021 12:42, Gregory Etelson: >> Announce change to add a union that will provide byte and bitfield >> access to TCP flags. >> >> Signed-off-by: Gregory Etelson <getelson@nvidia.com> >> --- >> +* net: structure ``rte_tcp_hdr`` will have a union that will provide >> + byte access to existing ``tcp_flags`` and add a bitfield for TCP flags.
Hello, The TCP flags bitfield patch complements similar changes proposed to IPv4 header in net: announce changes in IPv4 header - Patchwork (dpdk.org)<https://patchwork.dpdk.org/project/dpdk/patch/20210802103834.14263-1-getelson@nvidia.com/> Regards, Gregory From: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru> Sent: Friday, August 6, 2021 12:03 To: NBU-Contact-Thomas Monjalon <thomas@monjalon.net>; Gregory Etelson <getelson@nvidia.com> Cc: dev@dpdk.org; Matan Azrad <matan@nvidia.com>; Raslan Darawsheh <rasland@nvidia.com>; Ori Kam <orika@nvidia.com>; Ray Kinsella <mdr@ashroe.eu>; ajit.khaparde@broadcom.com; jerinj@marvell.com; akhil.goyal@nxp.com; cristian.dumitrescu@intel.com; ferruh.yigit@intel.com; konstantin.ananyev@intel.com; olivier.matz@6wind.com; maxime.coquelin@redhat.com; david.marchand@redhat.com Subject: Re: [dpdk-dev] [PATCH] net: announce changes in TCP header External email: Use caution opening links or attachments The patch lacks motivation. Why is the change required? Why is it useful? On August 5, 2021 18:28:31 Thomas Monjalon <thomas@monjalon.net<mailto:thomas@monjalon.net>> wrote: Nobody rejected this change but there is not enough ack to make it an accepted announce of change. 02/08/2021 12:42, Gregory Etelson: Announce change to add a union that will provide byte and bitfield access to TCP flags. Signed-off-by: Gregory Etelson <getelson@nvidia.com<mailto:getelson@nvidia.com>> --- +* net: structure ``rte_tcp_hdr`` will have a union that will provide + byte access to existing ``tcp_flags`` and add a bitfield for TCP flags.
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index d9c0e65921..a78a68108c 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/guides/rel_notes/deprecation.rst @@ -158,3 +158,6 @@ Deprecation Notices * security: The functions ``rte_security_set_pkt_metadata`` and ``rte_security_get_userdata`` will be made inline functions and additional flags will be added in structure ``rte_security_ctx`` in DPDK 21.11. + +* net: structure ``rte_tcp_hdr`` will have a union that will provide + byte access to existing ``tcp_flags`` and add a bitfield for TCP flags.
Announce change to add a union that will provide byte and bitfield access to TCP flags. Signed-off-by: Gregory Etelson <getelson@nvidia.com> --- doc/guides/rel_notes/deprecation.rst | 3 +++ 1 file changed, 3 insertions(+)