From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH 7/8] Do not recompute msgmni anymore if explicitely set by user Date: Tue, 12 Feb 2008 11:44:39 -0800 Message-ID: <20080212114439.e08085f1.akpm@linux-foundation.org> References: <20080211141646.948191000@bull.net> <20080211141816.094061000@bull.net> <20080211122408.5008902f.akpm@linux-foundation.org> <47B167AF.6010008@bull.net> <20080212014444.8bc3791b.akpm@linux-foundation.org> <47B1B7F4.8080009@bull.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <47B1B7F4.8080009-6ktuUTfB/bM@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Nadia Derbey Cc: containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org, cmm-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org, matthltc-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org List-Id: containers.vger.kernel.org On Tue, 12 Feb 2008 16:15:00 +0100 Nadia Derbey wrote: > Andrew Morton wrote: > > On Tue, 12 Feb 2008 10:32:31 +0100 Nadia Derbey wrote: > > > > > >>it builds fine, modulo some changes in ipv4 and ipv6 (see attached patch > >>- didn't find it in the hot fixes). > > > > > > OK, thanks for checking. Did you confirm that we don't have unneeded code > > in vmlinux when CONFIG_PROCFS=n? I guess before-and-after comparison of > > the size(1) output would tell us. > > > > Those networking build problems appear to have already been fixed. > > > > In future, please quote the compiler error output in the changelog when > > sending build fixes or warning fixes, thanks. > > > > > > > > BEFORE: > > lkernel@akt$ size vmlinux > text data bss dec hex filename > 4318525 454484 462848 5235857 4fe491 vmlinux > > > AFTER: > > lkernel@akt$ size vmlinux > text data bss dec hex filename > 4323161 454484 462848 5240493 4ff6ad vmlinux > > which makes it +4636 = +0.11% > > I've got the details for */built-in.o if needed. > That seems to be a lot of increase. Are you sure you had CONFIG_PROCFS=n in both cases? If so, the patch must have added a lot of code which will never be executed?