Message ID | 20180410061336.49844-1-shahafs@mellanox.com (mailing list archive) |
---|---|
State | Accepted, archived |
Delegated to: | Shahaf Shuler |
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 811E11BA32; Tue, 10 Apr 2018 08:13:52 +0200 (CEST) Received: from EUR01-VE1-obe.outbound.protection.outlook.com (mail-ve1eur01on0050.outbound.protection.outlook.com [104.47.1.50]) by dpdk.org (Postfix) with ESMTP id A6A3B1BA1F for <dev@dpdk.org>; Tue, 10 Apr 2018 08:13:50 +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=nzC6c9ztLoiJJ51P52Q8fxyNChtVwevv9s/XjTk4Uyc=; b=D8ErBlxZ7vXh4yTHjRfddZzIusT2VB9QE1XHpDbDPwhBo7cNjMZuDfbuL69QwBc/+Gm/jPFZ+Yx0KS88PBYBMsW5nuVIWJDjpfBaA+ry/ak7u/e/QMVh/I8cdo/Im/a3a8CT0Mm2axpRDtZhM4D/6x4hzM2s7gVc3r4JWzECwRY= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=shahafs@mellanox.com; Received: from mellanox.com (141.226.120.58) by AM0PR05MB4420.eurprd05.prod.outlook.com (2603:10a6:208:5a::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.653.12; Tue, 10 Apr 2018 06:13:48 +0000 From: Shahaf Shuler <shahafs@mellanox.com> To: nelio.laranjeiro@6wind.com, adrien.mazarguil@6wind.com, yskoh@mellanox.com Cc: dev@dpdk.org Date: Tue, 10 Apr 2018 09:13:36 +0300 Message-Id: <20180410061336.49844-1-shahafs@mellanox.com> X-Mailer: git-send-email 2.12.0 In-Reply-To: <20180403044817.27457-1-shahafs@mellanox.com> References: <20180403044817.27457-1-shahafs@mellanox.com> MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [141.226.120.58] X-ClientProxiedBy: AM5PR0102CA0033.eurprd01.prod.exchangelabs.com (2603:10a6:206::46) To AM0PR05MB4420.eurprd05.prod.outlook.com (2603:10a6:208:5a::17) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(48565401081)(5600026)(4534165)(4627221)(201703031133081)(201702281549075)(2017052603328)(7153060)(7193020); SRVR:AM0PR05MB4420; X-Microsoft-Exchange-Diagnostics: 1; AM0PR05MB4420; 3:ovrIQt6WkqVPxFCyzWFcPtaF8qZqrjukH/e7VUzCYMxBu/403UvhYH+uceJ2/7EE5JE9Sz8ZrtEsTB7DAz46R7irorz44O1XDrwmptkxxK7A3tN4dvxQG/fX6wuzBkYw6usk+dgwT8q5fJzfaa8XpQnk7s8tNZZVc1FGX51h8eymLbSwE80Adhgls+9buy/RupYpbYa2byt/W8tzR4kjJHJcaABTk0ZChHuR4Zna8lsscOsuKhPZLPgxRkLgcN4j; 25:OnLPsS/IVk55RyaXqnAD3Z/0vIxpw5uyom6aWVzhgAEQ7jUoy1VmS8/SVKlB7nbFroQm7JXt8MFZxyS/IVXvmYd+kAsXtrp7enhbKi/iNpbFTKzpeZ8KGvWpxNKjNu72f4/AaSkFVPKeP6QJNRLhsC/HayXHBGZ5Q2LsJpzSVSpGLYjPaUkM/k1RVMTx74JVjxej5UBk1WevTf9ZTsxXYtFXcKak3vd0YZ55XKTfojwR1vY7QcfkvsU3EQxKxgiLOaWdLnNgojlCe/JAthSCF8P+hrzaWsEDnjH5FgipKOsbJYATFzCwn60gbczLXg/S6+mS/gIF1tyRGCDccyfDWA==; 31:f5Ad6NcSKDNrzWbZFN3z5/T2civ77ptKa0JHKsw2dq+5KPd2YJ+D0dea8I+x2buybrd1F01kzYCaHC/ihDehxzTnvB0nTyDyP10llKP1hSyXkbBzogNrofnXYzqqSTLc6Lok+mQ7VDQ1rE077R3KupOMDbE0PmLqY4rL8lIFuPKbA2qEi7rvsN+Q74ngj//oR4BiZJwuTZww6SHYdVQyBdfLYVawrxWoqXN6xQ8Upz0= X-MS-TrafficTypeDiagnostic: AM0PR05MB4420: X-LD-Processed: a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtAddr X-Microsoft-Exchange-Diagnostics: 1; AM0PR05MB4420; 20:3pg+ws7urzT8N1+yaCN4IuQFQDOGg8Te9W/jimWDeK5hhYq1qBSSn77vnsFmYy1Zbc5hr8s1wPzgG+tz1CJtEE4x6ySXB++j2lacqfpyZ/mmWsB3c5o0Kef0oT+KcBqKEah/Ck8qcezqVjBzmilhd+vzy6tW8O0nyKsA1L/ETa5bDqvmaCGSzMYDnc2TTtneEOI0iOE1aezpF++l9BzpQzHzvniOu0KK/RQliEP4CYMRHbi00j2yEYD714zDm5QZp8G2IM/vf7cvD400BGPPODuOOjsCegnANdrPihcNmJNWMkMkTb0GfkBd8MclFpO493PMI2yh7M66VMXGy2TZe6QHOoKj8aqbEkObv5ExEcyYLRtObNv5gcrN2H6DB5DA01Z9Lo2fm8Q+SbQHyFl1ciykL3REy0oaYHrQvhax5K54gbTW3yNUptAm2haXmJTfXv/8Y2hnBBR3exjbNGrQx+yKX3dqYhf8MdSFykSOchCvYaU1cMxORwt/YkYV6Cx5; 4:54si9kLcumEgh1v/TcyyDnmcKLeupiF3Yg4PKVFR0OnxxsmRY1610Mq9XwWyxjgjj0e2D5FU0EzVAlqpX6wEHuXobMZGWKBWADHzV1mqpPCBEYMLmQLn83BRXvmJ7cz2qcE6ZGeJexbk9uTFQ3d09uOuiBvqIojimH5YfPrsDKM2fOr+m3gGQ8Z1qPkMeOcn6mk5tCiM6PkQ/pHFeKMW2AzCCXKKqEhYhOyvQJE47eAdW9Ykf8ql5EevgkGtxQySQL2cAQ504FI0Zp53e6d4Sw== X-Microsoft-Antispam-PRVS: <AM0PR05MB44201E4CED2DBEF62FD00A11C3BE0@AM0PR05MB4420.eurprd05.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(93006095)(93001095)(3231221)(944501327)(52105095)(3002001)(10201501046)(6055026)(6041310)(20161123558120)(20161123562045)(20161123560045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(6072148)(201708071742011); SRVR:AM0PR05MB4420; BCL:0; PCL:0; RULEID:; SRVR:AM0PR05MB4420; X-Forefront-PRVS: 0638FD5066 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(376002)(366004)(39380400002)(396003)(39860400002)(346002)(199004)(189003)(316002)(53936002)(97736004)(6636002)(6666003)(105586002)(16586007)(2906002)(47776003)(69596002)(106356001)(386003)(186003)(7736002)(8676002)(7696005)(52116002)(66066001)(21086003)(50226002)(55016002)(26005)(476003)(6116002)(4326008)(2616005)(478600001)(25786009)(486006)(956004)(3846002)(36756003)(305945005)(446003)(81166006)(1076002)(50466002)(5660300001)(16526019)(48376002)(51416003)(68736007)(81156014)(11346002)(76176011)(86362001)(59450400001)(8936002); DIR:OUT; SFP:1101; SCL:1; SRVR:AM0PR05MB4420; H:mellanox.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; Received-SPF: None (protection.outlook.com: mellanox.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; AM0PR05MB4420; 23:sjAW3PTlGh/tAby/CKw2EmxmkC/1w4rCMXFqetGio?= u3qEQrnfUX45KQlHr+XFAuqVPtOuQNtPgijbFIWtL4OqFkFfQC8FJ2iTcRWxobkAlo+fLo1DFV81NQ4zv+hd/g0CUYQr2CNEh5at3pyQSG1K61bpX/bAo6baPRJcSW6weDvYEjZpnHBXzJwj1TzlqCYWpa0bdOZrt6PyqW4or2h7rDoYjE7JY8q9TgD/IcG2SlARz4nNDcD6S9N4FUAo0h2ZVVrlOh54cOhayTWJ24oQ25tbh6F32MWxmRd3/GuYHu3kOEtWWPAxhUnTyvKvZk6i1R+lIXPHEfnXH/mvTn4OMB96xtVgZTaBdGnx6uGA7RDxZ8wcj3gvrYxNFz5HhslDSxnA5r3NdHiqhtJPEzWTVvu/hFSJZRc7OWwucAjKGK3o5mV/l2kkORcwxnhrbIfG8xU8xgUOzceQ6K9QhSOdksDNorOtu4RyI8G4NpFlzSeXV5n/dnXsO2x7wQ4LoTBy9IQI1bJ2TWlvfKS+ZEFJfu4jvXXGgRvIHTAGMavFUupqYYn+kHHTHAvZbhITZR6JeHnYxPFx9VhV0W/5P+i+M5eCg4ia40eucLErMnfhG9KJf6SZLQ06NQmoHh0PXtQ7CGpK0M9FmKLjDDAjvUH/UNvBu+ag8EWTOfQ04eTMHj/DCo7gZHEYmGO4U3Q2jkg1cWla/nK6k7TxcFTt3gZJUqEv7EOzEeggdGLvRBNIrZZccGIfaqk0q6wWTS5S8hlCYlkkvlENCOcjV/8l+nerayTdk1OmmCZTEHAre2NgN2/wBl59bFL8X6SVw2CD6knoEZO+CRGiob2+pYcIJoHcBKrV3ao744wab79prQhPszp0VpgpffoIEVa6gGU8icagiaWYTCP0EM6LEX2kvXWokm0UxkwP8Ubyve1aKc2oHrzY+14gql8/hTogfWEH6JLVJSdS3JvechmpI6DLPmbiuycptpjZ1KOM4v+1VoN0bXfLILnt60X+hI830GzsHpZzs07pkvmi2fGtdsriKxPZUb2RFE/4W7hJNMZuButqHg5RmUNbQKkDngvFXB8pzDTgkm9BQWscUXj4u/n7HIMk3UsudZMMG/XwVqJBNLa1nh3eF0JDewhGMtRf27447QdyNycZw3Kr5VOQicxa9KIpcp4zAU9p90SMQOYv0zFCcmw0WIjoh7tP5jSZErTuJgxWNGxWoW5pCS3Kod2Vr3sFbTzwMM6aVYRyg4Vxg1PIVE= X-Microsoft-Antispam-Message-Info: k7uLNT3ValREPGkqJUxvZhEkOuE6Nc5tgxenhsezJA3sVs6/5RTSzz/iHz2aqfdMtJnO7E+yaKPaSdYMtCk5GKk4Jz3+JGq1TmnyMUCTu7NyzMR2nG0c0RE0qAbnqLWdbUfP0DtUnmfZsRoPENGdc02EkLoihUn01Bwwr5V+CaLqTmmm/zkiZyh5gqZoEa9H X-Microsoft-Exchange-Diagnostics: 1; AM0PR05MB4420; 6:SQlCTTiUsOrAg7JjK4l+0Cm43OmiPtULmOUY7au1o2gCZrhozHUjVAY8S89IEIrieY7+v1p71rP1VFvPfB0chlNL0Cc5vEdbMVKmG0A5fjas4RIAIZBRBSap4L0oNNvlfjK4wAyc08ZKG3UYGejLKfclXJ+GAn5XB3D5sSsvvZVIgUAJV/hdd4KiuK06F4QTO5yBowBrLopBtnBOshnYr1rpjm2gGcna+pwUlA8zuFybtRliIYtsfZl4J+1ehW/5tpeYnI/vRzdmJ5Fc49AqNoWIz5q8gwnJ5fmQBragxfNt8dVrf51JuIXlmznwN7RJjO7+EY161iDGn5em0idRT3BPgqav9gIUpq8606/S+RzsWvvbAST5hzJdoxlv+GtEaisyM8yrFoIemDkaMDgR7EbwJyvAscQtTsEObxh9pd9apo+w3aBFKwiGn2mPRAj/k65HCooJRdRv0j3NuhVQ/w==; 5:hpZrKrl8SZl9tAFhqCo1SnLLCHPXPgxq3njYgIrQqkggibNxsUKuVbyH9YKc0zpyX8M4wkkctrGwUjoxqZuiZABOI3Ws1UzMIg+oaVZVj592fiYsOSjtQiCkFDr+QrlcGEd1qMU9lJHigY8h0BBfjG07vxRrFZDw432QC2NOVNc=; 24:DJELzoqrK9wzAjRfIcgVaRGgnpZZDg+zW8A7TUex2oXO8GhOC9mC7yna8BH18hv3tAiKwqTLcqVj3CypAhVtOnbOnfSmMIWgjhXmniib66s= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; AM0PR05MB4420; 7:O5bTZQrXs79IfntlQMHnYpC0ADxC7F92ymnwKAsUEFKtiVR3eB50VgK4ILbM0UAdvmIQc5UAO+XpqrbyeuHSQK3cvVRnVa8gbOyQyloAcQL185sSVw6oGFNht/lRCmBzSkZ0fCVhQBi0lC/6E/M35OTHo6F9mc80BZ57VsJ8VMNWB7saesfyUaEEfqXQH2D9vCJ4n6DSeK82lMguvhMneCXG17vTA0jhZ0j6ykUoFPEBLAOqZc+fixOvM9G+oprf X-MS-Office365-Filtering-Correlation-Id: 2654df45-1ea4-4be5-e78f-08d59eaa3983 X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Apr 2018 06:13:48.4367 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 2654df45-1ea4-4be5-e78f-08d59eaa3983 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR05MB4420 Subject: [dpdk-dev] [PATCH v2] net/mlx5: fix link status initialization X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://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: <https://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 | success | coding style OK |
ci/Intel-compilation | success | Compilation OK |
Commit Message
Shahaf Shuler
April 10, 2018, 6:13 a.m. UTC
Following commit 7ba5320baa32 ("net/mlx5: fix link status behavior") The initial link status is no longer set as part of the port start. When LSC interrupts are enabled, ethdev layer reads the link status directly from the device data instead of using the PMD callback. This may cause application to query the link as down while in fact it was already up before the DPDK application start (and no interrupt to fix it). Fixes: 7ba5320baa32 ("net/mlx5: fix link status behavior") Cc: nelio.laranjeiro@6wind.com Signed-off-by: Shahaf Shuler <shahafs@mellanox.com> Acked-by: Yongseok Koh <yskoh@mellanox.com> --- On v2: - Reworded commit log. - Cleared the wait_to_complete on the link update call. --- drivers/net/mlx5/mlx5.c | 6 ++++++ 1 file changed, 6 insertions(+)
Comments
On Tue, Apr 10, 2018 at 09:13:36AM +0300, Shahaf Shuler wrote: > Following commit 7ba5320baa32 ("net/mlx5: fix link status behavior") > The initial link status is no longer set as part of the port start. > > When LSC interrupts are enabled, ethdev layer reads the link status > directly from the device data instead of using the PMD callback. > This may cause application to query the link as down while in fact it was > already up before the DPDK application start (and no interrupt to fix > it). > > Fixes: 7ba5320baa32 ("net/mlx5: fix link status behavior") > Cc: nelio.laranjeiro@6wind.com > > Signed-off-by: Shahaf Shuler <shahafs@mellanox.com> > Acked-by: Yongseok Koh <yskoh@mellanox.com> > --- > > On v2: > - Reworded commit log. > - Cleared the wait_to_complete on the link update call. > > --- > drivers/net/mlx5/mlx5.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c > index cfab558979..3d222df707 100644 > --- a/drivers/net/mlx5/mlx5.c > +++ b/drivers/net/mlx5/mlx5.c > @@ -991,6 +991,12 @@ mlx5_pci_probe(struct rte_pci_driver *pci_drv __rte_unused, > DRV_LOG(DEBUG, "port %u forcing Ethernet interface up", > eth_dev->data->port_id); > mlx5_set_link_up(eth_dev); > + /* > + * Even though the interrupt handler is not installed yet, > + * interrupts will still trigger on the asyn_fd from > + * Verbs context returned by ibv_open_device(). > + */ > + mlx5_link_update(eth_dev, 0); > /* Store device configuration on private structure. */ > priv->config = config; > continue; > -- > 2.12.0 You should have linked this patch with the ethdev one[1], if the ethdev is refused, this patch won't solve anything. With a reserves of acceptance of [1]: Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com> Thanks, [1] https://dpdk.org/ml/archives/dev/2018-April/096387.html
Tuesday, April 10, 2018 11:18 AM, Nélio Laranjeiro: > Subject: Re: [PATCH v2] net/mlx5: fix link status initialization > > On Tue, Apr 10, 2018 at 09:13:36AM +0300, Shahaf Shuler wrote: > > Following commit 7ba5320baa32 ("net/mlx5: fix link status behavior") > > The initial link status is no longer set as part of the port start. > > > > When LSC interrupts are enabled, ethdev layer reads the link status > > directly from the device data instead of using the PMD callback. > > This may cause application to query the link as down while in fact it > > was already up before the DPDK application start (and no interrupt to > > fix it). > > > > Fixes: 7ba5320baa32 ("net/mlx5: fix link status behavior") > > Cc: nelio.laranjeiro@6wind.com > > > > Signed-off-by: Shahaf Shuler <shahafs@mellanox.com> > > Acked-by: Yongseok Koh <yskoh@mellanox.com> > > --- > > > > On v2: > > - Reworded commit log. > > - Cleared the wait_to_complete on the link update call. > > > > --- > > drivers/net/mlx5/mlx5.c | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c index > > cfab558979..3d222df707 100644 > > --- a/drivers/net/mlx5/mlx5.c > > +++ b/drivers/net/mlx5/mlx5.c > > @@ -991,6 +991,12 @@ mlx5_pci_probe(struct rte_pci_driver *pci_drv > __rte_unused, > > DRV_LOG(DEBUG, "port %u forcing Ethernet interface up", > > eth_dev->data->port_id); > > mlx5_set_link_up(eth_dev); > > + /* > > + * Even though the interrupt handler is not installed yet, > > + * interrupts will still trigger on the asyn_fd from > > + * Verbs context returned by ibv_open_device(). > > + */ > > + mlx5_link_update(eth_dev, 0); > > /* Store device configuration on private structure. */ > > priv->config = config; > > continue; > > -- > > 2.12.0 > > You should have linked this patch with the ethdev one[1], if the ethdev is > refused, this patch won't solve anything. > > With a reserves of acceptance of [1]: > Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com> As the ethdev pathc applied -> Applied to next-net-mlx, thanks
diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c index cfab558979..3d222df707 100644 --- a/drivers/net/mlx5/mlx5.c +++ b/drivers/net/mlx5/mlx5.c @@ -991,6 +991,12 @@ mlx5_pci_probe(struct rte_pci_driver *pci_drv __rte_unused, DRV_LOG(DEBUG, "port %u forcing Ethernet interface up", eth_dev->data->port_id); mlx5_set_link_up(eth_dev); + /* + * Even though the interrupt handler is not installed yet, + * interrupts will still trigger on the asyn_fd from + * Verbs context returned by ibv_open_device(). + */ + mlx5_link_update(eth_dev, 0); /* Store device configuration on private structure. */ priv->config = config; continue;