Message ID | 1508404312-16996-1-git-send-email-rasland@mellanox.com (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | Ferruh Yigit |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 069CC1B1A0; Thu, 19 Oct 2017 11:12:05 +0200 (CEST) Received: from EUR03-DB5-obe.outbound.protection.outlook.com (mail-eopbgr40069.outbound.protection.outlook.com [40.107.4.69]) by dpdk.org (Postfix) with ESMTP id AB0991B19F for <dev@dpdk.org>; Thu, 19 Oct 2017 11:12:03 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=MCagyldf5eFWewC21Vsgiw9j+SAq/McLI6svd2E9q+I=; b=hdmgmgl9DsnqvYLqJLMLUybpWl5lkrDMV2YW2j8mCz7URl8+aj8v06TlWkOKFxdDPaWWdqFW30c8FrKF/gvovYRb6Tk/c58LaZwzq+sD3KXMXiBgyAyUA7NUeNCJkwghJOb903lX9ZvGzQYzYgUpPxrXfi8nyQW0IFYz1t47Y78= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=rasland@mellanox.com; Received: from mellanox.com (37.142.13.130) by AM2PR05MB1218.eurprd05.prod.outlook.com (2a01:111:e400:8450::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Thu, 19 Oct 2017 09:12:01 +0000 From: Raslan Darawsheh <rasland@mellanox.com> To: gaetan.rivet@6wind.com Cc: dev@dpdk.org, thomas@monjalon.net Date: Thu, 19 Oct 2017 12:11:52 +0300 Message-Id: <1508404312-16996-1-git-send-email-rasland@mellanox.com> X-Mailer: git-send-email 2.7.4 MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [37.142.13.130] X-ClientProxiedBy: VI1PR0102CA0096.eurprd01.prod.exchangelabs.com (2603:10a6:803:15::37) To AM2PR05MB1218.eurprd05.prod.outlook.com (2a01:111:e400:8450::16) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 3a64646a-b85a-4449-109f-08d516d17577 X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(48565401081)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:AM2PR05MB1218; X-Microsoft-Exchange-Diagnostics: 1; AM2PR05MB1218; 3:JE/Q8Rj18J4PPoAAZH5yKj+k1ttqRumiNzu72vlkDK+1KwoSDQWuuxixmuI1KFBTAZcdHEyahnIN+/ml3dwXk+T8KTfFnpJDyJmrB0hmSJPfhJxm8XpGTLW9u5rd0EnMtLAAeMlkIByezaTgvrmXcAqQHk8UA5CUc0OndF+8veA7yoAfHEbw+ifsEukR/hvgdwvLS5HNr0cVh/g2bZ2lBZ0YOTxaPKvJCJ20FpdUtI/uLCQsxSgIMB+50eU/wp2W; 25:9u74RjZxh/C29EZz4IUaszd5mnmyiAcJbdEAAVbl7fZJERrh1oErPTlwzmglojlEQV66ips7qH0fZ6F3SHYz2kn33+Q5/lP2WyTAUxVFq79TQBuQjg641POfXu09GDcoKXcYVJ4kYaOntcVSW1qJgNwRZDjgoxUHyDvIIMEj1xfj2fNpQe/TBn6tIJgSwyQjMZ42WPL16J/leULWfOi7LA/xILMyV7xdG4D1Psp2wo3mCtdVeIHeHnOfwO5sQFiljRX369X50W3hMAhk5rGch2eggtOoVM5XSBfqaebJ4m1CwQ5D7A57amzi9At+Sk/uMDL1EPoRPF1Aa3WrJofgIA==; 31:5cALqU7fmiQNANvboLcn973w+JNquYJcYx1E9iNzJWIVtQytaXpomimI8BVdYFxq7fzDedljf3o6iJj1TWUIIUA7W73IXF1s9M73ndQAfNDciLpvQJ/z3i+Q3Q2TI8mGghhCklFdhhPAQHCydA7gZF1YiOl913t94uxAnQEuYuwLLYnSaNdznJSLDqIo7HaPlLKTolUEj2PVahpKAX1eaw0aRhScAF5t9YBAW47xulg= X-MS-TrafficTypeDiagnostic: AM2PR05MB1218: X-LD-Processed: a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtAddr X-Microsoft-Exchange-Diagnostics: 1; AM2PR05MB1218; 20:mAfS5gwtJCFm09CUxyzjC+MFEDg7+GgsR6bM0/ap6OdNLvNvM8LIhBOdiISoBfzei3BVQG2VGYpSnBXWCib6pRkXgPrSXg/6gRkqzSvZQbpKSQE8gcPnWc3HSaGILNR+Es8C4QyrbHDiJvzra2uQou/Ys5YZqXIBqI+3l0h3fto5Q2QeTlSN4DsZkgiyfEuMEcYumqR26NOmPa7s5MnCPNaY8Wbe/1c0bXAUE2pDyAQocokxv34TH5nDm5iTG9QbOskV7Qz7PTa1xGMG/CtQWQq094YYdUNqbXxNdL8sjWepWvQQZia8ilMxUg1GboxHG8QJcUyZ2sfrb9IyBYu2jMgMbNPdNbJmnhtOWnZCVZX15E/8OTM2GmSgVLa5nmvErIC5uXNYfn7mN7dAvPPFOee2cXuXgSwxnFl+r1lE6nU5b1oEGK6cGiIuf0neoB8xpkPQSmoyh49mLVuMWifwjgl7hgJlONktUykm0loUoHmI2d6wdSvt2+hz49Rfa9Xe; 4:Ov4caCYBtCW+OOgPvDfy8K6xNTp+/dr5EZXzW7zKDSX3HsL+qxhVgE33sFDxaelhG7ZcqiayKMUmkExGUNplIspsdT37LOvCRVBO5kx45jovl1DwIP3+duUCGa8ortAZVFQluviTILcu2o+W0HX+bw3/RjLjTHUdtqD7BBnUUqBD+lReitMtuu28ppAdEawH9l5SZasS8pB40dAFJrtqyHBNm/0TNhHGAQf1Ojl0woCZ+WQyH27nVfVA5HC1gATt X-Exchange-Antispam-Report-Test: UriScan:; X-Microsoft-Antispam-PRVS: <AM2PR05MB1218098C71D8E1AEB52D3870C2420@AM2PR05MB1218.eurprd05.prod.outlook.com> X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(10201501046)(100000703101)(100105400095)(3002001)(93006095)(93001095)(6055026)(6041248)(20161123564025)(20161123555025)(20161123558100)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123560025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:AM2PR05MB1218; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:AM2PR05MB1218; X-Forefront-PRVS: 0465429B7F X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(346002)(376002)(39860400002)(199003)(189002)(48376002)(189998001)(106356001)(2361001)(105586002)(305945005)(69596002)(6666003)(33646002)(2351001)(50986999)(3846002)(6116002)(36756003)(66066001)(7736002)(47776003)(86362001)(101416001)(50466002)(68736007)(478600001)(33026002)(5890100001)(316002)(55016002)(6916009)(16586007)(8936002)(21086003)(2906002)(25786009)(16526018)(97736004)(50226002)(5003940100001)(53936002)(4326008)(81156014)(8676002)(5660300001)(81166006); DIR:OUT; SFP:1101; SCL:1; SRVR:AM2PR05MB1218; H:mellanox.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: mellanox.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; AM2PR05MB1218; 23:wluDsWp0FXJABWikbtm928Xy2fKMPSmfNfuMPicpc?= 2kY4RevG44M07wHHT3DbznP5pj+F3f87WCAfpeauDF/QMLZTt4BubnJtr/Olx629M8ld1pBIWspQwRsohvlagwmobuKvri06a52uAtE8tLw+G7PmIEO01zaSZws1Ji8FZfUOzqLVFakekM+4eL/QbPJ/iqHOXoAyEMbMLMhyvy+U4N2DZgki/9wDQZWfPSUIQuYIj01KJFODjsBH2sGqhz7RzQc++zSE9H7Z/3oXDI4+yea9q2MKnKBdgTil9X9BFcTfyq19YJT5CvSnIMQdg6uxF/kgbTW/LbKAN0DJ8YPX8HY2JDCgzmTtyuCyJz6QKHTjYb+z864slUgl4c6VBXtkp47HFAJNfrLhUJaJzbHzFTD13dxSiirxEMi2Je7Wwfo0RyyQwFBbF2/XVG1//oCNrx99wW3DoFb2ZOaFOdFbHIaUgk+E8LQi71YkPvZGkQXcGWGrnrh5a2miJ6SDVKzAxXBhoEJp/1oepVPJk2QUORq0tXmOf9PVlTxLSGno1t0sfYpzs8Fb9fBEHkj2znetf8LlWayJmLQCiQ+YdLgIUQNkbsbQzyB9u/inCxE33YO36zbBCgHRLb3TDb7zV+06nIvVDdvDPdR0qiTXGbWvYC4E2gBOg6z/bISeuVvjv2tKjXIAlTDoRiTLerPA8SMgoOu6ds5l27DkQ5NEQJCYd9lPSC5GP1WxnvzV1VPOBwnJgrgsRIjkLLNHvALdZH2J7g8svx+LVwhwdLxKuv8evKqEKZ9yJdw9fTclj8OM/7rz2mY2jg2EssZBAxrf0lN0Suedh1HjijuQDqEf42iCIkmpshDcYePeytA/2CE445elwxndC0/tAmSqby1YeKLQPVub1qmBs5j7Xrvt79jsRiU83t0eihjHyBqxwR43RRwfl2KNnIM4X0Mbk6F2zALRfLjjOmqHAZ1JMG/TuMsUKjRgzQy3e2ERiOzMR7QRUvDu2S2meMNmNT0datj1kzreBDdl4ammO1d72bI+BdYhVCmfIr978LpSUoGZSRATXmz8FQEACBIgsKmlUoaRZKaFnMYYdIOUl0I9KzAs1i6Y57DJaTeEIMHZ0Trs4eBFS6ZunJFDYMQBHe5UMIzNq8X X-Microsoft-Exchange-Diagnostics: 1; AM2PR05MB1218; 6:++EmJmVqQ8SWpTMohuhhE6Dx1AnKvg8gUVg9hvx8N85fOFT/wbxfrv6rMIaXGXaT531Kkd1mJNgKNCVUN5EeNQ//Pt86lcaQ5w/zWF6GfMKnhHk7hALPLG3IwBGRMA5kH1Omqy9X52UJWQ3AE5iXUKLC2WQQjfWcJb1/+0wrdNeAMUv1ugnuahSPFjkvEBMfag7swfWTySsNORdK+UQbdu6m3pA5pXIR67ioAvHBC8cewaWBCZ9qcfGrmKd/Xuq5GRDP3iaEalNWYHFIRmipCTN+uimw/VJDIKnyYUkchOQ9eIH/bSRvEQ9fKbRdhF2iRXJz6+Km+mGFNL+czCxhzg==; 5:KO/c5686QPWI1s4P+7SNEei9Q1swazu420/1f43jQAqtIp5SQNI4faK1kztMUuSOoiOIRiMXr6pX7a2U20+A4ODmzsrivfzSONm7W7sciQIKI3L/mxweWO9aCNIr6woWZZn0m5Vn3s1l9WcvX6U5Tg==; 24:gxIQSDhboysiYOxDoARerURGVaqcwPAsv5eSfppmat9WJSM8DY97iKv8utRwo9E8n1UedWlpJ6ZhMRakNZnzl+usvKGBbZ3IhZ4yVWQ2JUE=; 7:Ck96FfuHVx45lZWT3DZCmzGSDMYEwGLEiOuiXjn7opWe91QFZNeqUURM/ZcIWmJcyHnICuNKu2fT5VTu+ANfndNI7WJ6hzVsY5fqyKZoIIA60EG/0HbjQRQPga7hJcp2ebiQ66iLRTdb5oGmRxBFUfBKZWWqGZxYfhpxWKQUse0eioXvFKgUVYkTAqSQpsHFk4b3nvKb5HHgb/t2Z2LPlAifXD2l7qdAzOoYQTDdd6s= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Oct 2017 09:12:01.0787 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM2PR05MB1218 Subject: [dpdk-dev] [PATCH] net/failsafe: fix PCI devices init X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list 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> |
Checks
Context | Check | Description |
---|---|---|
ci/checkpatch | warning | coding style issues |
ci/Intel-compilation | success | Compilation OK |
Commit Message
Raslan Darawsheh
Oct. 19, 2017, 9:11 a.m. UTC
When trying to attach a port as a sub-device, the ethdev port
was compared with devargs.
In the case of a PCI device, the name in devargs is the PCI address.
And since DPDK 17.08, the devargs name of the underlying device was
used to match an ethdev port:
a1e7c17555e8 ("ethdev: use device name from device structure")
But the recent commit 72e3efb149cc has reverted this wrong matching
to use the ethdev port name as identifier of the port.
It impacts functions like rte_eth_dev_allocated() used in failsafe
for matching ports with given devargs.
The fix is to search for matching devargs in underlying device of
all ethdev ports.
If many ports match the same PCI device, only the first one is matched.
This limitation was already present in previous implementation of
rte_eth_dev_allocated(), and must be adressed later with a better
devargs syntax.
Fixes: 72e3efb149cc ("ethdev: revert use port name from device structure")
Signed-off-by: Raslan Darawsheh <rasland@mellanox.com>
---
drivers/net/failsafe/failsafe_eal.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
Comments
Hello Raslan, Thanks for spotting the issue and fixing it. Small nit below. On Thu, Oct 19, 2017 at 12:11:52PM +0300, Raslan Darawsheh wrote: > When trying to attach a port as a sub-device, the ethdev port > was compared with devargs. > In the case of a PCI device, the name in devargs is the PCI address. > And since DPDK 17.08, the devargs name of the underlying device was > used to match an ethdev port: > a1e7c17555e8 ("ethdev: use device name from device structure") > > But the recent commit 72e3efb149cc has reverted this wrong matching > to use the ethdev port name as identifier of the port. > It impacts functions like rte_eth_dev_allocated() used in failsafe > for matching ports with given devargs. > The fix is to search for matching devargs in underlying device of > all ethdev ports. > If many ports match the same PCI device, only the first one is matched. > > This limitation was already present in previous implementation of > rte_eth_dev_allocated(), and must be adressed later with a better > devargs syntax. > > Fixes: 72e3efb149cc ("ethdev: revert use port name from device structure") > > Signed-off-by: Raslan Darawsheh <rasland@mellanox.com> With the nit below fixed: Acked-by: Gaetan Rivet <gaetan.rivet@6wind.com> > --- > drivers/net/failsafe/failsafe_eal.c | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/failsafe/failsafe_eal.c b/drivers/net/failsafe/failsafe_eal.c > index aeb87a0..734b25c 100644 > --- a/drivers/net/failsafe/failsafe_eal.c > +++ b/drivers/net/failsafe/failsafe_eal.c > @@ -41,6 +41,7 @@ fs_bus_init(struct rte_eth_dev *dev) > struct sub_device *sdev; > struct rte_devargs *da; > uint8_t i; > + uint16_t j; > int ret; > > FOREACH_SUBDEV(sdev, i, dev) { > @@ -57,7 +58,13 @@ fs_bus_init(struct rte_eth_dev *dev) > rte_errno ? ")" : ""); > continue; > } > - ETH(sdev) = rte_eth_dev_allocated(da->name); > + RTE_ETH_FOREACH_DEV(j) { > + if (!strcmp(rte_eth_devices[j].device->name, > + da->name)) { The explicit comparison to 0 should be preferred, as done in failsafe_args.c. > + ETH(sdev) = &rte_eth_devices[j]; > + break; > + } > + } > if (ETH(sdev) == NULL) { > ERROR("sub_device %d init went wrong", i); > return -ENODEV; > -- > 2.7.4 >
diff --git a/drivers/net/failsafe/failsafe_eal.c b/drivers/net/failsafe/failsafe_eal.c index aeb87a0..734b25c 100644 --- a/drivers/net/failsafe/failsafe_eal.c +++ b/drivers/net/failsafe/failsafe_eal.c @@ -41,6 +41,7 @@ fs_bus_init(struct rte_eth_dev *dev) struct sub_device *sdev; struct rte_devargs *da; uint8_t i; + uint16_t j; int ret; FOREACH_SUBDEV(sdev, i, dev) { @@ -57,7 +58,13 @@ fs_bus_init(struct rte_eth_dev *dev) rte_errno ? ")" : ""); continue; } - ETH(sdev) = rte_eth_dev_allocated(da->name); + RTE_ETH_FOREACH_DEV(j) { + if (!strcmp(rte_eth_devices[j].device->name, + da->name)) { + ETH(sdev) = &rte_eth_devices[j]; + break; + } + } if (ETH(sdev) == NULL) { ERROR("sub_device %d init went wrong", i); return -ENODEV;