Skip to content

Commit 60fd92f

Browse files
JannePeltonenMatiasElo
authored andcommitted
validation: pktio: set has_udp and has_sctp flags only when needed
Move setting of has_udp and has_sctp flags in the relevant checksum test prep functions to reduce clutter in the common checksum test function. Signed-off-by: Janne Peltonen <janne.peltonen@nokia.com> Reviewed-by: Matias Elo <matias.elo@nokia.com>
1 parent 609d7a4 commit 60fd92f

1 file changed

Lines changed: 19 additions & 12 deletions

File tree

test/validation/api/pktio/pktio.c

Lines changed: 19 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4242,21 +4242,14 @@ static void test_chksum(void (*config_fn)(odp_pktio_t, odp_pktio_t),
42424242
return;
42434243
}
42444244

4245-
/* Provide L3 and L4 proto for pktout HW checksum generation */
4246-
for (i = 0; i < TX_BATCH_LEN; i++) {
4247-
odp_packet_has_ipv4_set(pkt_tbl[i], true);
4248-
if (is_sctp)
4249-
odp_packet_has_sctp_set(pkt_tbl[i], true);
4250-
else
4251-
odp_packet_has_udp_set(pkt_tbl[i], true);
4252-
}
4253-
42544245
ret = odp_pktout_queue(pktio_tx, &pktout_queue, 1);
42554246
CU_ASSERT_FATAL(ret > 0);
42564247

4257-
for (i = 0; i < TX_BATCH_LEN; i++)
4248+
for (i = 0; i < TX_BATCH_LEN; i++) {
4249+
odp_packet_has_ipv4_set(pkt_tbl[i], 1);
42584250
if (prep_fn)
42594251
prep_fn(pkt_tbl[i]);
4252+
}
42604253

42614254
if (test_flags & TEST_WITH_REFS)
42624255
make_refs(ref_tbl, pkt_tbl, TX_BATCH_LEN);
@@ -4562,6 +4555,7 @@ static void pktio_test_chksum_out_udp_test(odp_packet_t pkt)
45624555
static void pktio_test_chksum_out_udp_no_ovr_prep(odp_packet_t pkt)
45634556
{
45644557
odph_ipv4_csum_update(pkt);
4558+
odp_packet_has_udp_set(pkt, 1);
45654559
odp_packet_l4_chksum_insert(pkt, false);
45664560
}
45674561

@@ -4583,6 +4577,7 @@ static void pktio_test_chksum_out_udp_no_ovr(void)
45834577

45844578
static void pktio_test_chksum_out_udp_ovr_prep(odp_packet_t pkt)
45854579
{
4580+
odp_packet_has_udp_set(pkt, 1);
45864581
odp_packet_l4_chksum_insert(pkt, true);
45874582
}
45884583

@@ -4621,10 +4616,15 @@ static void pktio_test_chksum_out_udp_pktio_config(odp_pktio_t pktio_tx,
46214616
CU_ASSERT_FATAL(odp_pktio_config(pktio_tx, &config) == 0);
46224617
}
46234618

4619+
static void pktio_test_chksum_out_udp_pktio_prep(odp_packet_t pkt)
4620+
{
4621+
odp_packet_has_udp_set(pkt, 1);
4622+
}
4623+
46244624
static void pktio_test_chksum_out_udp_pktio(void)
46254625
{
46264626
pktio_test_chksum(pktio_test_chksum_out_udp_pktio_config,
4627-
NULL,
4627+
pktio_test_chksum_out_udp_pktio_prep,
46284628
pktio_test_chksum_out_udp_test);
46294629
}
46304630

@@ -4668,6 +4668,7 @@ static void pktio_test_chksum_out_sctp_test(odp_packet_t pkt)
46684668
static void pktio_test_chksum_out_sctp_no_ovr_prep(odp_packet_t pkt)
46694669
{
46704670
odph_ipv4_csum_update(pkt);
4671+
odp_packet_has_sctp_set(pkt, 1);
46714672
odp_packet_l4_chksum_insert(pkt, false);
46724673
}
46734674

@@ -4689,6 +4690,7 @@ static void pktio_test_chksum_out_sctp_no_ovr(void)
46894690

46904691
static void pktio_test_chksum_out_sctp_ovr_prep(odp_packet_t pkt)
46914692
{
4693+
odp_packet_has_sctp_set(pkt, 1);
46924694
odp_packet_l4_chksum_insert(pkt, true);
46934695
}
46944696

@@ -4727,10 +4729,15 @@ static void pktio_test_chksum_out_sctp_pktio_config(odp_pktio_t pktio_tx,
47274729
CU_ASSERT_FATAL(odp_pktio_config(pktio_tx, &config) == 0);
47284730
}
47294731

4732+
static void pktio_test_chksum_out_sctp_pktio_prep(odp_packet_t pkt)
4733+
{
4734+
odp_packet_has_sctp_set(pkt, 1);
4735+
}
4736+
47304737
static void pktio_test_chksum_out_sctp_pktio(void)
47314738
{
47324739
pktio_test_chksum_sctp(pktio_test_chksum_out_sctp_pktio_config,
4733-
NULL,
4740+
pktio_test_chksum_out_sctp_pktio_prep,
47344741
pktio_test_chksum_out_sctp_test);
47354742
}
47364743

0 commit comments

Comments
 (0)