From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756643AbcE1AaV (ORCPT ); Fri, 27 May 2016 20:30:21 -0400 Received: from mail-it0-f54.google.com ([209.85.214.54]:35929 "EHLO mail-it0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753434AbcE1AaU (ORCPT ); Fri, 27 May 2016 20:30:20 -0400 MIME-Version: 1.0 In-Reply-To: <19769bfc-efc7-495f-9574-f771236a9765@default> References: <19769bfc-efc7-495f-9574-f771236a9765@default> Date: Fri, 27 May 2016 17:30:18 -0700 X-Google-Sender-Auth: D3epthyctfHP8VNSn5FIUUs4Cs4 Message-ID: Subject: Re: [GIT PULL] Please pull NFS client changes for Linux 4.7 From: Linus Torvalds To: Boris Ostrovsky Cc: Trond Myklebust , Anna Schumaker , Linux NFS Mailing List , Linux Kernel Mailing List Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 27, 2016 at 5:17 PM, Boris Ostrovsky wrote: > > It does fix it. Ok, I've committed that in my tree, will do my usual build test and push out. > This is not the first time we've hit this and people have always been > changing code to initialize fields in execution path (and I didn't fix > it here myself because I wasn't sure where to do that). I guess we > could have been just adding braces all along. Well, adding the braces also requires that you get the *order* of the member initialization right, since the braces will be unnamed. In this case, the unnamed union was the first member, so the trivial patch worked, but that's not always the case. In fact, in this case, I think an alternate and possibly better fix would have been to just remove the ".data = { 0 }" initializer entirely, since in its absence it would have been initialized to zero anyway. But since you already tested my uglier "just add a brace like the error message suggests" that's the one I'll be committing. It's a fairly annoying failure of older compilers, exactly because people with newer compilers won't even realize that they are causing problems. Linus