* [PATCH 01/15] drm: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
@ 2010-12-09 20:03 ` Joe Perches
2010-12-09 20:03 ` [PATCH 02/15] stmmac: " Joe Perches
` (14 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:03 UTC (permalink / raw)
To: linux-kernel; +Cc: Jiri Kosina, David Airlie, dri-devel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/gpu/drm/ttm/ttm_tt.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/ttm/ttm_tt.c b/drivers/gpu/drm/ttm/ttm_tt.c
index af789dc..fb83a90 100644
--- a/drivers/gpu/drm/ttm/ttm_tt.c
+++ b/drivers/gpu/drm/ttm/ttm_tt.c
@@ -538,7 +538,7 @@ int ttm_tt_swapout(struct ttm_tt *ttm, struct file *persistant_swap_storage)
swap_storage = shmem_file_setup("ttm swap",
ttm->num_pages << PAGE_SHIFT,
0);
- if (unlikely(IS_ERR(swap_storage))) {
+ if (IS_ERR(swap_storage)) {
printk(KERN_ERR "Failed allocating swap storage.\n");
return PTR_ERR(swap_storage);
}
@@ -552,7 +552,7 @@ int ttm_tt_swapout(struct ttm_tt *ttm, struct file *persistant_swap_storage)
if (unlikely(from_page == NULL))
continue;
to_page = read_mapping_page(swap_space, i, NULL);
- if (unlikely(IS_ERR(to_page))) {
+ if (IS_ERR(to_page)) {
ret = PTR_ERR(to_page);
goto out_err;
}
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 02/15] stmmac: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
2010-12-09 20:03 ` [PATCH 01/15] drm: Remove " Joe Perches
@ 2010-12-09 20:03 ` Joe Perches
2010-12-09 20:03 ` [PATCH 03/15] rtc: " Joe Perches
` (13 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:03 UTC (permalink / raw)
To: Giuseppe Cavallaro; +Cc: Jiri Kosina, netdev, linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/net/stmmac/stmmac_main.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/net/stmmac/stmmac_main.c b/drivers/net/stmmac/stmmac_main.c
index c0dc785..20f803d 100644
--- a/drivers/net/stmmac/stmmac_main.c
+++ b/drivers/net/stmmac/stmmac_main.c
@@ -949,7 +949,7 @@ static int stmmac_sw_tso(struct stmmac_priv *priv, struct sk_buff *skb)
skb, skb->len);
segs = skb_gso_segment(skb, priv->dev->features & ~NETIF_F_TSO);
- if (unlikely(IS_ERR(segs)))
+ if (IS_ERR(segs))
goto sw_tso_end;
do {
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 03/15] rtc: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
2010-12-09 20:03 ` [PATCH 01/15] drm: Remove " Joe Perches
2010-12-09 20:03 ` [PATCH 02/15] stmmac: " Joe Perches
@ 2010-12-09 20:03 ` Joe Perches
2010-12-09 20:06 ` Mike Frysinger
2010-12-09 20:03 ` [PATCH 04/15] s390: " Joe Perches
` (12 subsequent siblings)
15 siblings, 1 reply; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:03 UTC (permalink / raw)
To: Mike Frysinger, Alessandro Zummo
Cc: Jiri Kosina, uclinux-dist-devel, rtc-linux, linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/rtc/rtc-bfin.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/rtc/rtc-bfin.c b/drivers/rtc/rtc-bfin.c
index b4b6087..4ba3e33 100644
--- a/drivers/rtc/rtc-bfin.c
+++ b/drivers/rtc/rtc-bfin.c
@@ -383,7 +383,7 @@ static int __devinit bfin_rtc_probe(struct platform_device *pdev)
/* Register our RTC with the RTC framework */
rtc->rtc_dev = rtc_device_register(pdev->name, dev, &bfin_rtc_ops,
THIS_MODULE);
- if (unlikely(IS_ERR(rtc->rtc_dev))) {
+ if (IS_ERR(rtc->rtc_dev)) {
ret = PTR_ERR(rtc->rtc_dev);
goto err;
}
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [PATCH 03/15] rtc: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [PATCH 03/15] rtc: " Joe Perches
@ 2010-12-09 20:06 ` Mike Frysinger
2010-12-09 20:28 ` Joe Perches
0 siblings, 1 reply; 22+ messages in thread
From: Mike Frysinger @ 2010-12-09 20:06 UTC (permalink / raw)
To: Joe Perches
Cc: Alessandro Zummo, Jiri Kosina, uclinux-dist-devel, rtc-linux,
linux-kernel
On Thu, Dec 9, 2010 at 15:03, Joe Perches wrote:
> IS_ERR already uses unlikely, remove unlikely from the call sites.
Tobias already submitted a patch for this. and i cc-ed you on it.
-mike
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 03/15] rtc: Remove duplicate unlikely from IS_ERR
2010-12-09 20:06 ` Mike Frysinger
@ 2010-12-09 20:28 ` Joe Perches
0 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:28 UTC (permalink / raw)
To: Mike Frysinger, Tobias Klauser, Alessandro Zummo
Cc: Alessandro Zummo, Jiri Kosina, uclinux-dist-devel, rtc-linux,
linux-kernel
On Thu, 2010-12-09 at 15:06 -0500, Mike Frysinger wrote:
> On Thu, Dec 9, 2010 at 15:03, Joe Perches wrote:
> > IS_ERR already uses unlikely, remove unlikely from the call sites.
> Tobias already submitted a patch for this.
Thanks Tobias. Turns out you did them all after all.
Could you please cc linux-kernel for these sorts of patches?
As far as I can tell, the rtc-bfin patch didn't hit lkml,
only the c2port patch did for whatever reason.
The patches aren't marked as an n/m series which would also help
identify the patch scope. I thought it was a singleton.
Jiri and all others, please ignore my duplicate patchset.
cheers, Joe
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 04/15] s390: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (2 preceding siblings ...)
2010-12-09 20:03 ` [PATCH 03/15] rtc: " Joe Perches
@ 2010-12-09 20:03 ` Joe Perches
2010-12-09 20:03 ` [PATCH 05/15] osd: " Joe Perches
` (11 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:03 UTC (permalink / raw)
To: Christof Schmitt, Swen Schillig, linux390
Cc: Jiri Kosina, Martin Schwidefsky, Heiko Carstens, linux-s390,
linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/s390/scsi/zfcp_fsf.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/s390/scsi/zfcp_fsf.c b/drivers/s390/scsi/zfcp_fsf.c
index 2eb7dd5..722cdf5 100644
--- a/drivers/s390/scsi/zfcp_fsf.c
+++ b/drivers/s390/scsi/zfcp_fsf.c
@@ -1553,7 +1553,7 @@ int zfcp_fsf_open_wka_port(struct zfcp_fc_wka_port *wka_port)
SBAL_FLAGS0_TYPE_READ,
qdio->adapter->pool.erp_req);
- if (unlikely(IS_ERR(req))) {
+ if (IS_ERR(req)) {
retval = PTR_ERR(req);
goto out;
}
@@ -1606,7 +1606,7 @@ int zfcp_fsf_close_wka_port(struct zfcp_fc_wka_port *wka_port)
SBAL_FLAGS0_TYPE_READ,
qdio->adapter->pool.erp_req);
- if (unlikely(IS_ERR(req))) {
+ if (IS_ERR(req)) {
retval = PTR_ERR(req);
goto out;
}
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 05/15] osd: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (3 preceding siblings ...)
2010-12-09 20:03 ` [PATCH 04/15] s390: " Joe Perches
@ 2010-12-09 20:03 ` Joe Perches
2010-12-09 20:03 ` [PATCH 06/15] serial: " Joe Perches
` (10 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:03 UTC (permalink / raw)
To: Boaz Harrosh, Benny Halevy
Cc: Jiri Kosina, James E.J. Bottomley, osd-dev, linux-scsi,
linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/scsi/osd/osd_initiator.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/scsi/osd/osd_initiator.c b/drivers/scsi/osd/osd_initiator.c
index b798919..b37c8a3 100644
--- a/drivers/scsi/osd/osd_initiator.c
+++ b/drivers/scsi/osd/osd_initiator.c
@@ -951,7 +951,7 @@ static int _osd_req_finalize_cdb_cont(struct osd_request *or, const u8 *cap_key)
/* create a bio for continuation segment */
bio = bio_map_kern(req_q, or->cdb_cont.buff, or->cdb_cont.total_bytes,
GFP_KERNEL);
- if (unlikely(IS_ERR(bio)))
+ if (IS_ERR(bio))
return PTR_ERR(bio);
bio->bi_rw |= REQ_WRITE;
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 06/15] serial: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (4 preceding siblings ...)
2010-12-09 20:03 ` [PATCH 05/15] osd: " Joe Perches
@ 2010-12-09 20:03 ` Joe Perches
2010-12-09 21:08 ` David Brown
2010-12-09 20:04 ` [PATCH 07/15] brcm80211: " Joe Perches
` (9 subsequent siblings)
15 siblings, 1 reply; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:03 UTC (permalink / raw)
To: David Brown, Daniel Walker, Bryan Huntsman
Cc: Jiri Kosina, linux-arm-msm, linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/serial/msm_serial.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/serial/msm_serial.c b/drivers/serial/msm_serial.c
index f8c816e..8e43a7b 100644
--- a/drivers/serial/msm_serial.c
+++ b/drivers/serial/msm_serial.c
@@ -686,7 +686,7 @@ static int __init msm_serial_probe(struct platform_device *pdev)
msm_port = UART_TO_MSM(port);
msm_port->clk = clk_get(&pdev->dev, "uart_clk");
- if (unlikely(IS_ERR(msm_port->clk)))
+ if (IS_ERR(msm_port->clk))
return PTR_ERR(msm_port->clk);
port->uartclk = clk_get_rate(msm_port->clk);
printk(KERN_INFO "uartclk = %d\n", port->uartclk);
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [PATCH 06/15] serial: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [PATCH 06/15] serial: " Joe Perches
@ 2010-12-09 21:08 ` David Brown
0 siblings, 0 replies; 22+ messages in thread
From: David Brown @ 2010-12-09 21:08 UTC (permalink / raw)
To: Joe Perches
Cc: David Brown, Daniel Walker, Bryan Huntsman, Jiri Kosina,
linux-arm-msm, linux-kernel, Tobias Klauser
On Thu, Dec 09, 2010 at 12:03:59PM -0800, Joe Perches wrote:
> IS_ERR already uses unlikely, remove unlikely from the call sites.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/serial/msm_serial.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
Tobias Klauser <tklauser@distanz.ch> already posted a patch about
this. They are identical, so I'll grab the first one from Tobias.
Thanks,
David
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 07/15] brcm80211: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (5 preceding siblings ...)
2010-12-09 20:03 ` [PATCH 06/15] serial: " Joe Perches
@ 2010-12-09 20:04 ` Joe Perches
2010-12-09 20:04 ` [PATCH 08/15] gadget: " Joe Perches
` (8 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:04 UTC (permalink / raw)
To: linux-kernel
Cc: Jiri Kosina, Brett Rudley, Henry Ptasinski, Nohee Ko,
Greg Kroah-Hartman, linux-wireless, devel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c b/drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c
index 0d7aa4a..edf300d 100644
--- a/drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c
+++ b/drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c
@@ -3196,7 +3196,7 @@ s32 wl_cfg80211_attach(struct net_device *ndev, void *data)
}
WL_DBG(("func %p\n", wl_cfg80211_get_sdio_func()));
wdev = wl_alloc_wdev(sizeof(struct wl_iface), &wl_cfg80211_get_sdio_func()->dev);
- if (unlikely(IS_ERR(wdev)))
+ if (IS_ERR(wdev))
return -ENOMEM;
wdev->iftype = wl_mode_to_nl80211_iftype(WL_MODE_BSS);
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 08/15] gadget: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (6 preceding siblings ...)
2010-12-09 20:04 ` [PATCH 07/15] brcm80211: " Joe Perches
@ 2010-12-09 20:04 ` Joe Perches
2010-12-09 20:04 ` [PATCH 09/15] exofs: " Joe Perches
` (7 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:04 UTC (permalink / raw)
To: David Brownell; +Cc: Jiri Kosina, Greg Kroah-Hartman, linux-usb, linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/usb/gadget/f_fs.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/usb/gadget/f_fs.c b/drivers/usb/gadget/f_fs.c
index a21349c..1499f9e 100644
--- a/drivers/usb/gadget/f_fs.c
+++ b/drivers/usb/gadget/f_fs.c
@@ -426,7 +426,7 @@ static ssize_t ffs_ep0_write(struct file *file, const char __user *buf,
}
data = ffs_prepare_buffer(buf, len);
- if (unlikely(IS_ERR(data))) {
+ if (IS_ERR(data)) {
ret = PTR_ERR(data);
break;
}
@@ -499,7 +499,7 @@ static ssize_t ffs_ep0_write(struct file *file, const char __user *buf,
spin_unlock_irq(&ffs->ev.waitq.lock);
data = ffs_prepare_buffer(buf, len);
- if (unlikely(IS_ERR(data))) {
+ if (IS_ERR(data)) {
ret = PTR_ERR(data);
break;
}
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 09/15] exofs: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (7 preceding siblings ...)
2010-12-09 20:04 ` [PATCH 08/15] gadget: " Joe Perches
@ 2010-12-09 20:04 ` Joe Perches
2010-12-09 20:04 ` [PATCH 10/15] ext2: " Joe Perches
` (6 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:04 UTC (permalink / raw)
To: Boaz Harrosh, Benny Halevy; +Cc: Jiri Kosina, osd-dev, linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
fs/exofs/super.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/exofs/super.c b/fs/exofs/super.c
index 79c3ae6..b2b12a8 100644
--- a/fs/exofs/super.c
+++ b/fs/exofs/super.c
@@ -488,7 +488,7 @@ static int exofs_read_lookup_dev_table(struct exofs_sb_info **psbi,
}
od = osduld_info_lookup(&odi);
- if (unlikely(IS_ERR(od))) {
+ if (IS_ERR(od)) {
ret = PTR_ERR(od);
EXOFS_ERR("ERROR: device requested is not found "
"osd_name-%s =>%d\n", odi.osdname, ret);
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 10/15] ext2: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (8 preceding siblings ...)
2010-12-09 20:04 ` [PATCH 09/15] exofs: " Joe Perches
@ 2010-12-09 20:04 ` Joe Perches
2010-12-09 20:04 ` [PATCH 11/15] ext3: " Joe Perches
` (5 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:04 UTC (permalink / raw)
To: Jan Kara; +Cc: Jiri Kosina, linux-ext4, linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
fs/ext2/namei.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/ext2/namei.c b/fs/ext2/namei.c
index f8aecd2..2e1d834 100644
--- a/fs/ext2/namei.c
+++ b/fs/ext2/namei.c
@@ -67,7 +67,7 @@ static struct dentry *ext2_lookup(struct inode * dir, struct dentry *dentry, str
inode = NULL;
if (ino) {
inode = ext2_iget(dir->i_sb, ino);
- if (unlikely(IS_ERR(inode))) {
+ if (IS_ERR(inode)) {
if (PTR_ERR(inode) == -ESTALE) {
ext2_error(dir->i_sb, __func__,
"deleted inode referenced: %lu",
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 11/15] ext3: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (9 preceding siblings ...)
2010-12-09 20:04 ` [PATCH 10/15] ext2: " Joe Perches
@ 2010-12-09 20:04 ` Joe Perches
2010-12-09 20:04 ` [PATCH 12/15] ext4: " Joe Perches
` (4 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:04 UTC (permalink / raw)
To: Jan Kara, Andrew Morton, Andreas Dilger
Cc: Jiri Kosina, linux-ext4, linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
fs/ext3/namei.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/ext3/namei.c b/fs/ext3/namei.c
index bce9dce..268f776 100644
--- a/fs/ext3/namei.c
+++ b/fs/ext3/namei.c
@@ -1047,7 +1047,7 @@ static struct dentry *ext3_lookup(struct inode * dir, struct dentry *dentry, str
return ERR_PTR(-EIO);
}
inode = ext3_iget(dir->i_sb, ino);
- if (unlikely(IS_ERR(inode))) {
+ if (IS_ERR(inode)) {
if (PTR_ERR(inode) == -ESTALE) {
ext3_error(dir->i_sb, __func__,
"deleted inode referenced: %lu",
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 12/15] ext4: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (10 preceding siblings ...)
2010-12-09 20:04 ` [PATCH 11/15] ext3: " Joe Perches
@ 2010-12-09 20:04 ` Joe Perches
2010-12-09 20:04 ` [PATCH 13/15] nfs: " Joe Perches
` (3 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:04 UTC (permalink / raw)
To: Theodore Ts'o, Andreas Dilger; +Cc: Jiri Kosina, linux-ext4, linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
fs/ext4/namei.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c
index 92203b8..cec88ce 100644
--- a/fs/ext4/namei.c
+++ b/fs/ext4/namei.c
@@ -1036,7 +1036,7 @@ static struct dentry *ext4_lookup(struct inode *dir, struct dentry *dentry, stru
return ERR_PTR(-EIO);
}
inode = ext4_iget(dir->i_sb, ino);
- if (unlikely(IS_ERR(inode))) {
+ if (IS_ERR(inode)) {
if (PTR_ERR(inode) == -ESTALE) {
EXT4_ERROR_INODE(dir,
"deleted inode referenced: %u",
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 13/15] nfs: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (11 preceding siblings ...)
2010-12-09 20:04 ` [PATCH 12/15] ext4: " Joe Perches
@ 2010-12-09 20:04 ` Joe Perches
2010-12-09 20:04 ` [PATCH 14/15] mm: " Joe Perches
` (2 subsequent siblings)
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:04 UTC (permalink / raw)
To: Trond Myklebust; +Cc: Jiri Kosina, linux-nfs, linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
fs/nfs/mount_clnt.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/nfs/mount_clnt.c b/fs/nfs/mount_clnt.c
index eceafe7..3f2a6be 100644
--- a/fs/nfs/mount_clnt.c
+++ b/fs/nfs/mount_clnt.c
@@ -248,7 +248,7 @@ void nfs_umount(const struct nfs_mount_request *info)
args.flags |= RPC_CLNT_CREATE_NONPRIVPORT;
clnt = rpc_create(&args);
- if (unlikely(IS_ERR(clnt)))
+ if (IS_ERR(clnt))
goto out_clnt_err;
dprintk("NFS: sending UMNT request for %s:%s\n",
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 14/15] mm: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (12 preceding siblings ...)
2010-12-09 20:04 ` [PATCH 13/15] nfs: " Joe Perches
@ 2010-12-09 20:04 ` Joe Perches
2010-12-09 20:04 ` [PATCH 15/15] ipv6: " Joe Perches
2010-12-09 20:32 ` [trivial PATCH 00/15] remove " Joe Perches
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:04 UTC (permalink / raw)
To: linux-kernel; +Cc: Jiri Kosina, linux-mm
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
mm/vmalloc.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/mm/vmalloc.c b/mm/vmalloc.c
index eb5cc7d..31dcb64 100644
--- a/mm/vmalloc.c
+++ b/mm/vmalloc.c
@@ -748,7 +748,7 @@ static struct vmap_block *new_vmap_block(gfp_t gfp_mask)
va = alloc_vmap_area(VMAP_BLOCK_SIZE, VMAP_BLOCK_SIZE,
VMALLOC_START, VMALLOC_END,
node, gfp_mask);
- if (unlikely(IS_ERR(va))) {
+ if (IS_ERR(va)) {
kfree(vb);
return ERR_CAST(va);
}
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 15/15] ipv6: Remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (13 preceding siblings ...)
2010-12-09 20:04 ` [PATCH 14/15] mm: " Joe Perches
@ 2010-12-09 20:04 ` Joe Perches
2010-12-09 20:32 ` [trivial PATCH 00/15] remove " Joe Perches
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:04 UTC (permalink / raw)
To: David S. Miller, Alexey Kuznetsov, Pekka Savola (ipv6),
James Morris, Hideaki YOSHIFUJI, Patrick McHardy
Cc: Jiri Kosina, netdev, linux-kernel
IS_ERR already uses unlikely, remove unlikely from the call sites.
Signed-off-by: Joe Perches <joe@perches.com>
---
net/ipv6/af_inet6.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/net/ipv6/af_inet6.c b/net/ipv6/af_inet6.c
index 54e8e42..059a3de 100644
--- a/net/ipv6/af_inet6.c
+++ b/net/ipv6/af_inet6.c
@@ -810,7 +810,7 @@ static struct sk_buff *ipv6_gso_segment(struct sk_buff *skb, int features)
}
rcu_read_unlock();
- if (unlikely(IS_ERR(segs)))
+ if (IS_ERR(segs))
goto out;
for (skb = segs; skb; skb = skb->next) {
--
1.7.3.3.464.gf80b6
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR
2010-12-09 20:03 ` [trivial PATCH 00/15] remove duplicate unlikely from IS_ERR Joe Perches
` (14 preceding siblings ...)
2010-12-09 20:04 ` [PATCH 15/15] ipv6: " Joe Perches
@ 2010-12-09 20:32 ` Joe Perches
15 siblings, 0 replies; 22+ messages in thread
From: Joe Perches @ 2010-12-09 20:32 UTC (permalink / raw)
To: netdev, Tobias Klauser
Cc: uclinux-dist-devel, rtc-linux, linux-s390, osd-dev, linux-arm-msm,
linux-usb, linux-ext4, linux-nfs, linux-mm, Jiri Kosina,
dri-devel, linux-kernel, linux-scsi, linux-wireless, devel
On Thu, 2010-12-09 at 12:03 -0800, Joe Perches wrote:
> Tobias Klauser <tklauser@distanz.ch> sent a patch to remove
> an unnecessary unlikely from drivers/misc/c2port/core.c,
> https://lkml.org/lkml/2010/12/9/199
It seems that Tobias did send all the appropriate patches,
not as a series, but as individual patches to kernel-janitor.
c2port was the only one that went to lkml.
Please ignore this series and apply Tobias' patches.
^ permalink raw reply [flat|nested] 22+ messages in thread