From patchwork Fri Mar 28 11:20:40 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ciprian Barbu X-Patchwork-Id: 27256 Return-Path: X-Original-To: linaro@patches.linaro.org Delivered-To: linaro@patches.linaro.org Received: from mail-we0-f200.google.com (mail-we0-f200.google.com [74.125.82.200]) by ip-10-151-82-157.ec2.internal (Postfix) with ESMTPS id 91AB320062 for ; Fri, 28 Mar 2014 11:20:49 +0000 (UTC) Received: by mail-we0-f200.google.com with SMTP id p61sf5680936wes.3 for ; Fri, 28 Mar 2014 04:20:48 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:from:to:cc:subject:date:message-id :x-original-sender:x-original-authentication-results:precedence :mailing-list:list-id:list-post:list-help:list-archive :list-unsubscribe:content-type; bh=1VjKoOYaxiRfB/llKacja5sGOn3USaiDLOPaiIO8pww=; b=jWpJGQC/vj3lUfmQ1AtKKaN/0e2Jopf5z9Z/VVkrLzZ0j0NayW5mX88k8Nl4NNjSCL GRmfnOfq8sVWYnM9Hi9IndRB/76YCmFsnEolHGdhoBlmTKPcjucgREqriMGY8PWnsABc pellaB9Fa7x1idaKUVI8+eEXvLZWDC2B286lxMJv22xqKmu/TW+nTK9CaBKAHjC1+IWU Nr0GlJaqvZFX7o8Vnqc2KYOYRaumQdnsoUQbkcZdT5EGBZxRfBfLH7ozONw2sRQtaiHK JYKkMIljHHyANJI4zhjThPm/8WVHZBWeMi1pJyCRy3f0t/JFOVHnlor2yLIpw4E+hKBL Jmvg== X-Gm-Message-State: ALoCoQknx4UJ4432uQlF1jXZ1z6Sx4eF/xDSATSZl5I1/39Y85+8NnbcPWrw4vR2x5Qhl+pIKBKg X-Received: by 10.180.82.35 with SMTP id f3mr17925323wiy.0.1396005648039; Fri, 28 Mar 2014 04:20:48 -0700 (PDT) MIME-Version: 1.0 X-BeenThere: lng-odp@linaro.org Received: by 10.152.36.101 with SMTP id p5ls214068laj.41.gmail; Fri, 28 Mar 2014 04:20:47 -0700 (PDT) X-Received: by 10.112.139.166 with SMTP id qz6mr4854932lbb.13.1396005647663; Fri, 28 Mar 2014 04:20:47 -0700 (PDT) Received: from mail-lb0-f171.google.com (mail-lb0-f171.google.com [209.85.217.171]) by mx.google.com with ESMTPS id o5si3175721lae.14.2014.03.28.04.20.47 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 28 Mar 2014 04:20:47 -0700 (PDT) Received-SPF: neutral (google.com: 209.85.217.171 is neither permitted nor denied by best guess record for domain of ciprian.barbu@linaro.org) client-ip=209.85.217.171; Received: by mail-lb0-f171.google.com with SMTP id w7so3587038lbi.16 for ; Fri, 28 Mar 2014 04:20:47 -0700 (PDT) X-Received: by 10.112.128.231 with SMTP id nr7mr4917018lbb.9.1396005647572; Fri, 28 Mar 2014 04:20:47 -0700 (PDT) Received: from sestofb10.enea.se (sestofw01.enea.se. [192.36.1.252]) by mx.google.com with ESMTPSA id wk4sm3480827lbb.11.2014.03.28.04.20.46 for (version=TLSv1.1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 28 Mar 2014 04:20:46 -0700 (PDT) From: Ciprian Barbu To: lng-odp@linaro.org Cc: Ciprian Barbu Subject: [lng-odp] [lng-patch][PATCH v2] packet parsing modifications Date: Fri, 28 Mar 2014 12:20:40 +0100 Message-Id: <1396005641-19127-1-git-send-email-ciprian.barbu@linaro.org> X-Mailer: git-send-email 1.7.10.4 X-Original-Sender: ciprian.barbu@linaro.org X-Original-Authentication-Results: mx.google.com; spf=neutral (google.com: 209.85.217.171 is neither permitted nor denied by best guess record for domain of ciprian.barbu@linaro.org) smtp.mail=ciprian.barbu@linaro.org Precedence: list Mailing-list: list lng-odp@linaro.org; contact lng-odp+owners@linaro.org List-ID: X-Google-Group-Id: 474323889996 List-Post: , List-Help: , List-Archive: List-Unsubscribe: , Signed-off-by: Ciprian Barbu Here is a simpler version, no return code, just check if the minimum eth header is present. --- platform/linux-generic/source/odp_packet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/platform/linux-generic/source/odp_packet.c b/platform/linux-generic/source/odp_packet.c index 5f07374..b990222 100644 --- a/platform/linux-generic/source/odp_packet.c +++ b/platform/linux-generic/source/odp_packet.c @@ -154,7 +154,7 @@ 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 < ODP_ETH_LEN_MIN)) { + if (odp_unlikely(len < ODP_ETHHDR_LEN)) { pkt_hdr->error_flags.frame_len = 1; return; } else if (len > ODP_ETH_LEN_MAX) {