From mboxrd@z Thu Jan  1 00:00:00 1970
From: Yongseok Koh <yskoh@mellanox.com>
Subject: Re: Mellanox ConnectX-5 crashes and mbuf leak
Date: Sat, 7 Oct 2017 22:19:14 +0000
Message-ID: <FFCEA33F-02F0-4056-99B1-A8CA004BEE51@mellanox.com>
References: <5d1f07c4-5933-806d-4d11-8fdfabc701d7@allegro-packets.com>
 <F1EE998E-D9C7-440E-836A-00CE6822D496@mellanox.com>
 <eecae93a-8652-4b56-4685-5c80a1f58617@allegro-packets.com>
 <374F8C13-CFB0-42FD-8993-BF7F0401F891@mellanox.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Cc: Adrien Mazarguil <adrien.mazarguil@6wind.com>,
 =?iso-8859-1?Q?N=E9lio_Laranjeiro?= <nelio.laranjeiro@6wind.com>,
 "dev@dpdk.org" <dev@dpdk.org>, Ferruh Yigit <ferruh.yigit@intel.com>
To: Martin Weiser <martin.weiser@allegro-packets.com>
Return-path: <dev-bounces@dpdk.org>
Received: from EUR01-VE1-obe.outbound.protection.outlook.com
 (mail-ve1eur01on0074.outbound.protection.outlook.com [104.47.1.74])
 by dpdk.org (Postfix) with ESMTP id 416762B9E
 for <dev@dpdk.org>; Sun,  8 Oct 2017 00:19:17 +0200 (CEST)
In-Reply-To: <374F8C13-CFB0-42FD-8993-BF7F0401F891@mellanox.com>
Content-Language: en-US
Content-ID: <991FBEDDB0E21C4DB3653AE28C90E6A3@eurprd05.prod.outlook.com>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

> On Oct 6, 2017, at 3:30 PM, Yongseok Koh <yskoh@mellanox.com> wrote:
>=20
> Hi, Martin
>=20
> Even though I had done quite serious tests before sending out the patch,
> I figured out deadlock could happen if the Rx queue size is smaller. It i=
s 128
> by default in testpmd while I usually use 256.
>=20
> I've fixed the bug and submitted a new patch [1], which actually reverts =
the
> previous patch. So, you can apply the attached with disregarding the old =
one.
>=20
> And I have also done extensive tests for this new patch but please let me=
 know
> your test results.
>=20
> [1]
> "net/mlx5: fix deadlock due to buffered slots in Rx SW ring"
> at http://dpdk.org/dev/patchwork/patch/29847

Hi Martin

I've submitted v2 of the patch [1]. I just replaced vector insns with regul=
ar
statements.  This is just for ease of maintenance because I'm about to add
vectorized PMD for ARM NEON.  In terms of functionality and performance it =
is
identical.

Please proceed your testing with this and let me know the result.

[1]
[dpdk-dev,v2] net/mlx5: fix deadlock due to buffered slots in Rx SW ring
, which is at http://dpdk.org/dev/patchwork/patch/29879/

Thanks,
Yongseok