> On Jul 31, 2015, at 16:56, Kirill A. Shutemov <kirill@xxxxxxxxxxxxx> 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 <viresh.kumar@xxxxxxxxxx> > > Acked-by: Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx> > > -- > Kirill A. Shutemov > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ search in code, there are lots of using like this , does need add this check into checkpatch ? # 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))) { ./fs/ext4/namei.c: if (unlikely(IS_ERR(bh))) { ./fs/ntfs/lcnalloc.c: if (likely(page && !IS_ERR(page))) { ./fs/ntfs/runlist.c: if (likely(!IS_ERR(old_rl))) ./fs/ntfs/mft.c: if (likely(!IS_ERR(page))) { ./fs/ntfs/mft.c: if (likely(!IS_ERR(m))) ./fs/ntfs/mft.c: if (likely(!IS_ERR(m))) { ./fs/ntfs/mft.c: if (unlikely(IS_ERR(rl) || !rl->length || rl->lcn < 0)) { ./fs/ntfs/mft.c: if (unlikely(IS_ERR(rl) || !rl->length || rl->lcn < 0)) { ./fs/ntfs/mft.c: if (likely(!IS_ERR(rl2))) ./fs/ntfs/inode.c: if (unlikely(err || IS_ERR(m))) { ./fs/ntfs/namei.c: if (likely(!IS_ERR(dent_inode))) { ./fs/ntfs/super.c: if (unlikely(IS_ERR(tmp_ino) || is_bad_inode(tmp_ino))) { ./fs/namei.c: if (unlikely(IS_ERR(s))) ./fs/namei.c: if (unlikely(IS_ERR(filename))) ./fs/gfs2/dir.c: if (unlikely(dent == NULL || IS_ERR(dent))) { ./fs/ecryptfs/inode.c: if (unlikely(IS_ERR(ecryptfs_inode))) { ./fs/ncpfs/dir.c: if (unlikely(IS_ERR(newdent))) ./fs/proc/proc_sysctl.c: if (unlikely(IS_ERR(subdir))) { Binary file ./.git/objects/pack/pack-4a5df920db8b8d9df9a91893c9567b4b2f15b782.pack matches ./drivers/target/tcm_fc/tfc_cmd.c: if (unlikely(IS_ERR(fp))) { ./drivers/thermal/intel_powerclamp.c: if (likely(!IS_ERR(thread))) { ./drivers/thermal/intel_powerclamp.c: if (likely(!IS_ERR(thread))) { ./drivers/gpu/drm/vmwgfx/vmwgfx_drv.c: if (unlikely(IS_ERR(vmaster))) { ./drivers/gpu/drm/vmwgfx/vmwgfx_context.c: if (unlikely(IS_ERR(uctx->man))) { ./drivers/gpu/drm/ttm/ttm_tt.c: if (unlikely(IS_ERR(swap_storage))) { ./drivers/gpu/drm/ttm/ttm_tt.c: if (unlikely(IS_ERR(to_page))) { ./drivers/scsi/bnx2fc/bnx2fc_fcoe.c: if (likely(!IS_ERR(thread))) { ./drivers/scsi/bnx2i/bnx2i_init.c: if (likely(!IS_ERR(thread))) { ./drivers/scsi/fcoe/fcoe.c: if (likely(!IS_ERR(thread))) { ./drivers/base/power/opp.c: if (unlikely(IS_ERR_OR_NULL(dev))) { ./drivers/base/power/opp.c: if (unlikely(IS_ERR_OR_NULL(tmp_opp)) || !tmp_opp->available) ./drivers/base/power/opp.c: if (unlikely(IS_ERR_OR_NULL(tmp_opp)) || !tmp_opp->available) ./drivers/tty/serial/serial_core.c: if (likely(!IS_ERR(tty_dev))) { ./drivers/rtc/rtc-bfin.c: if (unlikely(IS_ERR(rtc->rtc_dev))) ./drivers/rtc/rtc-gemini.c: if (likely(IS_ERR(rtc->rtc_dev))) ./drivers/rtc/interface.c: if (unlikely(IS_ERR_OR_NULL(rtc))) ./drivers/md/dm-snap-persistent.c: if (unlikely(IS_ERR(area))) { ./drivers/md/dm-verity.c: if (unlikely(IS_ERR(data))) ./drivers/md/persistent-data/dm-block-manager.c: if (unlikely(IS_ERR(p))) ./drivers/md/persistent-data/dm-block-manager.c: if (unlikely(IS_ERR(p))) ./drivers/md/persistent-data/dm-block-manager.c: if (unlikely(IS_ERR(p))) ./drivers/md/persistent-data/dm-block-manager.c: if (unlikely(IS_ERR(p))) ./drivers/staging/lustre/include/linux/libcfs/libcfs.h: if (unlikely(IS_ERR(ptr) || ptr == NULL)) ./drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c: if (likely(!IS_ERR(pfmr))) { ./drivers/staging/lustre/lustre/obdclass/lu_object.c: if (unlikely(IS_ERR(o))) ./drivers/staging/lustre/lustre/obdclass/lu_object.c: if (unlikely(IS_ERR(o))) ./drivers/staging/lustre/lustre/obdclass/lu_object.c: if (likely(IS_ERR(shadow) && PTR_ERR(shadow) == -ENOENT)) { ./drivers/staging/lustre/lustre/obdclass/lu_object.c: if (unlikely(IS_ERR(value))) ./drivers/staging/android/ashmem.c: if (unlikely(IS_ERR(vmfile))) { ./drivers/input/mouse/alps.c: } else if (unlikely(IS_ERR_OR_NULL(priv->dev3))) { ./drivers/net/ethernet/marvell/sky2.c: if (unlikely(status & Y2_IS_ERROR)) ./drivers/net/ethernet/ti/netcp_core.c: if (unlikely(IS_ERR_OR_NULL(desc))) { ./drivers/net/ethernet/ti/netcp_core.c: if (unlikely(IS_ERR_OR_NULL(ndesc))) { ./drivers/devfreq/devfreq.c: if (unlikely(IS_ERR_OR_NULL(dev))) { ./drivers/devfreq/devfreq.c: if (unlikely(IS_ERR_OR_NULL(name))) { ./drivers/misc/c2port/core.c: if (unlikely(IS_ERR(c2dev->dev))) { ./net/socket.c: if (unlikely(IS_ERR(file))) { ./net/socket.c: if (likely(!IS_ERR(newfile))) { ./net/socket.c: if (unlikely(IS_ERR(newfile1))) { ./net/socket.c: if (unlikely(IS_ERR(newfile))) { ./net/openvswitch/datapath.c: if (unlikely(IS_ERR(reply))) { ./net/openvswitch/datapath.c: if (likely(!IS_ERR(reply))) { ./net/sctp/socket.c: if (unlikely(IS_ERR(newfile))) { ./mm/huge_memory.c: if (unlikely(IS_ERR(khugepaged_thread))) { Thanks -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href