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/mousedev.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/mousedev.c')
| -rw-r--r-- | drivers/input/mousedev.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/input/mousedev.c b/drivers/input/mousedev.c index 7630273e9474..257e033986e4 100644 --- a/drivers/input/mousedev.c +++ b/drivers/input/mousedev.c | |||
| @@ -508,7 +508,6 @@ static void mousedev_attach_client(struct mousedev *mousedev, | |||
| 508 | spin_lock(&mousedev->client_lock); | 508 | spin_lock(&mousedev->client_lock); |
| 509 | list_add_tail_rcu(&client->node, &mousedev->client_list); | 509 | list_add_tail_rcu(&client->node, &mousedev->client_list); |
| 510 | spin_unlock(&mousedev->client_lock); | 510 | spin_unlock(&mousedev->client_lock); |
| 511 | synchronize_rcu(); | ||
| 512 | } | 511 | } |
| 513 | 512 | ||
| 514 | static void mousedev_detach_client(struct mousedev *mousedev, | 513 | static void mousedev_detach_client(struct mousedev *mousedev, |