From patchwork Tue May 30 01:02:59 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yongseok Koh X-Patchwork-Id: 24876 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id A8BD82A5B; Tue, 30 May 2017 03:03:22 +0200 (CEST) Received: from EUR03-AM5-obe.outbound.protection.outlook.com (mail-eopbgr30073.outbound.protection.outlook.com [40.107.3.73]) by dpdk.org (Postfix) with ESMTP id AD6A12647; Tue, 30 May 2017 03:03:20 +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=GSaAnrh9GlwpdVIiDTY8/HO3wEC0Zz5wH/A7T+g30Os=; b=pPP/l+UFQ5DZBkLckC3wvSjiEAwZJ8wUrFSQNlLHn/pQVzKldRJsYyClbHYfuwQ04kg8SzzzZyZAhz1P6HTaXiyj3Gfox7GjCfUeNOu3x4bLUp/O3koLVzCz47Bajl3YfP4D88BKwaQ6ZODAllqBNRGwvi+CT2iEvCBuhS6u7IQ= Authentication-Results: intel.com; dkim=none (message not signed) header.d=none; intel.com; dmarc=none action=none header.from=mellanox.com; Received: from mellanox.com (12.250.235.110) by DB6PR0501MB2039.eurprd05.prod.outlook.com (2603:10a6:4:6::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1124.9; Tue, 30 May 2017 01:03:15 +0000 From: Yongseok Koh To: CC: , , , Yongseok Koh , Date: Mon, 29 May 2017 18:02:59 -0700 Message-ID: <20170530010259.48823-1-yskoh@mellanox.com> X-Mailer: git-send-email 2.11.0 MIME-Version: 1.0 X-Originating-IP: [12.250.235.110] X-ClientProxiedBy: CY4PR1001CA0008.namprd10.prod.outlook.com (2603:10b6:910:3d::21) To DB6PR0501MB2039.eurprd05.prod.outlook.com (2603:10a6:4:6::21) X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DB6PR0501MB2039: X-MS-Office365-Filtering-Correlation-Id: c77bfb70-5868-4093-883c-08d4a6f7a80e X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(48565401081)(201703131423075)(201703031133081); SRVR:DB6PR0501MB2039; X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2039; 3:3yYi5fCXbQ6ADHkLGtFj0RhCgtZhOZNcEpewxRQbgM0S9fFh9BlOXdSf8UBRA15MDds7E3uAWVH9GjkGrUcfqzAlr7uSTwDOdUriaQNHcHxGTANBGfVT31LBCNJOvN/t89TPb2f0tMmdjCrL7Y4si3Atoa2qVObpr79Fia1l9aJYX80BA9WTiJOpG2FUcrtPQltFJf4FNnezpz4vaxbrMuBD4p5UkNyQ6wueb3Re6TzUkR3m+t/CR0HZab7IIG+7YNSU3S4wVF/vIs+kF+cmgPhrpeN/YTq+n2WNB7qqwZTdxn2bu4AqNpJYVje4iClkjaP8dpvkHJRO8SPYMKVP4w6qX+eR4k7M9SDrI+5qZlo=; 25:xVHT20VBGOSbPjzH3q+Qo+Ueoh0/qkop9+FSfVEUsceiIf+88VYSEsUI7BK3LnQIsNsfG1c4C0sBYL0TVFN9wSCHFW0qkexNGT11HpvYdxwLlURgC1gStXBSg6JYkfdhoHnvcOqhNhkINhQfpygNqGyBy4DttJwavKiLn65lcl6epQSUfj7xEO/3C7mWKRZ5yLoaxWy2Lhuao4HleEP44ZsnYB0WbM8RjamO+KXCDt4Wes4abCgWS8YZ4wioNNl83Vp96ZGcTfAQnnQxLGXUbzArTgJP2RGoDrqju6xneAREkoQHEz5R7Xnbn0SrlEyvHG4r2n2y+FFUJ8630ufwvoFVyb8JkN+RAzLCB0yplvznvp1fhbKF260NR+ol2aXl/fQ6P5u1SRtj3b3esInNIxTFFXHEutV7M8FuWpcpz4LhpifLpirsIFXh4TttU/30cfJJdrKCCzWILlIV5535gH+ybPePk9GMy4M0RbzXrFc= X-LD-Processed: a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtAddr X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2039; 31:YrCCaXX09v8cvAeeNDyJYbWkcb2r5D550aCyaZb5NdUZi1eOYAR26EVXBlk8+OWYlu2RJl5AwapTO1sYj8uJr3HWK/dGBglX9afmFOn+HXE6cb6prUCS98IWcYodyAXY0lZv9aGXqQoTf/rV8gnQG/LtrWM39/iCVhQj2s6oWaI9iheDoFfqaeMKleVcQa6//YKVRWQvcKpT3fPrGd4rqWRUDEoVCBiWEowUt+mIcOk=; 20:3Xb4wIcJk2KslfH9ConWe39eGGzaZ7/YOZMyUWf7EEYBpAXgsUtOll0LOD4BrnTRsr0ItAJ6TMSY1F+kHf9KmL7qsy9Rcjr5LbAJn0NzPx2c69bJKtYF/V5rMTH4B/Kmu2Pm87wX4MzI1cJvq0+XfdOKSWeN3ZT6MypFItjrWo9ZpEnlZiW6aAj2V1LdONrqpQNNK++tATd3lZLF9Qbxm1cPQuF+G+mkJ18CJ896UkW0QIDEkfe7SbUpbs1i43dh4o4idX9qs9+NSJcpNIA8Of/HIqKqdaIN3YQXvQ1J9af1f5blfU2ACx/d1Le4aaV1oQwkGE5KE8Vu28OArLBbxL05pv+OEgVvsqqbIHECp3Pc4a96ozOzctJ4NKwT33SRdcCg8o2Z72Cy2Me6nER5yWp/hL6Mr9DujfhkVH8Cr54xqDhQWzJmOTC67XVO620yUV3ibFXaX5yCq3fkL9qARjnwvLYlMjZwEIrgb9Tjsfkeu6NQH2Fg+iRGEgW9bCLH X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700073)(100105000095)(100000701073)(100105300095)(100000702073)(100105100095)(6040450)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(100000703073)(100105400095)(93006095)(93001095)(6055026)(6041248)(20161123555025)(20161123560025)(20161123562025)(20161123564025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(6072148)(100000704073)(100105200095)(100000705073)(100105500095); SRVR:DB6PR0501MB2039; BCL:0; PCL:0; RULEID:(100000800073)(100110000095)(100000801073)(100110300095)(100000802073)(100110100095)(100000803073)(100110400095)(100000804073)(100110200095)(100000805073)(100110500095); SRVR:DB6PR0501MB2039; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; DB6PR0501MB2039; 4:J7RXEaTzXHWRafzIyDykEQJTUIprVRl4ZwrR5ht9?= UmSWo7OlBzpI/amD13izTkttR2UqaoOi5jkIlTpQGxIzjEsQizBqq5rxF8PXBw0ML84QOQZrgfWaM0IzTnokfr3Nb3ZXXG2P55i/ZdHo5aEJw/Z9IvFVHzFufrx1liJSytIY4x/uR9NpaWlDWYRPqxqQtxUCAgxJ1+WxPx8tcs8B5/C9i3rppzmfyeKHW+SUZ45uAsqzpqDIcWX/oaaQOUrR7NpgzLV6nxef/gd9dufKqxRvXsZPLG0f+CCS5vEQ09IewpWfHTRVClCx+ZH1h244+hAP2Id9/pjR2z+rP/UNs3LIy/OCJR0HxBS38itArG1f/+nZpJMbh7tfrIXqkRtazEbleHhPQuwowZ0CqKOX0DGGY1O/bmxB6JhvgCN9zxn9rbkMmNj4NeQ5eThrlYfBdMbVh8T4wnHAPcRjlFlxJokreN+75A6kiXjH6d38eYBTdZlojCR5FbrnLHDsDMCwAwcTMU2LujATVx20FH2+QHgtZADfYzqXGt2jrVb/mk1z4oEEGkajnj+P/r9R7DAmZZKsSBeuTBiDohAJ2SnO3Pa+Pcn7IQ/bjotkRLekHG9WOUSc5Ra0tK5q2LVU9OeFsLknhdpwW3C8LxT4UfLPqCGXxS/qUMIpSt4zFxRn2bxcexCe4d8lB0igKYsGhYfv6x9M+QWcosM9WZU/pfDCk5UawBAT1ftIEqwI2AYS7B7tnRUzj+Q4PEVsaJxkTwdMaQy5aZ6+YT9bqkMl7+0PXFotyzmN7Gy8nW/9J/e7fIGDLCblI6n89DaDcX00YswD2VEsVnpTbcMZ7HoSADInNlfxdTB5QGMLIzcXroTARNlNUKLoNMfP4UkOk1y1qFT9tzJf5jgbs7u7maY31unCfpDTQVQB32OHSDszSrBPFZceNiVygoSKVKOxE8XlPX60njBkeAxY7aWzw6tuTkOkCHkegEdnAuqz8wNzMaLzROoCXHCQarXAgZXPpzRpPEm4joLOZlyA6eGvwdojuTmKMaSFG7mLtgafuhrbyUymK9mC5Kegbd8Ay16zDDIagl2JUByVKnE78US/4sICv93COwI9a0ZX+LIbvEh8Ey7ZVh8/8Zjm9dET/e1RHG3m3kTuaqOkTHiEWX/cSlINGuFKsg== X-Forefront-PRVS: 032334F434 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6009001)(39410400002)(39400400002)(39850400002)(39860400002)(39840400002)(39450400003)(478600001)(6666003)(36756003)(2351001)(54906002)(42186005)(55016002)(33646002)(6916009)(189998001)(50986999)(38730400002)(110136004)(33026002)(53936002)(5660300001)(305945005)(3846002)(25786009)(1076002)(2906002)(50466002)(47776003)(48376002)(81166006)(8676002)(66066001)(7736002)(5003940100001)(86362001)(50226002)(6116002); DIR:OUT; SFP:1101; SCL:1; SRVR:DB6PR0501MB2039; H:mellanox.com; FPR:; SPF:None; MLV:sfv; LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; DB6PR0501MB2039; 23:f5vmISiW6nA738UHsPHiSibjEOurbbbBcgwjq2W?= ZjDgRXRaitC58j+4Zn6UYwk/MCGEJanm3N2Mj35u3Tamr4Y3UgiZDJt/+PrCxeZizLSs/dMARcpg8XEFnMOOJY+eI2hcXqemcUfsgdBlQGQwQZ2VhOTOYrlOsUuXBZ5KkCDCmYkhHI4xCNC1sCD0cMo/0amzbRjaSeFt/erdR35ogsdh4Ndv1bWWs+dvm20d0hKjK1nDoNkcyA4aWYxlSx0+Cw9zqA0i6DQdC4RcPdJRiKf/ZY/trU7ndnNQJivDzhNLiFRRoPSWIQNUlN6UV0ETWfw9Ht7V4TpjsJnsTHfmP44p0Q91W3CM9PkTEYjsMemmVwvYapxA+a5gYq/NfyO5fBIAt5ieSo/6rkvFd2YhCZ5tZS9Xq9RbSWd/wOSWFtJUmxiGcCaqDNgeL8vMK5rqp6FasKI1hteEjq7kiK7jOrc+cVbBBEBD9i8f+kRs97So2M2eJAOmCMlvC6lg+ADYpvhAQa1aZmIxff8qXWMPhaBUvHkNYgFqvvMc/5dVv5Q0ANpB502slQUuj+96r/N6TQK2PuEngVYB+e/uG0+4mu8ViWRm2nJ6HofAMX9QBmB77kZ/L9fQKGbucCV4ggRRVgah438MH7ulN4b7vBZeAEXfK2dDAvxOFSgUZo14gI3aoaTe1x0u3gkERyjnSHgWtDdfndEBgMUFXkELWv0ifkWp4Hfjk/5A2CQqsdAxGxw8ao97cxhg1qs4LrxKgwIBcY6C4ytS20gQTOPfwRYiUwi6lpPXmWZmbpLkPYZUXwzPSkf5UakvxathyR/TC1Lmqulyn/Zk9FONfZSPso61O2NSbp254LrDqczL/HURRvZG1fvgBWOIKS5IrMlen92EVEEUdneRPLncBZmhFaiMjj679kcFrYqF0zoDZ4Yl5+HAvm7XvT7+gwoDBjAShs3JgmllNdEKbCbA0ltl7iqL2nLQOBJBLh+fBzKuPpr7yjPpwF5NRaqMhZ0G3i78OWkLazi4Sp3yZuo20XcDrdIYoiw== X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2039; 6:HXIUVQ2f23BSxxcsUr8TKiOrNZFz8H0xz3nmoW8l+yK9RQyBC/ZC6EJfel7uEhQOq2/Q7LMrZx4etGVhInYiFV2faS5hHNSj1/GecFvyPraHB3DYbSe+5vnfjr2o5yBkIH/D8Tfwgqv8+4AcbZY7quv0E2TZU6D6dcaTh/JuO0lJ6VaHQePcoX4ByH/E/5KNxT6KWkiXN97+eKzxmHJSzr5nfiDklgRPFpwG9+e2JgT9xb0/3BXpIcDkJ9+rUcPiTgb2AXEZnDJPZfhFp6ournZ4uAb2VhGN58PPcRuD/+HtTf+deUbD4+O3G4AQLiM0wcHH2ZAxIHORFqOiXfqHLKaj9FFvmS9pHrA12jdNRbOy9Rbvv3zPFBbXRaTdYcOL0DLkn8WROka3M3NdwxxlaKgVR3W+HMDP9PY7CCSOuvMpVzXdd/pc1bKidrMg0teqeq5HyxEmoexR7ngXpHJMZXRtSUb7v02G3iH3+tcABJL/FUsN7fgnTSGgWAifCDv5xgHx/9tO6SSCZFQZplZKMVyGqEEKxlU3R4ol9reLQ4a+cdSEsHv4gMKNOS3nauXbGbqfuEJZjuNHO4i9cKfTdA== X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2039; 5:rI6EnI7GDSVYQV0/8AI/9QSbt91+1UncW6hoX9mB2K8I0/0SQb/DiuB42RYE5yKKUjJdCvXc6kIgmkloOoHfTd3CCC4UmNLTl3R642XQEwmNEjYo6zwy3dGZHvNLL1NywnECHlUr+1g3Q/cs1dkgVZi7bLIrJkB4InnT8STs2tG+msR0rQ+rBsO6uzaTPWl1R0uG+uyr/zszmf5vXsCuqZd9oiGcqPt8emQL8ZDooybvnhL3mboV4JogyMDtWrHvP1701euJszSEkMKpwLrwno+yLiVh2IE9IdhZEnfhHdaeTGr2VMJalBOJhJUtsl6gaeC3JNYwKOT4kNhAz2g7k8xvCCo5JJWhiLPzX0TQtumoZtqbe+/mXtDHXOCIuCGdTUvbjB2C2x63sWJ0RENoTvzN7kZksnEBxsWYSXhJFUSPnQ4U1yHBvUGqIJKAyP3G7310ggb4iaVDJ9/C9uZrJ1FEjXoGKZTCi16wh6MiKTsAkHV/ruGuly81ityxS28V; 24:P3VconZ4PC+oahd4+3xKGkPw3a82VMhtNaOCJ5ne87x4oVetlr9tShCWwuXFVAs/8FbbvEP7nO3LH7bLzxwwVk0RXLSqGTUC4rMBi/f2avE= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2039; 7:EOuj+A+tYFRuG2p8w+2IRBuZ0YYuSN07+kcip4V5+EqPlZG3J1DPdnUDta+qX+pdv+2o0lpDpWdB81bBADwYWKWhVft5kTUnMFGfDXfqySwp8iDFVD9P51wJF43fLvrZHvj6LkNn05Fv5MIPGRhubNu4KLo4m/tFquuLR2rEB8LDbZjE2/kZ3u3XV8WmR3g9wYTO1nL5/boqtCVF4lryyqcmz6Utpi1Gh5cPWWQBrVhhVLjM/2JlT2hU05kALL+OLeH5zQvBP/AX3kZzTkfYsvDybp5uvdOMYoqy8vMNbQcSn8HzyFOWd8UY24y5u2Y+99iljbw0QxV1jsWHW0iZXQ== X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 May 2017 01:03:15.9302 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0501MB2039 Subject: [dpdk-dev] [PATCH] net/mlx5: fix wrong exception handling X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" A sanity check is required in priv_fdir_disable(). If resizing Rx queue fails, this can cause a crash by referencing a NULL pointer. Cc: stable@dpdk.org Fixes: 76f5c99e6840 ("mlx5: support flow director") Fixes: 0cdddf4d0626 ("net/mlx5: split Rx queue structure") Signed-off-by: Yongseok Koh Acked-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_fdir.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/net/mlx5/mlx5_fdir.c b/drivers/net/mlx5/mlx5_fdir.c index f80c58b4d..1cff41bab 100644 --- a/drivers/net/mlx5/mlx5_fdir.c +++ b/drivers/net/mlx5/mlx5_fdir.c @@ -733,9 +733,11 @@ priv_fdir_disable(struct priv *priv) /* Destroy flow director context in each RX queue. */ for (i = 0; (i != priv->rxqs_n); i++) { - struct rxq_ctrl *rxq_ctrl = - container_of((*priv->rxqs)[i], struct rxq_ctrl, rxq); + struct rxq_ctrl *rxq_ctrl; + if (!(*priv->rxqs)[i]) + continue; + rxq_ctrl = container_of((*priv->rxqs)[i], struct rxq_ctrl, rxq); if (!rxq_ctrl->fdir_queue) continue; priv_fdir_queue_destroy(priv, rxq_ctrl->fdir_queue);