>AFAIK, we can't copy-paste code from Linux kernel.
>As you noted it is under GPL, while DPDK is under BSD-3 license.
Well, I'll rewrite the code.
>Library routine has no idea would original IP packet will be used later or not.
>In your particular case it might be not needed, but there might be
Hi Huichao
>>As a nit, why not 'uint8_t *', to keep style the same through all file?
>Yes,I can use 'uint8_t *.Thank you for your correction.
>
>>We already done such calculation in rte_ipv4_fragment_packet(),
>>so can re-use header_len value here.
>Yes,I can re-use header_len.Thank you for your
A small problem.Why is the content of the email just sent to you not visible at
Patchwork (this patch).
Hi,Konstantin
Thank you for your reply！
>As a nit, why not 'uint8_t *', to keep style the same through all file?
Yes,I can use 'uint8_t *.Thank you for your correction.
>We already done such calculation in rte_ipv4_fragment_packet(),
>so can re-use header_len value here.
Yes,I can re-use hea
Hi everyone,
This patch hasn't changed status for a long time, I want to know what the
current situation of this patch is?
Huichao Cai
Perhaps performance is more important.This code comes from the linux
kernel(5.10.9 and so on). :)
It is more performance-focused based on comments. :)
I didn't look at it in detail yet,
just wonder would be real gain in terms of space?
From: Huichao Cai
Sent: Thursday, December 2, 2021 11:39 AM
To: Dariusz Sosnowski
Cc: Ananyev, Konstantin ; dev@dpdk.org
Subject: Re:Re: [PATCH] ip_frag: add IPv4 options fragment and unit test data
If op
If options with copied flag unset are not copied, then IPv4 headers in the
fragments (despite 1st fragment) will be shorter. This leaves more byte space
for the payload and in effect fragmentation might produce less fragments.
--Do I need to modify it this way?
Hi Dariusz
Substituting options with NOOP might cause rte_ipv4_fragment_packet to produce
more fragments than necessary, since options with copied flag unset will still
occupy space in IPv4 header.
--The "ip_options_fragment" just make a replacement and doesn't change the
length of the IPv4
9 matches
Mail list logo
