summaryrefslogtreecommitdiff
path: root/drivers/input/evdev.c
diff options
context:
space:
mode:
authorEric Dumazet <eric.dumazet@gmail.com>2011-05-12 02:38:10 +0400
committerDmitry Torokhov <dmitry.torokhov@gmail.com>2011-05-12 19:28:57 +0400
commit7cbbb758d3c93b24b45b169af55440d2e7d5b7f6 (patch)
tree2eda8bad7a0f92c67b2255e9076e892309e85011 /drivers/input/evdev.c
parent1201e7e67634624aec89317cf308cd0d3d8d4770 (diff)
downloadlinux-7cbbb758d3c93b24b45b169af55440d2e7d5b7f6.tar.xz
Input: remove useless synchronize_rcu() calls
There is no need to call synchronize_rcu() after a list insertion, or a NULL->ptr assignment. However, the reverse operations do need this call. Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com> Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Diffstat (limited to 'drivers/input/evdev.c')
-rw-r--r--drivers/input/evdev.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c
index 75c4e8683033..be0921ef6b52 100644
--- a/drivers/input/evdev.c
+++ b/drivers/input/evdev.c
@@ -164,7 +164,6 @@ static int evdev_grab(struct evdev *evdev, struct evdev_client *client)
return error;
rcu_assign_pointer(evdev->grab, client);
- synchronize_rcu();
return 0;
}
@@ -187,7 +186,6 @@ static void evdev_attach_client(struct evdev *evdev,
spin_lock(&evdev->client_lock);
list_add_tail_rcu(&client->node, &evdev->client_list);
spin_unlock(&evdev->client_lock);
- synchronize_rcu();
}
static void evdev_detach_client(struct evdev *evdev,