Updated the `get_ethertype_by_ptype` function to correctly parse VLAN headers.
Fixes: 76730c7b9b5a ("app/testpmd: use packet type parsing API") Cc: haij...@huawei.com Signed-off-by: Raslan Darawsheh <rasl...@nvidia.com> --- app/test-pmd/csumonly.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/app/test-pmd/csumonly.c b/app/test-pmd/csumonly.c index 5b906eaa53..302cc4cc66 100644 --- a/app/test-pmd/csumonly.c +++ b/app/test-pmd/csumonly.c @@ -468,6 +468,7 @@ get_ethertype_by_ptype(struct rte_ether_hdr *eth_hdr, uint32_t ptype) { struct rte_vlan_hdr *vlan_hdr; uint16_t ethertype; + uint32_t i = 0; switch (ptype) { case RTE_PTYPE_L3_IPV4: @@ -486,10 +487,11 @@ get_ethertype_by_ptype(struct rte_ether_hdr *eth_hdr, uint32_t ptype) return _htons(RTE_ETHER_TYPE_IPV6); default: ethertype = eth_hdr->ether_type; - while (eth_hdr->ether_type == _htons(RTE_ETHER_TYPE_VLAN) || - eth_hdr->ether_type == _htons(RTE_ETHER_TYPE_QINQ)) { + while (ethertype == _htons(RTE_ETHER_TYPE_VLAN) || + ethertype == _htons(RTE_ETHER_TYPE_QINQ)) { vlan_hdr = (struct rte_vlan_hdr *) - ((char *)eth_hdr + sizeof(*eth_hdr)); + ((char *)eth_hdr + sizeof(*eth_hdr) + + (i * sizeof(struct rte_vlan_hdr))); ethertype = vlan_hdr->eth_proto; } return ethertype; -- 2.39.5 (Apple Git-154)