From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: [PATCH 2/2] net/virtio: fix lgtm static checker complaint Date: Wed, 27 Mar 2019 14:08:43 -0700 Message-ID: <20190327210843.13813-3-stephen@networkplumber.org> References: <20190327210843.13813-1-stephen@networkplumber.org> Cc: dev@dpdk.org, Stephen Hemminger To: maxime.coquelin@redhat.com, tiwei.bie@intel.com, zhihong.wang@intel.com Return-path: Received: from mail-pl1-f195.google.com (mail-pl1-f195.google.com [209.85.214.195]) by dpdk.org (Postfix) with ESMTP id D1EE411A4 for ; Wed, 27 Mar 2019 22:08:49 +0100 (CET) Received: by mail-pl1-f195.google.com with SMTP id p19so3888031plo.2 for ; Wed, 27 Mar 2019 14:08:49 -0700 (PDT) In-Reply-To: <20190327210843.13813-1-stephen@networkplumber.org> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Since previous test is for mtu < 1519 the next else if is always true. This causes the lgtm static tool to complain. Not a real issue, just cosmetic. Signed-off-by: Stephen Hemminger --- drivers/net/virtio/virtio_rxtx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/virtio/virtio_rxtx.c b/drivers/net/virtio/virtio_rxtx.c index 5f6796bdba37..268fe3b8f156 100644 --- a/drivers/net/virtio/virtio_rxtx.c +++ b/drivers/net/virtio/virtio_rxtx.c @@ -1110,7 +1110,7 @@ virtio_update_packet_stats(struct virtnet_stats *stats, struct rte_mbuf *mbuf) stats->size_bins[0]++; else if (s < 1519) stats->size_bins[6]++; - else if (s >= 1519) + else stats->size_bins[7]++; } -- 2.17.1