Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752275AbbGaK2J (ORCPT ); Fri, 31 Jul 2015 06:28:09 -0400 Received: from smtprelay0042.hostedemail.com ([216.40.44.42]:60943 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751113AbbGaK2H (ORCPT ); Fri, 31 Jul 2015 06:28:07 -0400 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Spam-Summary: 2,0,0,,d41d8cd98f00b204,joe@perches.com,:::::::::::::::::::::::::::::,RULES_HIT:41:355:379:541:599:800:960:973:988:989:1260:1277:1311:1313:1314:1345:1359:1373:1431:1437:1515:1516:1518:1534:1541:1593:1594:1711:1730:1747:1777:1792:2393:2553:2559:2562:2693:2828:3138:3139:3140:3141:3142:3353:3622:3865:3866:3867:3868:3870:3871:3872:3874:4321:5007:6261:6742:7903:8531:10004:10400:10450:10455:10848:11232:11233:11658:11914:12043:12262:12296:12438:12517:12519:12555:12679:12740:13069:13311:13357:19904:19999:21080,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:fn,MSBL:0,DNSBL:none,Custom_rules:0:0:0 X-HE-Tag: birds28_4630ceaf91e3b X-Filterd-Recvd-Size: 3324 Message-ID: <1438338481.19675.72.camel@perches.com> Subject: Re: [PATCH 14/15] mm: Drop unlikely before IS_ERR(_OR_NULL) From: Joe Perches To: Viresh Kumar Cc: yalin wang , "Kirill A. Shutemov" , Andrew Morton , linaro-kernel@lists.linaro.org, open list , "Aneesh Kumar K.V" , David Rientjes , Ebru Akagunduz , Johannes Weiner , "Kirill A. Shutemov" , "open list:MEMORY MANAGEMENT" , Mel Gorman , Michal Hocko , Vlastimil Babka Date: Fri, 31 Jul 2015 03:28:01 -0700 In-Reply-To: <20150731093450.GA7505@linux> References: <91586af267deb26b905fba61a9f1f665a204a4e3.1438331416.git.viresh.kumar@linaro.org> <20150731085646.GA31544@node.dhcp.inet.fi> <20150731093450.GA7505@linux> Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: Evolution 3.12.11-0ubuntu3 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1994 Lines: 53 On Fri, 2015-07-31 at 15:04 +0530, Viresh Kumar wrote: > On 31-07-15, 17:32, yalin wang wrote: > > > > > On Jul 31, 2015, at 16:56, Kirill A. Shutemov wrote: > > > > > > On Fri, Jul 31, 2015 at 02:08:34PM +0530, Viresh Kumar wrote: > > >> IS_ERR(_OR_NULL) already contain an 'unlikely' compiler flag and there > > >> is no need to do that again from its callers. Drop it. > > >> > > >> Signed-off-by: Viresh Kumar > > > > > > Acked-by: Kirill A. Shutemov > > > search in code, there are lots of using like this , does need add this check into checkpatch ? > > cc'd Joe for that. :) > > > # grep -r 'likely.*IS_ERR' . > > ./include/linux/blk-cgroup.h: if (unlikely(IS_ERR(blkg))) > > ./fs/nfs/objlayout/objio_osd.c: if (unlikely(IS_ERR(od))) { > > ./fs/cifs/readdir.c: if (unlikely(IS_ERR(dentry))) > > ./fs/ext4/extents.c: if (unlikely(IS_ERR(bh))) { > > ./fs/ext4/extents.c: if (unlikely(IS_ERR(path1))) { > > ./fs/ext4/extents.c: if (unlikely(IS_ERR(path2))) { > > Btw, my series has fixed all of them :) If it's all fixed, then it's unlikely to be needed in checkpatch. But given the unlikely was added when using gcc3.4, I wonder if it's still appropriate to use unlikely in IS_ERR at all. --- commit b5acea523151452c37cd428437e7576a291dd146 Author: Andrew Morton Date: Sun Aug 22 23:04:49 2004 -0700 [PATCH] mark IS_ERR as unlikely() It seems fair to assume that it is always unlikely that IS_ERR will return true. This patch changes the gcc-3.4-generated kernel text by ~500 bytes (less) so it's fair to assume that the compiler is indeed propagating unlikeliness out of inline functions. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/