Reviewed-by: Chengwen Feng <[email protected]>
This commit also fix one access not-initialized inner_l2_len problem, consider:
If the packet is one tunnel type which recognize by ptype_tunnel_without_udp(),
and
application don't init "struct rte_net_hdr_lens hdr_lens;" , then the
inner_l2_len will
be random without this commit because later process the inner_l2_len both are
both += ops.
On 9/24/2025 10:41 PM, Thomas Monjalon wrote:
> Looking for review, please.
>
>
> 23/07/2025 12:47, Dengdui Huang:
>> The meaning of L2_len in the parsing result of the rte_net_get_ptype()
>> is the same as the L2_len field in the mbuf. For tunnel packets,
>> the L2_len should include protocol header of tunnel packets.
>>
>> Bugzilla ID: 1754
>> Fixes: d21d855464ff ("net: support GRE in software packet type parser")
>> Cc: [email protected]
>>
>> Signed-off-by: Dengdui Huang <[email protected]>
>> ---
>> lib/net/rte_net.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/lib/net/rte_net.c b/lib/net/rte_net.c
>> index 44fb6c0f51..c70b57fdc0 100644
>> --- a/lib/net/rte_net.c
>> +++ b/lib/net/rte_net.c
>> @@ -481,6 +481,7 @@ uint32_t rte_net_get_ptype(const struct rte_mbuf *m,
>>
>> pkt_type |= ptype_tunnel_without_udp(&proto, m, &off);
>> hdr_lens->tunnel_len = off - prev_off;
>> + hdr_lens->inner_l2_len = off - prev_off;
>> }
>>
>> /* same job for inner header: we need to duplicate the code
>>
>
>
>
>
>
