Message ID | 1496835418-22723-1-git-send-email-markb@mellanox.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show
Return-Path: <netdev-owner@vger.kernel.org> X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3wjRNm2PQ5z9s78 for <patchwork-incoming@ozlabs.org>; Wed, 7 Jun 2017 21:37:12 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=Mellanox.com header.i=@Mellanox.com header.b="pwziCsd9"; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751523AbdFGLhK (ORCPT <rfc822;patchwork-incoming@ozlabs.org>); Wed, 7 Jun 2017 07:37:10 -0400 Received: from mail-eopbgr40089.outbound.protection.outlook.com ([40.107.4.89]:28000 "EHLO EUR03-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751476AbdFGLhH (ORCPT <rfc822;netdev@vger.kernel.org>); Wed, 7 Jun 2017 07:37:07 -0400 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=freFVvfi6D9HLk7QbdPbIobNc974RFAdP8Cw1Yiv84Y=; b=pwziCsd90R8TRTP8zW73gXgjxTV/64dm/cgGdCxQPmZurDG3En32G3YdU9nyxy7m/Um1DMUrsOJ+1nhvxjvrY02itBSfBP22RXLw4npkY4QctEgEDOqWWysbmSd5b/WJUMd3Yk9bkZftQoNR/J4pFiz+JWk81fI27yn2VpsMfZg= Authentication-Results: redhat.com; dkim=none (message not signed) header.d=none; redhat.com; dmarc=none action=none header.from=mellanox.com; Received: from r-vnc06.mtr.labs.mlnx (37.142.13.130) by HE1PR05MB1337.eurprd05.prod.outlook.com (2a01:111:e400:588a::30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1143.10; Wed, 7 Jun 2017 11:37:04 +0000 From: Mark Bloch <markb@mellanox.com> To: jbenc@redhat.com, stephen@networkplumber.org Cc: netdev@vger.kernel.org Subject: [net-next] vxlan: use a more suitable function when assigning NULL Date: Wed, 7 Jun 2017 14:36:58 +0300 Message-Id: <1496835418-22723-1-git-send-email-markb@mellanox.com> X-Mailer: git-send-email 1.8.4.3 MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [37.142.13.130] X-ClientProxiedBy: VI1P190CA0002.EURP190.PROD.OUTLOOK.COM (2603:10a6:802:2b::15) To HE1PR05MB1337.eurprd05.prod.outlook.com (2a01:111:e400:588a::30) X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: HE1PR05MB1337: X-MS-Office365-Filtering-Correlation-Id: 4106918c-fbb4-46a0-3167-08d4ad998590 X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(48565401081)(201703131423075)(201703031133081); SRVR:HE1PR05MB1337; X-Microsoft-Exchange-Diagnostics: 1; HE1PR05MB1337; 3:cIF9UEfGV2PbP+rxnzH9F3JysBzkOJDG9jsXiM4Z+sQr8oNH5JMt9XKMUcW08bAhFL1XeNu+s/WvvX1gMAO7sx7uTlAZZGPlqOfUoOoma6G9NAGrDUnv+fkChg+3DPESwgbABd3texIs76DW7MZ3lCvYTr/KYNQg/CWi21ejxMV3h7Yw+qhXGY0emxrall7QIbyC+1Ul6xoSKL6Y4a0c/AEMnR46zBVJNEPafJhd30lwL7ZXGZcpVzjRnWm7AHKyhxZ2JTZ6BwJvleoXIEL3qnl2FxfVDVizidGi+MIbF1lXrVmgd+VVEcRfvNUKgd+iu53ti5XpOBMkr9Lke1lIWfie8OqTuB2X+BhzjMCWfRE=; 25:EB+iAmQpYpzjpkbNJS8nXiX/Id2Bly4atlWeaBU/D5y1veIBGy6Y7UOUTXv3wFRZLdCjxfdmfw+1Ien2+cC15mez6MLhi+KU5aLtHQgMj+vNQrqwdy5SFJAnTItHSiNJ0yNgD9UoGWEmOZfgbTDlNRclrQOBoVef2lQljpYdsDnycmg5dA12zGxh088aeKNqF2gQSoCrw1CtKHRnSZHGi7HjJTXyZEwrV6oD8/7xaG4Ajmz6odX/kzxhqN/RzxKCteVNRrdNctlIoum7lekls3BVUI7X6aYbLLl+tbvPOdvwDj0n7dR4fADS9N5G2E48Pwy664RQZpqvPmGQXGFMB4x6t8SnM0e3fL2E4skeBtB0JX6ki0x80iAY5asXsiZehpldQrTH9b4m/5EZIE2aABDAWTAL12YmPdhEo37q3RSVIqcuVbBzHe0aYW7I6x9t2NVLkrezQaCRsv34GJ5qRNV9ZX7ZJrY+Bmj7N6H8CVs= X-Microsoft-Exchange-Diagnostics: 1; HE1PR05MB1337; 31:X/mDRKTHFUCnh80LtIPupJpF7578wWaLZA7Yf/u7byHGbZw5O6owalg6/3Ee8KIXrHcU15gKRnFDfnsIglCBUukLN2jlADikrxAnspZtum4Pm0wkpvSbe4DeuBTLxWGCQxuz6oWOjb0455hAsXi/+LCj2i92Fij1XR3uduq7tM3Lpx6EP80d0IBoV0+8g8h+HsvKGpxuTw0QdwPAolA6iE+rpu4Lwpui23MFK5REnLmtgrXJnov8C+4npnXJ60dNTR/MwrtL0MohfrxueI0FJg==; 20:/OguK/ja5v0F3IQPf8Ch1Usp4S6M5NkShPV9ezIkl8J4fx7Su0hi3ee+5RaYZZFMQP9EJW18X9Grt0X5RSLEdQGqiW7udyjzuDXY0iQUicxXmPHcuypQhW25c2wnM+OyQncw+bMXp3YmCI3EGAxNz3jKHafY80jaQarnT88kwV8FI1C8Dv2porBvixGyLG2JVjZ4FBSROHfwzN392/JiIG350b54KtVFVx+X3y51CndaHKHkHUlkO5o4snFeiUF4EA2q35cd3R+qt40TaXD1Ezlu9fJ2R9XCpu8pA3MX0q1c2ziJD+ecjTO3rerZ9FcPp1Yatt9cPXoZS7uXfOLQ/pLVysHtEZdY8PBKKdk6BCwcC3jU4VLVnFRznVT5ElPXtD54KHI/gEZcpuYfvsnPY3XrNPiUAVTShU4S+zvfEiifAjyEv6R6Kzmu68XAYqhCnLp09KSpCnf4GV0aGUALlL7+MKG6flGNW2R2QHNDCIKIoVRAcqYVySd0/ExgqKza X-Microsoft-Antispam-PRVS: <HE1PR05MB13370EB22C36BD55E4F1D617D2C80@HE1PR05MB1337.eurprd05.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(8121501046)(5005006)(93006095)(93001095)(10201501046)(3002001)(100000703101)(100105400095)(6055026)(6041248)(20161123560025)(20161123562025)(20161123564025)(20161123555025)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(6072148)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:HE1PR05MB1337; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:HE1PR05MB1337; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; HE1PR05MB1337; 4:FlR2lGvqr2+vD5NPdPXm6W7sxtrUA3bruSWZf7X9Wz?= =?us-ascii?Q?apigFHg8j4NuQfToe6sZMdKWdWzB5lepyT3urgc3GfL+Z1rtKKEx13ubEnDc?= =?us-ascii?Q?d2LO286qmlGn3b18iOU63UWvgA43s2UTh22ipAD2L9Zbjnccfg8Oy8tO6Kyo?= =?us-ascii?Q?1vA4bxoep69Ro3buzQC1hVcQNtJS+8VqQvWG4nYKGtZMM3WoC7P+qUnFwb2o?= =?us-ascii?Q?H8R5NfS0yJThwTwGS6739lZwPqjYL0PA4WXpzNSgqJgRTJLIMQb+j42gUEiI?= =?us-ascii?Q?eCFy2MfNvbpzZ07xzeaZ6uOIjDCx3DBz3Uur7pBySD5Tf0YGjD95S6MTlfrw?= =?us-ascii?Q?GYE6dfeSNbj5CW3W7R4KoFnL6SvmDp9U57Yb4xhJTAESO22QzXXBx+tPK4xh?= =?us-ascii?Q?DdOci3ZVZLOQ5YR2dY9oN7NnNAbdSgIuDXAKSrkKSQ04oh1l3bcF5lehx7wf?= =?us-ascii?Q?HxWQ33X02s1LEuYLlClgitbqP8Sx+KQFmI4D9RTgwiuN+A6VLDpwQCPEMksD?= =?us-ascii?Q?Uzp+s0v7vtAb/bvonegG0kObADVBiF1np0azNjrfXF87XoOdmdiILHele/BZ?= =?us-ascii?Q?vacNBmt+ii932gFzz4X0HN9LSNPfg4uOVxoQwKbLf+F0UTeyuF34Iuzqqdtz?= =?us-ascii?Q?Zp4QJ1BUGnv/j1P1jNT+lr8BCjd2PgRNPERl77vga6Xqxt7vUg1xC5mGHsej?= =?us-ascii?Q?dyOJJJZDO0wiwKjSmogRWOfspU9UO84ziYFUBzf/lLBkVwQQC1NuB82zOZBQ?= =?us-ascii?Q?4LhO6w++XBPg2xUmgNFute18Z2ecXFhaxLI7HS5fDrrjfl1bNtSDJx8yPXXR?= =?us-ascii?Q?3ZzuTjyzDM3zawr2tpfaOGFbsovT4Ln8aDmJ2mI8As9JlUhlzZXd0Zct1Hj+?= =?us-ascii?Q?qTu/OV2+9uh7B5vEsmRT7aGtTI7sLvG9LjTxdSFW0Nxaq8mz3vR/Pt2XXjPK?= =?us-ascii?Q?y9KZ8vuHdscMhYkNrJOxcb/+bf1EL1EWwkiLPSdKME6Ox8qS6+DK/aPrUcDq?= =?us-ascii?Q?xdQkltHQ59YlxpmEvw1gTmzPwsMNi6DocAHD+eJrGJdQTyMiosP16/u6qNOY?= =?us-ascii?Q?foSV2TMT/q1VrFBtuLJv5vyFpFF2ajBhns/kxUqZRFHN8k2r4Fzuxtx/10UU?= =?us-ascii?Q?7wcuH6YwGceEdiv+VoKfxiiItPxt9nXAd5gAor2EI/tftdbERQ7w=3D=3D?= X-Forefront-PRVS: 03319F6FEF X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6009001)(39840400002)(39860400002)(39850400002)(39450400003)(39400400002)(39410400002)(86362001)(189998001)(2906002)(50986999)(305945005)(7736002)(47776003)(5660300001)(42186005)(50226002)(48376002)(50466002)(66066001)(6486002)(5003940100001)(6512007)(478600001)(53936002)(6506006)(110136004)(4326008)(38730400002)(6116002)(3846002)(33646002)(8676002)(36756003)(25786009)(81166006)(6666003); DIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR05MB1337; H:r-vnc06.mtr.labs.mlnx; FPR:; SPF:None; MLV:sfv; LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; HE1PR05MB1337; 23:a5w7mQDaemHk2a/HtZrityLABP3QHhCILU3t8WNRk?= =?us-ascii?Q?y1qAiYhW3aylVEdLX1jzN+y+92FNiMks9MnVt7j+zBh+/JowOW/Zm+xxh3S6?= =?us-ascii?Q?RxRvu0a2J6SENVJwWTH2AYsbnMGZVVIBHSdQNJdc+GENs9jen8y+Bdczirmz?= =?us-ascii?Q?49t1pGwvzwyR1Ay2/9HiA9KH6MwpBkW2c/yEHeGU7btiG9kfXyl8NZpJvXVi?= =?us-ascii?Q?/f0J7F6KPulmBXZupENLj9cx40de/RWaU0IwK5X4hPoWv5I5GPjdZQm1cjT5?= =?us-ascii?Q?gwYnt9gTi1rsi7IW3oesKbhB6S+Lu7z/zd2UieuSkz6QZxEKr1qMvaZhKSB8?= =?us-ascii?Q?HvNJtKbwZ3JY/OffX97r36QurV9QU1l7BSuzG046CV9hFwJJ1iDugTPjK88j?= =?us-ascii?Q?gRMkvp295TX+li8QqmqUbQoH9u8ZQEJ/Mx72MWweL8Heeyhcz/Jt/FaDb0J0?= =?us-ascii?Q?vclashislJ4Dh7O3pGMIrS7ouObiCkS3+qpp18jaLC6q1z+xEQm5YLdGr+Dg?= =?us-ascii?Q?779KT+y4DmX76bzRu6f2do/Hwxr4OKsdiAFUjZxhOLGopb5W6qibJonLIsHA?= =?us-ascii?Q?WxYC73SoCQE/TQp8daq0CHVCyCxgdrm0K3PVYLjcWaFBGTgE/wU5AZ1u9ISs?= =?us-ascii?Q?lf4wFQl87qzSyyfwLbuiRBfqwX9e0W2hALohmD8e0i2EnMDst7vmySl3UZ31?= =?us-ascii?Q?j4GLKAnODuxWrs/kUA+OFnQOl3z5sr3sNu4N3AYFpvtDoOMwSMH3mvK3YFag?= =?us-ascii?Q?RR+2Izgoyfm5THI6y+usnrHlGVXiayt6t4I/VbtKF+88XpaK/IE7G4Y4L9Qo?= =?us-ascii?Q?bzFv1wE3Zg8XvmbUEsVMblsbXyPly5JZD+ohq+v9e5gQKEzfQFiMTTOCWIkO?= =?us-ascii?Q?+uDj/1qbfCi5Sv2JUIe+jocMzRqb3EfeSB1/d9WYhlSnDtucRLMdkNpd+qnT?= =?us-ascii?Q?lrWmYywx2XruLRuhhrkYVDUsSde4pxwNr8/GX6xITy5LAPSmO5NdATo/ewbL?= =?us-ascii?Q?Fc=3D?= X-Microsoft-Exchange-Diagnostics: 1; HE1PR05MB1337; 6:lFnqngL7/w3Q8lCSdQmGUaLwNvFRdX2WvxnkjAuG2i9bFesvPHZZA8sGLiAtB5+uJC7FR8gH3H7pcVTjwaHieQp1Ao/WkMLxn5rBI/6luxIL+TwkN4oDENY8r6E/wa8yJS6C6uUVgDOMYGQos/e8Ae1QuH/n1pg7iCkHmveal3+lX0mMolr1H/tU25oPkJMLlw6H3lP/FxmJyEgNIVCaTcptTWo+t2d86ruufNeOzBnR2AjEqaraK6vf9Hqj/Gat3YwKmw3iiR7RX3ApRrvE+AWnRIefvY6zVYoCjrK72n/996gk4xeDPq1RFMz3RJGWz3Qmgs5Sv92oKmBs0hUtsa5nkI97x06YFyq7RJi434cs0/ybXxMm5YzCY9dXAIsbRTisgNUzG/tWtbXFM2AiVboPG/qzFZ/mwQ/mTmn/kN1roXwsKPsrkjohQrHU3h7yvN3TqpEy7yQ57aktsmDv52lDQFFm+vY3Q1tD/wW+zlORFrLq2ppRwDwA7unP3gblm/NeI+c0cpji918+HfiMrtf0vN3C+p9Gh2aHeGVTyq8= X-Microsoft-Exchange-Diagnostics: 1; HE1PR05MB1337; 5:/68bX0dl7qg9+/F0Di8KgeF53XTeQ60fZNL4xGS1uVsWpkwKMuv1QiniT+cfonkPy51BIlU7CvnN6v8a8j9fzOrl5bWv8o0iD4ESH5OFXIDKGBakw/mpR1vIvZnwAoxYA4vRH78GW5kL7ipON4XHJFpS/VvshdGkb4WO3ZieV9aoWJmD0CTaxy0DSc9Gd+0701Ui0mdp+kCH/SRIV/2Wfy5gVuKIx4+EZCq4pqOzW6vP7URcxvNvpwyhCepkZIUAHVMq4CMSgxqontbZlKLXc0F0KcuDiG6S1x0soo65lGn9qfrrnhgsz/7xLhTQlZGPp5UsWL80RThtkIyRgVgEuuUEUFIGvssiBKCgYo/5AuTC1vS21T/XBICbfgtTcCRc24Z2FCgx5KgqiKndsI1Zw5Wsz7flbbJLQKeMjbiGU12kMDOg5rtPzEb4GFFM8Qg1G1BLAIagy0VHL5Xsun2T1n+cmtyrQbg28acjhzihMxXdElGtv8YTBXnPFCmGCl9+; 24:neDAvCDM7T8q7iYfNrTZ+HXf4+DUN0hlcF1IZylV298ZKdQg+PMh/f10glLnIzAv4K73ExKKUUq+f1mmQnSlEgqraD8XuIP8axgK5G3NeL0= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; HE1PR05MB1337; 7:SW9PMfdxFKU2dW2HQ8SrdegwGEXmRXamJTsrArRKA6nPl5clYjA8p7S5dI7zgmJsBA9amXEz8uKguD/CBBjCiZeON1zhbgYZaxw3qMSw7uDhYSgBIqnvgQXJU12IipzwjlgBJ7DoQyqSBCaGhUTMLu2N0FQXTbZsgKKVyAVqSsz9vcZwGY9ISyrHQZ9m2FEM4ya3nGNLBmQ5Wq/IBNI1BPMpbnsfcOSvqOJIwjEIjvIrMHlrroyg90tSYybCHV5N5FElMbBVHJPsQ+7J3vsvhROLLo/oYVJ7dj7hRog6rZ2b89Pds7/xi6GfJLwWFwkMasA8cjMYjcQnsKvgfUsRhg== X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Jun 2017 11:37:04.0806 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR05MB1337 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: <netdev.vger.kernel.org> X-Mailing-List: netdev@vger.kernel.org |
From: Mark Bloch <markb@mellanox.com> Date: Wed, 7 Jun 2017 14:36:58 +0300 > When stopping the vxlan interface we detach it from the socket. > Use RCU_INIT_POINTER() and not rcu_assign_pointer() to do so. > > Suggested-by: Stephen Hemminger <stephen@networkplumber.org> > Signed-off-by: Mark Bloch <markb@mellanox.com> Applied, thanks Mark.
diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c index a6b5052..7cb21a0 100644 --- a/drivers/net/vxlan.c +++ b/drivers/net/vxlan.c @@ -1077,10 +1077,10 @@ static void vxlan_sock_release(struct vxlan_dev *vxlan) #if IS_ENABLED(CONFIG_IPV6) struct vxlan_sock *sock6 = rtnl_dereference(vxlan->vn6_sock); - rcu_assign_pointer(vxlan->vn6_sock, NULL); + RCU_INIT_POINTER(vxlan->vn6_sock, NULL); #endif - rcu_assign_pointer(vxlan->vn4_sock, NULL); + RCU_INIT_POINTER(vxlan->vn4_sock, NULL); synchronize_net(); vxlan_vs_del_dev(vxlan);
When stopping the vxlan interface we detach it from the socket. Use RCU_INIT_POINTER() and not rcu_assign_pointer() to do so. Suggested-by: Stephen Hemminger <stephen@networkplumber.org> Signed-off-by: Mark Bloch <markb@mellanox.com> --- drivers/net/vxlan.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)