From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oliver Neukum Subject: Re: [PATCH]use kzalloc in vfs where appropriate Date: Mon, 20 Mar 2006 13:52:52 +0100 Message-ID: <200603201352.56288.oliver@neukum.org> References: <200603192150.23444.oliver@neukum.org> <84144f020603192325h54fd3212l1f4846fd40b9f074@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Cc: "Arjan van de Ven" , "Matthew Wilcox" , viro@zeniv.linux.org.uk, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Return-path: Received: from mail1.kontent.de ([81.88.34.36]:14291 "EHLO Mail1.KONTENT.De") by vger.kernel.org with ESMTP id S932270AbWCTMxl (ORCPT ); Mon, 20 Mar 2006 07:53:41 -0500 To: "Pekka Enberg" In-Reply-To: <84144f020603192325h54fd3212l1f4846fd40b9f074@mail.gmail.com> Content-Disposition: inline Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org > > Rewriting the test as: > > n!=0 && n > INT_MAX / size > > saves the division because size is much likelier to be a constant, and indeed > > the code is better: > > > > cmpq $268435455, %rax > > movq $0, 40(%rsp) > > ja .L313 > > > > Is there anything I am missing? > > Did you check allyesconfig vmlinux size before and after? If it helps, > like it probably does, post a patch! Hi Pekka, it saves 18K of 232M. I am making a patch. Regards Oliver