From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Ricardo Leitner Subject: Re: [PATCH v2] sctp: fully initialize the IPv6 address in sctp_v6_to_addr() Date: Tue, 15 Aug 2017 13:31:41 -0300 Message-ID: <20170815163141.GD18688@localhost.localdomain> References: <20170814184304.82747-1-glider@google.com> <20170815015814.GB18688@localhost.localdomain> <20170814.194051.1408830683580606508.davem@davemloft.net> <20170815150557.GC18688@localhost.localdomain> <1502811469.4936.84.camel@edumazet-glaptop3.roam.corp.google.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: David Miller , hideaki.yoshifuji@miraclelinux.com, glider@google.com, dvyukov@google.com, kcc@google.com, edumazet@google.com, lucien.xin@gmail.com, vyasevich@gmail.com, linux-kernel@vger.kernel.org, linux-sctp@vger.kernel.org, netdev@vger.kernel.org, yoshfuji@linux-ipv6.org To: Eric Dumazet Return-path: Content-Disposition: inline In-Reply-To: <1502811469.4936.84.camel@edumazet-glaptop3.roam.corp.google.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Tue, Aug 15, 2017 at 08:37:49AM -0700, Eric Dumazet wrote: > On Tue, 2017-08-15 at 12:05 -0300, Marcelo Ricardo Leitner wrote: > > > Ok, but I should see a difference in the generated code, right? > > Depends on the compiler. Have you tried older versions ? > This was with gcc 6.4.1, fc25 standard. Only tested with it and didn't check clang either. > One argument is that following struct member definition eases code > review. > > (It is easier to catch a field init is missing) And a good one, yes. Thanks, Marcelo