* [PATCH] input: mouse: appletouch: Replace GFP_ATOMIC with GFP_KERNEL
@ 2018-07-27 2:26 Jia-Ju Bai
2018-07-27 18:36 ` Dmitry Torokhov
0 siblings, 1 reply; 2+ messages in thread
From: Jia-Ju Bai @ 2018-07-27 2:26 UTC (permalink / raw)
To: dmitry.torokhov, gustavo, arvind.yadav.cs
Cc: linux-input, linux-kernel, Jia-Ju Bai
atp_open(), atp_recover() and atp_resume() are never
called in atomic context.
They call usb_submit_urb() with GFP_ATOMIC, which is not necessary.
GFP_ATOMIC can be replaced with GFP_KERNEL.
This is found by a static analysis tool named DCNS written by myself.
Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
---
drivers/input/mouse/appletouch.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/input/mouse/appletouch.c b/drivers/input/mouse/appletouch.c
index 032d27983b6c..f593ec96c95f 100644
--- a/drivers/input/mouse/appletouch.c
+++ b/drivers/input/mouse/appletouch.c
@@ -810,7 +810,7 @@ static int atp_open(struct input_dev *input)
{
struct atp *dev = input_get_drvdata(input);
- if (usb_submit_urb(dev->urb, GFP_ATOMIC))
+ if (usb_submit_urb(dev->urb, GFP_KERNEL))
return -EIO;
dev->open = true;
@@ -976,7 +976,7 @@ static int atp_recover(struct atp *dev)
if (error)
return error;
- if (dev->open && usb_submit_urb(dev->urb, GFP_ATOMIC))
+ if (dev->open && usb_submit_urb(dev->urb, GFP_KERNEL))
return -EIO;
return 0;
@@ -994,7 +994,7 @@ static int atp_resume(struct usb_interface *iface)
{
struct atp *dev = usb_get_intfdata(iface);
- if (dev->open && usb_submit_urb(dev->urb, GFP_ATOMIC))
+ if (dev->open && usb_submit_urb(dev->urb, GFP_KERNEL))
return -EIO;
return 0;
--
2.17.0
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] input: mouse: appletouch: Replace GFP_ATOMIC with GFP_KERNEL
2018-07-27 2:26 [PATCH] input: mouse: appletouch: Replace GFP_ATOMIC with GFP_KERNEL Jia-Ju Bai
@ 2018-07-27 18:36 ` Dmitry Torokhov
0 siblings, 0 replies; 2+ messages in thread
From: Dmitry Torokhov @ 2018-07-27 18:36 UTC (permalink / raw)
To: Jia-Ju Bai; +Cc: gustavo, arvind.yadav.cs, linux-input, linux-kernel
On Fri, Jul 27, 2018 at 10:26:18AM +0800, Jia-Ju Bai wrote:
> atp_open(), atp_recover() and atp_resume() are never
> called in atomic context.
> They call usb_submit_urb() with GFP_ATOMIC, which is not necessary.
> GFP_ATOMIC can be replaced with GFP_KERNEL.
>
> This is found by a static analysis tool named DCNS written by myself.
>
> Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
Applied, thank you.
> ---
> drivers/input/mouse/appletouch.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/input/mouse/appletouch.c b/drivers/input/mouse/appletouch.c
> index 032d27983b6c..f593ec96c95f 100644
> --- a/drivers/input/mouse/appletouch.c
> +++ b/drivers/input/mouse/appletouch.c
> @@ -810,7 +810,7 @@ static int atp_open(struct input_dev *input)
> {
> struct atp *dev = input_get_drvdata(input);
>
> - if (usb_submit_urb(dev->urb, GFP_ATOMIC))
> + if (usb_submit_urb(dev->urb, GFP_KERNEL))
> return -EIO;
>
> dev->open = true;
> @@ -976,7 +976,7 @@ static int atp_recover(struct atp *dev)
> if (error)
> return error;
>
> - if (dev->open && usb_submit_urb(dev->urb, GFP_ATOMIC))
> + if (dev->open && usb_submit_urb(dev->urb, GFP_KERNEL))
> return -EIO;
>
> return 0;
> @@ -994,7 +994,7 @@ static int atp_resume(struct usb_interface *iface)
> {
> struct atp *dev = usb_get_intfdata(iface);
>
> - if (dev->open && usb_submit_urb(dev->urb, GFP_ATOMIC))
> + if (dev->open && usb_submit_urb(dev->urb, GFP_KERNEL))
> return -EIO;
>
> return 0;
> --
> 2.17.0
>
--
Dmitry
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-07-27 18:36 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-07-27 2:26 [PATCH] input: mouse: appletouch: Replace GFP_ATOMIC with GFP_KERNEL Jia-Ju Bai
2018-07-27 18:36 ` 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).