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)

Reply via email to