From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f52.google.com (mail-wm0-f52.google.com [74.125.82.52]) by dpdk.org (Postfix) with ESMTP id AB8FB5699 for ; Thu, 17 Nov 2016 10:50:16 +0100 (CET) Received: by mail-wm0-f52.google.com with SMTP id g23so299146226wme.1 for ; Thu, 17 Nov 2016 01:50:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id; bh=DoZU17A4Acfxdnfmz1sel3n5Ksay+JX7g+ZTOegc6AA=; b=0JK9QIWtamTUyvik8VXwUSRrvBzd5YaJoS7WT90NTXVSc9qLWJqk1VpAyknGHdLPpF sefPxO8Ug0EHNuKFPgx+XM5gdxkXfr+5PDEjygtx7bekfSsN0FNZZaWl+36SYygV22oc PHI0jAv57xlxnkcsqEDxBr9FzvTViVH6E68ejKqigjd9sLpRL94YQ5PlJx69G387NIc6 sQ9Q9Ta3zmUzFmSpA3xPz69oU+X1XEm2z8X40uIYtw6pivSPVPi8rMco0GO5qV0vi7Th o/ZXFRlAs5fBLZVhGNmMkf5JLAFxf0Qfi4PtLBH/3LNi3v+j3iANIym5P2tsxt2NLIv+ aQFQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=DoZU17A4Acfxdnfmz1sel3n5Ksay+JX7g+ZTOegc6AA=; b=D3aQdUS4lRGn9Xa58QHxZHumqrHssuxFpU3wwqlVx/s0M3oZ1vCR2mrSnPJLgfzCGP 5NL522MluY/dacDDjsSrgCfYJBPIVyAuxXQitTNNkgjP9a2jV9W6G3g6aauPQ1qIy1LY XCr95aXmrJOmix5c7W9pJxHDgymG2Q2UUcfl3AcSGdJi6UXTS6oT5LTL7UQ60b9yS/Ed JMo9k6MUzWCAr0MemKLlHUWqo4jW0MhufxQtMubLQgCkBWtB+puTNMDWdWD4KOP4L9NS kEOosIGwLPfUJT01h0UejREqabEJRnv23MbUhiW9+WBzVgN4ktnPNXLsn5A1S1QrH2Ui gASg== X-Gm-Message-State: AKaTC03l/E5V8JH4v3em6T50Rau64kFtvXcSpgHwd+Fjl2JDWcxvdGYuGwtBOh5+uZn7x7FN X-Received: by 10.194.71.228 with SMTP id y4mr1685177wju.136.1479376216333; Thu, 17 Nov 2016 01:50:16 -0800 (PST) Received: from ping.vm.6wind.com (guy78-3-82-239-227-177.fbx.proxad.net. [82.239.227.177]) by smtp.gmail.com with ESMTPSA id c202sm17059625wme.1.2016.11.17.01.50.15 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Thu, 17 Nov 2016 01:50:15 -0800 (PST) From: Nelio Laranjeiro To: dev@dpdk.org Cc: Adrien Mazarguil , stable@dpdk.org Date: Thu, 17 Nov 2016 10:49:54 +0100 Message-Id: X-Mailer: git-send-email 2.1.4 Subject: [dpdk-stable] [PATCH 1/3] net/mlx5: fix leak when starvation occurs X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Nov 2016 09:50:16 -0000 The list of segments to free was wrongly manipulated ending by only freeing the first segment instead of freeing all of them. The last one still belongs to the NIC and thus should not be freed. Fixes: a1bdb71a32da ("net/mlx5: fix crash in Rx") CC: stable@dpdk.org Reported-by: Liming Sun Signed-off-by: Nelio Laranjeiro --- drivers/net/mlx5/mlx5_rxtx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxtx.c b/drivers/net/mlx5/mlx5_rxtx.c index beff580..9bd4d80 100644 --- a/drivers/net/mlx5/mlx5_rxtx.c +++ b/drivers/net/mlx5/mlx5_rxtx.c @@ -1312,10 +1312,10 @@ mlx5_rx_burst(void *dpdk_rxq, struct rte_mbuf **pkts, uint16_t pkts_n) } while (pkt != seg) { assert(pkt != (*rxq->elts)[idx]); - seg = NEXT(pkt); + rep = NEXT(pkt); rte_mbuf_refcnt_set(pkt, 0); __rte_mbuf_raw_free(pkt); - pkt = seg; + pkt = rep; } break; } -- 2.1.4