* [PATCH 2/3] Input: alps: drop unlikely before IS_ERR_OR_NULL [not found] <03d18502ed7ed417f136c091f417d2d88c147ec6.1443667610.git.geliangtang@163.com> @ 2015-10-01 2:55 ` Geliang Tang 2015-10-01 7:30 ` Pali Rohár 0 siblings, 1 reply; 3+ messages in thread From: Geliang Tang @ 2015-10-01 2:55 UTC (permalink / raw) To: Dmitry Torokhov, Hans de Goede, Pali Rohár, Masaki Ota Cc: Geliang Tang, linux-input, linux-kernel IS_ERR_OR_NULL already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang <geliangtang@163.com> --- drivers/input/mouse/alps.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c index 4d24686..b4f146a 100644 --- a/drivers/input/mouse/alps.c +++ b/drivers/input/mouse/alps.c @@ -1367,7 +1367,7 @@ static void alps_report_bare_ps2_packet(struct psmouse *psmouse, /* On V2 devices the DualPoint Stick reports bare packets */ dev = priv->dev2; dev2 = psmouse->dev; - } else if (unlikely(IS_ERR_OR_NULL(priv->dev3))) { + } else if (IS_ERR_OR_NULL(priv->dev3)) { /* Register dev3 mouse if we received PS/2 packet first time */ if (!IS_ERR(priv->dev3)) psmouse_queue_work(psmouse, &priv->dev3_register_work, -- 2.5.0 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 2/3] Input: alps: drop unlikely before IS_ERR_OR_NULL 2015-10-01 2:55 ` [PATCH 2/3] Input: alps: drop unlikely before IS_ERR_OR_NULL Geliang Tang @ 2015-10-01 7:30 ` Pali Rohár 2015-10-02 18:29 ` Dmitry Torokhov 0 siblings, 1 reply; 3+ messages in thread From: Pali Rohár @ 2015-10-01 7:30 UTC (permalink / raw) To: Geliang Tang Cc: Dmitry Torokhov, Hans de Goede, Masaki Ota, linux-input, linux-kernel On Thursday 01 October 2015 10:55:30 Geliang Tang wrote: > IS_ERR_OR_NULL already contain an unlikely compiler flag. Drop it. > > Signed-off-by: Geliang Tang <geliangtang@163.com> > --- > drivers/input/mouse/alps.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c > index 4d24686..b4f146a 100644 > --- a/drivers/input/mouse/alps.c > +++ b/drivers/input/mouse/alps.c > @@ -1367,7 +1367,7 @@ static void alps_report_bare_ps2_packet(struct psmouse *psmouse, > /* On V2 devices the DualPoint Stick reports bare packets */ > dev = priv->dev2; > dev2 = psmouse->dev; > - } else if (unlikely(IS_ERR_OR_NULL(priv->dev3))) { > + } else if (IS_ERR_OR_NULL(priv->dev3)) { > /* Register dev3 mouse if we received PS/2 packet first time */ > if (!IS_ERR(priv->dev3)) > psmouse_queue_work(psmouse, &priv->dev3_register_work, Hm... I do not like this change. If I read code if (unlikely(IS_ERR_OR_NULL(priv->dev3))) then I know that it is really unlikely that condition will be truth and so this is some case of error/exception or something that normally does not happen too much. But if I read code if (IS_ERR_OR_NULL(priv->dev3)) I know nothing about chance that this condition will be truth. Explicit unlikely in previous example give me more information. -- Pali Rohár pali.rohar@gmail.com -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 2/3] Input: alps: drop unlikely before IS_ERR_OR_NULL 2015-10-01 7:30 ` Pali Rohár @ 2015-10-02 18:29 ` Dmitry Torokhov 0 siblings, 0 replies; 3+ messages in thread From: Dmitry Torokhov @ 2015-10-02 18:29 UTC (permalink / raw) To: Pali Rohár Cc: Geliang Tang, Hans de Goede, Masaki Ota, linux-input, linux-kernel On Thu, Oct 01, 2015 at 09:30:19AM +0200, Pali Rohár wrote: > On Thursday 01 October 2015 10:55:30 Geliang Tang wrote: > > IS_ERR_OR_NULL already contain an unlikely compiler flag. Drop it. > > > > Signed-off-by: Geliang Tang <geliangtang@163.com> > > --- > > drivers/input/mouse/alps.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c > > index 4d24686..b4f146a 100644 > > --- a/drivers/input/mouse/alps.c > > +++ b/drivers/input/mouse/alps.c > > @@ -1367,7 +1367,7 @@ static void alps_report_bare_ps2_packet(struct psmouse *psmouse, > > /* On V2 devices the DualPoint Stick reports bare packets */ > > dev = priv->dev2; > > dev2 = psmouse->dev; > > - } else if (unlikely(IS_ERR_OR_NULL(priv->dev3))) { > > + } else if (IS_ERR_OR_NULL(priv->dev3)) { > > /* Register dev3 mouse if we received PS/2 packet first time */ > > if (!IS_ERR(priv->dev3)) > > psmouse_queue_work(psmouse, &priv->dev3_register_work, > > Hm... I do not like this change. If I read code > > if (unlikely(IS_ERR_OR_NULL(priv->dev3))) > > then I know that it is really unlikely that condition will be truth and > so this is some case of error/exception or something that normally does > not happen too much. > > But if I read code > > if (IS_ERR_OR_NULL(priv->dev3)) > > I know nothing about chance that this condition will be truth. Explicit > unlikely in previous example give me more information. Yes, given that this is in packet processing path I prefer having explicit unlikely there. Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-10-02 18:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <03d18502ed7ed417f136c091f417d2d88c147ec6.1443667610.git.geliangtang@163.com>
2015-10-01 2:55 ` [PATCH 2/3] Input: alps: drop unlikely before IS_ERR_OR_NULL Geliang Tang
2015-10-01 7:30 ` Pali Rohár
2015-10-02 18:29 ` Dmitry Torokhov
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).