Re: [PATCH v1 2/2] dts: checksum offload test suite

2024-08-19 Thread Dean Marx
> > > > +def send_packet_and_verify( > > Should this not be `send_packets_and_verify(` as the argument is > `packet_list`. > Yeah that would definitely make more sense, I'll change that in the next version. > > > +isL4 = any( > > +VerboseOLFlag.RTE_MBUF_F_RX_L4_CKSUM_GO

Re: [PATCH v1 2/2] dts: checksum offload test suite

2024-08-19 Thread Alex Chapman
Hi Dean, Just though I would point out a few things. On 8/16/24 15:20, Dean Marx wrote: +def send_packet_and_verify( Should this not be `send_packets_and_verify(` as the argument is `packet_list`. +isL4 = any( +VerboseOLFlag.RTE_MBUF_F_RX_L4_CKSUM_GOOD in packet.ol

[PATCH v1 2/2] dts: checksum offload test suite

2024-08-16 Thread Dean Marx
test suite for verifying layer 3/4 checksum offload features on poll mode driver. Depends-on: patch-143033 ("dts: add text parser for testpmd verbose output") Depends-on: patch-142691 ("dts: add send_packets to test suites and rework packet addressing") Signed-off-by: Dean Marx --- dts/tests/Te

[PATCH v1 2/2] dts: checksum offload test suite

2024-08-12 Thread Dean Marx
test suite for verifying layer 3/4 checksum offload features on poll mode driver. Depends-on: patch-142762 ("dts: add text parser for testpmd verbose output") Depends-on: patch-142691 ("dts: add send_packets to test suites and rework packet addressing") Signed-off-by: Dean Marx --- dts/framewor