From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from tyo202.gate.nec.co.jp (TYO202.gate.nec.co.jp [210.143.35.52]) by dpdk.org (Postfix) with ESMTP id 6688D6930 for ; Thu, 3 Apr 2014 09:35:09 +0200 (CEST) Received: from mailgate3.nec.co.jp ([10.7.69.197]) by tyo202.gate.nec.co.jp (8.13.8/8.13.4) with ESMTP id s337aiuV027087 for ; Thu, 3 Apr 2014 16:36:44 +0900 (JST) Received: from mailsv3.nec.co.jp (imss61.nec.co.jp [10.7.69.156]) by mailgate3.nec.co.jp (8.11.7/3.7W-MAILGATE-NEC) with ESMTP id s337ai308232 for ; Thu, 3 Apr 2014 16:36:44 +0900 (JST) Received: from mail01b.kamome.nec.co.jp (mail01b.kamome.nec.co.jp [10.25.43.2]) by mailsv3.nec.co.jp (8.13.8/8.13.4) with ESMTP id s337ahSP017050 for ; Thu, 3 Apr 2014 16:36:43 +0900 (JST) Received: from bpxc99gp.gisp.nec.co.jp ([10.38.151.141] [10.38.151.141]) by mail02.kamome.nec.co.jp with ESMTP id BT-MMP-651943; Thu, 3 Apr 2014 16:35:50 +0900 Received: from BPXM14GP.gisp.nec.co.jp ([169.254.1.238]) by BPXC13GP.gisp.nec.co.jp ([10.38.151.141]) with mapi id 14.02.0328.011; Thu, 3 Apr 2014 16:35:50 +0900 From: Hiroshi Shimamoto To: "dev@dpdk.org" Thread-Topic: [memnic PATCH] linux: fix race condition Thread-Index: Ac9PD0vv875TbkScT5qU1gq0J8OeDQ== Date: Thu, 3 Apr 2014 07:35:50 +0000 Message-ID: <7F861DC0615E0C47A872E6F3C5FCDDBD0109D69F@BPXM14GP.gisp.nec.co.jp> Accept-Language: ja-JP, en-US Content-Language: ja-JP X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.205.5.123] Content-Type: text/plain; charset="iso-2022-jp" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: Hayato Momma Subject: [dpdk-dev] [memnic PATCH] linux: fix race condition X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 03 Apr 2014 07:35:10 -0000 From: Hiroshi Shimamoto Same as PMD, there is a race condition between host and guest in the guest MEMNIC kernel driver. Guest PMD Host Thread-A Thread-B vSwitch |idx=3D0 |idx=3D0 |p[0] st!=3D2 |cmpxchg | | |p[0] st->1 | | |idx=3D1 | | |fill data | | |p[0] st->2 | |p[0] st=3D=3D2 | | |receive data | | |p[0] st->0 | |cmpxchg | | |success |p[1] st!=3D2 | |p[0] st->1 | This is BAD Signed-off-by: Hiroshi Shimamoto --- linux/memnic_net.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/linux/memnic_net.c b/linux/memnic_net.c index 9019258..fadece6 100644 --- a/linux/memnic_net.c +++ b/linux/memnic_net.c @@ -209,6 +209,15 @@ retry: goto retry; } =20 + if (idx !=3D ACCESS_ONCE(memnic->down)) { + /* + * host freed this and got false positive, + * need to recover the status and retry. + */ + p->status =3D MEMNIC_PKT_ST_FREE; + goto retry; + } + if (++idx >=3D MEMNIC_NR_PACKET) idx =3D 0; memnic->down =3D idx; --=20 1.8.4