On Wed, Nov 26, 2014 at 07:11:33PM +0000, Shmulik Ladkani wrote:
> This is unnecessary for linux, as HW and OS take care of it.
>
> Removing the validation allows to process frames arriving at virtual
> interfaces (e.g. frames originated by host network stack).
>
> Signed-off-by: Shmulik Ladkani <shmulik.ladk...@gmail.com>
Reviewed-and-tested-by: Stuart Haslam <stuart.has...@arm.com>
> ---
> platform/linux-generic/odp_packet.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/platform/linux-generic/odp_packet.c
> b/platform/linux-generic/odp_packet.c
> index a574de162c..f8fd8ef596 100644
> --- a/platform/linux-generic/odp_packet.c
> +++ b/platform/linux-generic/odp_packet.c
> @@ -172,12 +172,8 @@ void odp_packet_parse(odp_packet_t pkt, size_t len,
> size_t frame_offset)
> pkt_hdr->frame_offset = frame_offset;
> pkt_hdr->frame_len = len;
>
> - if (odp_unlikely(len < ODPH_ETH_LEN_MIN)) {
> - pkt_hdr->error_flags.frame_len = 1;
> - return;
> - } else if (len > ODPH_ETH_LEN_MAX) {
> + if (len > ODPH_ETH_LEN_MAX)
> pkt_hdr->input_flags.jumbo = 1;
> - }
>
> /* Assume valid L2 header, no CRC/FCS check in SW */
> pkt_hdr->input_flags.l2 = 1;
> --
> 1.9.1
>
>
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> http://lists.linaro.org/mailman/listinfo/lng-odp
>
--
Stuart.
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/lng-odp
