From: Wei Yongjun <yongjun_...@trendmicro.com.cn>

Using list_move() instead of list_del() + list_add().

spatch with a semantic match is used to found this problem.
(http://coccinelle.lip6.fr/)

Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn>
---
 kernel/time/clockevents.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/kernel/time/clockevents.c b/kernel/time/clockevents.c
index 7e1ce01..8c4aac5 100644
--- a/kernel/time/clockevents.c
+++ b/kernel/time/clockevents.c
@@ -267,8 +267,7 @@ static void clockevents_notify_released(void)
        while (!list_empty(&clockevents_released)) {
                dev = list_entry(clockevents_released.next,
                                 struct clock_event_device, list);
-               list_del(&dev->list);
-               list_add(&dev->list, &clockevent_devices);
+               list_move(&dev->list, &clockevent_devices);
                clockevents_do_notify(CLOCK_EVT_NOTIFY_ADD, dev);
        }
 }
@@ -386,8 +385,7 @@ void clockevents_exchange_device(struct clock_event_device 
*old,
         */
        if (old) {
                clockevents_set_mode(old, CLOCK_EVT_MODE_UNUSED);
-               list_del(&old->list);
-               list_add(&old->list, &clockevents_released);
+               list_move(&old->list, &clockevents_released);
        }
 
        if (new) {

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to