diff options
| author | Eric Dumazet <eric.dumazet@gmail.com> | 2011-05-11 18:38:10 -0400 |
|---|---|---|
| committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2011-05-12 11:28:57 -0400 |
| commit | 7cbbb758d3c93b24b45b169af55440d2e7d5b7f6 (patch) | |
| tree | 2eda8bad7a0f92c67b2255e9076e892309e85011 /drivers/input/joydev.c | |
| parent | 1201e7e67634624aec89317cf308cd0d3d8d4770 (diff) | |
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/joydev.c')
| -rw-r--r-- | drivers/input/joydev.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/input/joydev.c b/drivers/input/joydev.c index 5688b5c88f24..c24ec2d5f926 100644 --- a/drivers/input/joydev.c +++ b/drivers/input/joydev.c | |||
| @@ -180,7 +180,6 @@ static void joydev_attach_client(struct joydev *joydev, | |||
| 180 | spin_lock(&joydev->client_lock); | 180 | spin_lock(&joydev->client_lock); |
| 181 | list_add_tail_rcu(&client->node, &joydev->client_list); | 181 | list_add_tail_rcu(&client->node, &joydev->client_list); |
| 182 | spin_unlock(&joydev->client_lock); | 182 | spin_unlock(&joydev->client_lock); |
| 183 | synchronize_rcu(); | ||
| 184 | } | 183 | } |
| 185 | 184 | ||
| 186 | static void joydev_detach_client(struct joydev *joydev, | 185 | static void joydev_detach_client(struct joydev *joydev, |