Merged, thanks!

Maxim.

On 11/27/2014 09:32 PM, Stuart Haslam wrote:
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 <[email protected]>
Reviewed-and-tested-by: Stuart Haslam <[email protected]>

---
  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
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp



_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to