From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: [PATCH] x86: bpf_jit_comp: simplify trivial boolean return Date: Thu, 04 Dec 2014 01:26:27 -0800 Message-ID: <1417685187.2721.1.camel@perches.com> References: <1417032059.16355.4.camel@perches.com> <063D6719AE5E284EB5DD2968C1650D6D1C9FDA63@AcuExch.aculab.com> <1417114175.4305.3.camel@perches.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Cc: David Laight , Quentin Lambert , "David S. Miller" , Alexey Kuznetsov , James Morris , Hideaki YOSHIFUJI , Patrick McHardy , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , "x86@kernel.org" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" To: Alexei Starovoitov Return-path: In-Reply-To: <1417114175.4305.3.camel@perches.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Thu, 2014-11-27 at 10:49 -0800, Joe Perches wrote: > On Thu, 2014-11-27 at 12:25 +0000, David Laight wrote: > > Why the change in data? > > btw: without gcov and using -O2 > > $ size arch/x86/net/bpf_jit_comp.o* > text data bss dec hex filename > 9671 4 0 9675 25cb arch/x86/net/bpf_jit_comp.o.new > 10679 4 0 10683 29bb arch/x86/net/bpf_jit_comp.o.old Alexei? Is this 10% reduction in size a good reason to change the code?