Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S938807AbcJRRFL (ORCPT ); Tue, 18 Oct 2016 13:05:11 -0400 Received: from mail-db5eur01on0061.outbound.protection.outlook.com ([104.47.2.61]:2496 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1755582AbcJRRFA (ORCPT ); Tue, 18 Oct 2016 13:05:00 -0400 Authentication-Results: spf=pass (sender IP is 193.47.165.134) smtp.mailfrom=mellanox.com; vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=pass action=none header.from=mellanox.com; Subject: Re: [PATCH 03/28] [v2] infiniband: shut up a maybe-uninitialized warning To: Arnd Bergmann References: <20161017220342.1627073-1-arnd@arndb.de> <20161017220557.1688282-3-arnd@arndb.de> <33302790-0a4c-e2b3-868d-3e7dadbd3c07@mellanox.com> <9469876.gij7f9Mh1b@wuerfel> CC: Doug Ledford , Linus Torvalds , , Sean Hefty , Hal Rosenstock , "Matan Barak" , Leon Romanovsky , "Sagi Grimberg" , Bart Van Assche , Alex Vesker , Guy Shapiro , From: Haggai Eran Message-ID: <08ccbb7b-b265-999c-56c9-2fa0b28c85a9@mellanox.com> Date: Tue, 18 Oct 2016 13:32:13 +0300 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <9469876.gij7f9Mh1b@wuerfel> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.8.18.238] X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:193.47.165.134;IPV:NLI;CTRY:IL;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(7916002)(2980300002)(438002)(199003)(377454003)(24454002)(189002)(87936001)(36756003)(81156014)(5660300001)(92566002)(6916009)(305945005)(11100500001)(106466001)(31686004)(2950100002)(7736002)(110136003)(31696002)(86362001)(65806001)(50466002)(7416002)(33646002)(54356999)(189998001)(6116002)(83506001)(65826007)(3846002)(64126003)(93886004)(356003)(23746002)(81166006)(586003)(230700001)(8676002)(2906002)(77096005)(76176999)(4326007)(65956001)(4001350100001)(7846002)(626004)(47776003)(50986999)(8936002)(8666005)(7059030)(3940600001);DIR:OUT;SFP:1101;SCL:1;SRVR:AM4PR0501MB2772;H:mtlcas13.mtl.com;FPR:;SPF:Pass;PTR:mail13.mellanox.com;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;DB3FFO11FD026;1:kazVCDD0hFPTZL9meHJTzKzBLSDEzDGhN1JcTuOf6XZE9g9YPLfEDAyWuFUxZINbliO0BmndhQwhfUky0MT7quM5cIt65ZdtfrUbpl3qahWDoY2Nh7zDSWxfu9ViYv67PxoKcsN10KOMJ+S7JmTw1WvcUCZq7Urlf3JcMsTkIzwW2sDAhsGBIlIcC5fTiP+E/S2AEOUXgkwwhsGCLscEfN7rDueMyl9lQVSn1XEBc33rHh6OGn+cjcHqXPdrt8bRFpsiUmCSSvU2n3BGIh64BI0KK3+J0rwI9p7B6DJyTsH3y4tuZqnn7aIkUzTvkwjf+SsM9goelWfDmGktVIog44z5kLZ//uvzu+WlvIKKu9nTM4pfFxW5//Kjyg19/CPABuJvazhjWzr1stW7u0yIalmlWHKNyBcl4KNiMHYWNmwamMCB72JRlMtQVHzCaWdfk8C6sIAJftnCZze5jcuBbyOjp9A0E75cLoaUdSc/k5ZUZ47dgmQVjApNxEt0OifEdVRGovKBqpu7EVCDI8Dc3jVEcWIKqSfz7OxjB97ac89vrdolAbQZgHlJY6j7XtFuL4g4OvMt9EEOuoAUd0/k4Dim4slArDacv1R77Zn/Mu0= X-MS-Office365-Filtering-Correlation-Id: 947b7cc0-99cb-45f8-cdea-08d3f7420969 X-Microsoft-Exchange-Diagnostics: 1;AM4PR0501MB2772;2:CAxkH4JR5BupyG4QfXa/ngcVQVdeN4/7mGFAd3Gt0l9x9bHUrkR3vH6lOB4zOUEZBydp1Njrc5cg8eh9dT+MK4BYDDSu6MXidalsHj2KNd9srLuJ9XlXgABgk7ynuBzgnAIYrZxXMQeP+lUUPlbr88d7B5BX4b1OPKIZbAfKEpStoGWFHJfqmFDuQ/UH8ff/TvtcJfg6DgCmqEAADxHsbw==;3:n/iiJwi/LOJC2AkAaUom9hSSoc+ToknVMJgHKuoMwZ6z76bNVi4mJDVG/lRCbu7VHKXVJv2LvuGTeMxaG74Sn1mPuP+dz6Sz27jaWERpNV+eiPA75t26MtOLU6tanzpcfKw3TYC/LzOY7dZ1vVrdQv/cd4G8ofDraSACpriiO2YFTNDaVLtIOTOZataYhOaJyX6N/MJoxHy6w/yR//okt0PbreyLiaDGCM6UtU7XCgn+SXFPgZh27/UePwmBMJA4o2I7AtFqsMQB6vYNBrRd0fIRv2X+chTiY9WF2n2Ko94= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(8251501002);SRVR:AM4PR0501MB2772; X-Microsoft-Exchange-Diagnostics: 1;AM4PR0501MB2772;25:bach6L3hOKM9c1F/pANFV9BF87VfjnEmtJAOwlmdXjGU3HNokwzzBoi7iRRNKaKl2jEJ/KjYvD1ZI+iNWX5UCdyORVyigkXLbY4QRqW+vyKVQDHCTOZ0sbLzs+1iMcKW5cYnUL+dwbQ3dVf0HtCpDM1Ad1FyMbAizWzNvVB/eYEFb93LTSaG6m6Z/QxakMmWGZj/NGbvmbClqCevIfIEMih1cHAjkUgWMQVaR4Q3XC6wHxBLehfXO343wKim32Q4cMLMspZCs6pcrix8/M1J0DR2Tbkrr/3atZYMN60ntQJlCRV1BEfxdhT1Pedf46aPu4T8frzZmbktBUcB5ctfmvmTa0V72ekEs9Va46B2dw2rss6Qtx7b+00N2KQklNXuFOYVeFROEV6fE8ljtZD5fISbXuxhGSdiBy2kpRpWHPB59Tk8BdZy9L8FNdukebFp3K63fkSkaN4szxcwZyBu5KQrhm8XFS2sHkJ0rzahyBD3JNR9MI+2k0m0bh1XVhvWnTLAjDnN7Ch6jyp7qKTE+eJMhmflDuFFDikxOmJuaZI8K49xNWPpm7p53nwigRAvhMtd+ZR8gAmHu7gq9Uq36VaT14mjKatYpXlbs6OUNmf8OAf/tisdubzbjF1nIxQr2gu1nr4rAQtTOFD2RCuta/C6avSZWBuEZ9QF+UaplPY3hFJnAcq2/Tzl2HlAT2/Wg7BlZpl75f3rPmVv54UsLoWHumXJf6AolkpOMF73xBNFBEu3SGb/zftNLAzYbCs28mdF/8PzPkRgoO7Qs5K3jQ== X-Microsoft-Exchange-Diagnostics: 1;AM4PR0501MB2772;31:9OKlM6thcC+MIDbxcdMT7vCF1Y+s49W7Ce/IijIeHDKtyMvJzYxDQZrljzFTQbSsa49PwL1WUKYWIb/XW1A7pbNljg0CRfltE1GyDuxMLKoSkMW0SY/EVh+Q8zXgZDUHIuuxp6g9al3tG9TXcYsjvwOqgGxzHKXII+jy+KrQmDNCBP118re23yf+t3KdyuRSj7FkQaXpLiKczFPmEpgr35Uu3ktAJ1k/iqhPn1mBBrwNLIfkx0i0c/iTPaStD7B2;20:L7MkI8GxRe4rIQP2bUGmb5oq28iC8wI47LSo6C5s/igxhdiL8PqTYK8/wLCyEV1oFMljHLCyGTKnBaNo8smXg3/F8ayIvipeXucGNIzv3b9IPRe0/xoxSYwgbkvS4ueXZRx7Svh+E4SwXBTAFSMoky2jAWWqw0/CIeuzYXS+AAN+BHTtXPjdA0ZptOe0PsXciQ0kTLWlHnbAQ6VhiVYDVSsV1SmXCzczbMGYxXwvGy3ltN+amMDxyPIp6KeextaammAE1l4+gdBcZufL7Vq5dZ/rcwFv3r3++Mm4eZxh9szE3HgmzNLZHvSjvDA9f93DRNpGRrYOecxdcaTrmyih6l69sixz+jjx1u0cnV5Inm0iS3Vxczg+LF2ZIFdA/pLHBnmPZFq+fLkq0MWyTsSC9MpgbGwTdWIRfChCq8xu/GUQUnF1rTWBk6AFNwEmf4eDBDref5UCgFbraDqaElW2E9vjARIzY0xsvDCbg4q+PBkLZbBG9T5vaphf2nSBuZtN X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040176)(601004)(2401047)(8121501046)(13016025)(13018025)(5005006)(10201501046)(3002001)(6055026);SRVR:AM4PR0501MB2772;BCL:0;PCL:0;RULEID:;SRVR:AM4PR0501MB2772; X-Microsoft-Exchange-Diagnostics: 1;AM4PR0501MB2772;4:0FpuQ7BsV0U8qNgjisO2RdsB6QMh7Gbg14VGE4Fd8/ltyHUJ6gerULanZwO6lbwn5jgT8s2FKAriWzixWu5re3r0eOyKnoXBO5rXKZ9QLbclznHY3M677P9iw23vE79bzWlfeuhekotifaWWhziy2I1G1kWtpU25hZxeyuola0JWY3UUkBWIwvzfkVx29SMAWnRXu6b2TjIZrSPAmVHMYx3K7fSxp7nr00le5G7JK2JnBLXsNEi4fskBb3mCrj8mRxMUbIOmVNyDVzkr98S88V0mU5Y6j1HNLt3QRjHVKfKZv8pAfvFaHPifwQrDqz5v6eq8gZ6rlzM45buoWei8FQh721UqsAmempaL9+fF7Vo1W07v97Yhtb68IIqAqix35N/yhBBEkTPdlbnOoAVIie0hpyc94mXCKuIsalzzb6Q4hihf00mObnRaWmDieRKKMkzpvffS/TlngKYb5vmL5w== X-Forefront-PRVS: 00997889E7 X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;AM4PR0501MB2772;23:wjwH9KqkROkBwIRAAOmhaHpIL17BGjCWzC8?= =?Windows-1252?Q?ZYGadC1Rg5bW2uTMq0dYHfZPy82zlZj7+SeyEP8IqVKASCYZnKJBS0o3?= =?Windows-1252?Q?qTDjnLzi4sIxkCmGa+7AGmpSsxbNHA6MWxktR8yrN64c2Qggljvltrt5?= =?Windows-1252?Q?5WFbScPxnfJIhl4DvYO2Ni9TCA/AKxOhWesrJVeDDSu1wp4FJhfMwj2m?= =?Windows-1252?Q?jS9nBvUXsvmIWpz1GD9ckmXseFdH96NRdJfun/gftY5mzCAgSgBV1Vdw?= =?Windows-1252?Q?ntRM/DCjtKOZ09xuX3HT+bHiD6zM/ahbkJmKITvvdKGkVjKmOjtXF7MN?= =?Windows-1252?Q?XD0ExbgET1jFRYct+5RzqNvYlVRjwhAH+45FKjwwnfSCz+snHH3ooUJj?= =?Windows-1252?Q?3nmBpjKgV+Hne6p4uOqp+a4AoXM9EpGummQFFE/uu/MSRxFjanSW5xOY?= =?Windows-1252?Q?O8zVGiId2eOHUTfk7cu1I53J7IKz7iUWNGRUtd0vgblwY0drYIYH7VI8?= =?Windows-1252?Q?BZrHFQd8hBfUxFxaROtffUCic8HoC2DoOG680NsbmpTPy7ZmTCZua4Gk?= =?Windows-1252?Q?JelRkC/q2wv8sSGCIfFP9zJPgGmOT1cB92QFfb+FZGmK82O1lBUVxOcZ?= =?Windows-1252?Q?iHQaK+7FEwW/n1nCxpkz3uxI0jAJ7o7fvSm5SOG1mFE4eGrD3ql6AGpn?= =?Windows-1252?Q?JrRPDAiSr4r3EiO8p7pdjXFzP7uq3FDOIVa3x3zdUHDt4ehIaOaKM+UW?= =?Windows-1252?Q?R1ScVMHuewNrRkrCqlCqdcUts1vboaJIqTTWEdFbiGHejP6wpp7knoqd?= =?Windows-1252?Q?1IgIE5jshKBri8JG3vod1XICIysq0ovwJfQmPcye9HfOZeK0nmgB66fo?= =?Windows-1252?Q?dqdhNVD5SCKssZWvxSRQLWVnF8iX7S0dHJ2kV7dS+ZGSI6eEFBG4J6ny?= =?Windows-1252?Q?KRf8F9neDZ2EjcJQqJMIKy88RBtYfKDTzplhP0kLy2fIp/SuasXvzr39?= =?Windows-1252?Q?mLdDKHYEQiBADZp1Tn8JDzuLt3YoK+kL5IKYHR4T6ek7MXMSFFFIqtUQ?= =?Windows-1252?Q?yFcFuGcBJhrqxL0ilfO2I0aBjc+PaBpuk/vg6hyaWgHyTstjjbQB5rg9?= =?Windows-1252?Q?sj8v14gVuerXfhSfifYZ5XFZI5eU1SpX/9kWRDTj75tlV6kWrT7FehTs?= =?Windows-1252?Q?BlU68KtruCyEkseSyBKuJuTM1SgKKmcMd/ryYdsJYahpdTFf7oa20pJi?= =?Windows-1252?Q?kx7HM887UmHszSmT8nBDOJJwkTnlttEyyv2Gg3YKZcDHqrtfPNYEn/WO?= =?Windows-1252?Q?1OoOIKqvgUgQvpmRpzdaDL4CJt4HUyXqAotG4TJ8HtrXwpsAR0gptkLF?= =?Windows-1252?Q?UuCkUkwPxEglRfx3ZgeUOAgBhr1Fdojxex1DCLM3SITYtuN4osaDkSPU?= =?Windows-1252?Q?=3D?= X-Microsoft-Exchange-Diagnostics: 1;AM4PR0501MB2772;6:O5HF6l1VDl8SekrdaUU+/N8yFznmA5V3R4aZExDKxFkB7Bp5gCcf0av2xprsM1b+YkmLAhPfRhV7+qjV0vQSguQV1T5Or+Trvgtnkd/PPnEV+wiCU+la6dUst+he/WMFLiBguib+z3GqrIrZLvEsOITvAuT2TE3ElMNukbOC+yUnTi9PS6a1NQoGBOu7iOVBqUmuobz5PYi3swtf4hhOf22caZxNWgysWOVZliFffkjYIHA3461M+pxyte/Ms3niAOlfDWtvdKnqp0U8vGIj/9EG331dPwVSH6rV2FMFS0SS9C83JjLlKaQR79c3DFwkmV/o7J7YesAvU+svYFXhtQ==;5:Epw9mHfioZ9jaiiVIvF36AhdHs7aJ0UHINgSXuoctNlLQX0ivosivp1GQZgHe9GUaiLpt1muJ5kJyofM95eHEZ54kMHDYNeTMsoCzCUiH8KjzVRCzVd4Q/RYHeo5dQt8+abje6oWmyvMtdbaBo8fNGobEFKQMOgANinSdId0JeY=;24:E7myr/kSYN+ssXMos3M4BH+SB6Q+YOqqOBvzRdQv1LI2hvBZWt1Cr+EePk0AgNx2eu0F+zvogxZtVXFUoZo8As2A4+bYNTADIpbklMufe8U= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;AM4PR0501MB2772;7:QZlqU9Jiq6eBECxm1Ar6aElzQsaXbEMcy09kHNwgQHaAR0HINo1yChTDLFIDk1FZ2DMezR8C9J42/EXOrKFVhDkrWrOa6nn3nxwAR55xFKYIrTFcMo72IGFJhOo8/Ospd6tyRL7niCkf75UJMu+hSuhZYrARW0aBMv+JXWsLJ/q1P5Abguzxb1gYNee5vpCl7XXIU3C6ZtKjyqZMuLMXvSd71Sr2WZI+RZmD9jTDWFTJISLItEeDGIKHFyIQexx/Kg8jNdn/FdpZ3cNQMllxojvRWg8PTv4mowXrDMnyGvWOiO0RwxjL9tNw3XI7Wxt2GjIYBE8bTo9+bQ4YupBvjXC88Nq5rlumimmNUAosKQY= X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Oct 2016 10:32:18.5175 (UTC) X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=a652971c-7d2e-4d9b-a6a4-d149256f461b;Ip=[193.47.165.134];Helo=[mtlcas13.mtl.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM4PR0501MB2772 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 902 Lines: 18 On 10/18/2016 1:18 PM, Arnd Bergmann wrote: > On Tuesday, October 18, 2016 9:47:31 AM CEST Haggai Eran wrote: >> On 10/18/2016 1:05 AM, Arnd Bergmann wrote: >>> @@ -1309,7 +1311,7 @@ static bool validate_net_dev(struct net_device *net_dev, >>> static struct net_device *cma_get_net_dev(struct ib_cm_event *ib_event, >>> const struct cma_req_info *req) >>> { >>> - struct sockaddr_storage listen_addr_storage, src_addr_storage; >>> + struct sockaddr_storage listen_addr_storage = {}, src_addr_storage = {}; >> >> Doesn't this still translate to an extra initialization that Doug was >> worried about? > > Thanks for spotting this. I must have screwed up while rebasing the patch > at some point, this one change should not be there, the other changes by > themselves sufficiently address the warning. Okay, other than this the patch looks good to me.