diff options
author | Pavel Emelyanov <xemul@openvz.org> | 2008-01-24 00:20:07 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 17:53:35 -0500 |
commit | b24b8a247ff65c01b252025926fe564209fae4fc (patch) | |
tree | 8a9e0ea1e24b4733d8b9433d41877659505e9da4 /net/bluetooth | |
parent | a92aa318b4b369091fd80433c80e62838db8bc1c (diff) |
[NET]: Convert init_timer into setup_timer
Many-many code in the kernel initialized the timer->function
and timer->data together with calling init_timer(timer). There
is already a helper for this. Use it for networking code.
The patch is HUGE, but makes the code 130 lines shorter
(98 insertions(+), 228 deletions(-)).
Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
Acked-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/bluetooth')
-rw-r--r-- | net/bluetooth/hci_conn.c | 9 | ||||
-rw-r--r-- | net/bluetooth/hidp/core.c | 5 | ||||
-rw-r--r-- | net/bluetooth/l2cap.c | 13 | ||||
-rw-r--r-- | net/bluetooth/rfcomm/core.c | 4 | ||||
-rw-r--r-- | net/bluetooth/sco.c | 9 |
5 files changed, 7 insertions, 33 deletions
diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c index 34d1a3c822bf..5fc7be206f62 100644 --- a/net/bluetooth/hci_conn.c +++ b/net/bluetooth/hci_conn.c | |||
@@ -208,13 +208,8 @@ struct hci_conn *hci_conn_add(struct hci_dev *hdev, int type, bdaddr_t *dst) | |||
208 | 208 | ||
209 | skb_queue_head_init(&conn->data_q); | 209 | skb_queue_head_init(&conn->data_q); |
210 | 210 | ||
211 | init_timer(&conn->disc_timer); | 211 | setup_timer(&conn->disc_timer, hci_conn_timeout, (unsigned long)conn); |
212 | conn->disc_timer.function = hci_conn_timeout; | 212 | setup_timer(&conn->idle_timer, hci_conn_idle, (unsigned long)conn); |
213 | conn->disc_timer.data = (unsigned long) conn; | ||
214 | |||
215 | init_timer(&conn->idle_timer); | ||
216 | conn->idle_timer.function = hci_conn_idle; | ||
217 | conn->idle_timer.data = (unsigned long) conn; | ||
218 | 213 | ||
219 | atomic_set(&conn->refcnt, 0); | 214 | atomic_set(&conn->refcnt, 0); |
220 | 215 | ||
diff --git a/net/bluetooth/hidp/core.c b/net/bluetooth/hidp/core.c index 4bbacddeb49d..782a22602b86 100644 --- a/net/bluetooth/hidp/core.c +++ b/net/bluetooth/hidp/core.c | |||
@@ -811,10 +811,7 @@ int hidp_add_connection(struct hidp_connadd_req *req, struct socket *ctrl_sock, | |||
811 | session->intr_sock = intr_sock; | 811 | session->intr_sock = intr_sock; |
812 | session->state = BT_CONNECTED; | 812 | session->state = BT_CONNECTED; |
813 | 813 | ||
814 | init_timer(&session->timer); | 814 | setup_timer(&session->timer, hidp_idle_timeout, (unsigned long)session); |
815 | |||
816 | session->timer.function = hidp_idle_timeout; | ||
817 | session->timer.data = (unsigned long) session; | ||
818 | 815 | ||
819 | skb_queue_head_init(&session->ctrl_transmit); | 816 | skb_queue_head_init(&session->ctrl_transmit); |
820 | skb_queue_head_init(&session->intr_transmit); | 817 | skb_queue_head_init(&session->intr_transmit); |
diff --git a/net/bluetooth/l2cap.c b/net/bluetooth/l2cap.c index 477e052b17b5..a8811c0a0cea 100644 --- a/net/bluetooth/l2cap.c +++ b/net/bluetooth/l2cap.c | |||
@@ -99,13 +99,6 @@ static void l2cap_sock_clear_timer(struct sock *sk) | |||
99 | sk_stop_timer(sk, &sk->sk_timer); | 99 | sk_stop_timer(sk, &sk->sk_timer); |
100 | } | 100 | } |
101 | 101 | ||
102 | static void l2cap_sock_init_timer(struct sock *sk) | ||
103 | { | ||
104 | init_timer(&sk->sk_timer); | ||
105 | sk->sk_timer.function = l2cap_sock_timeout; | ||
106 | sk->sk_timer.data = (unsigned long)sk; | ||
107 | } | ||
108 | |||
109 | /* ---- L2CAP channels ---- */ | 102 | /* ---- L2CAP channels ---- */ |
110 | static struct sock *__l2cap_get_chan_by_dcid(struct l2cap_chan_list *l, u16 cid) | 103 | static struct sock *__l2cap_get_chan_by_dcid(struct l2cap_chan_list *l, u16 cid) |
111 | { | 104 | { |
@@ -395,9 +388,7 @@ static struct l2cap_conn *l2cap_conn_add(struct hci_conn *hcon, u8 status) | |||
395 | 388 | ||
396 | conn->feat_mask = 0; | 389 | conn->feat_mask = 0; |
397 | 390 | ||
398 | init_timer(&conn->info_timer); | 391 | setup_timer(&conn->info_timer, l2cap_info_timeout, (unsigned long)conn); |
399 | conn->info_timer.function = l2cap_info_timeout; | ||
400 | conn->info_timer.data = (unsigned long) conn; | ||
401 | 392 | ||
402 | spin_lock_init(&conn->lock); | 393 | spin_lock_init(&conn->lock); |
403 | rwlock_init(&conn->chan_list.lock); | 394 | rwlock_init(&conn->chan_list.lock); |
@@ -622,7 +613,7 @@ static struct sock *l2cap_sock_alloc(struct net *net, struct socket *sock, int p | |||
622 | sk->sk_protocol = proto; | 613 | sk->sk_protocol = proto; |
623 | sk->sk_state = BT_OPEN; | 614 | sk->sk_state = BT_OPEN; |
624 | 615 | ||
625 | l2cap_sock_init_timer(sk); | 616 | setup_timer(&sk->sk_timer, l2cap_sock_timeout, (unsigned long)sk); |
626 | 617 | ||
627 | bt_sock_link(&l2cap_sk_list, sk); | 618 | bt_sock_link(&l2cap_sk_list, sk); |
628 | return sk; | 619 | return sk; |
diff --git a/net/bluetooth/rfcomm/core.c b/net/bluetooth/rfcomm/core.c index e7ac6ba7ecab..d3e4e1877e6a 100644 --- a/net/bluetooth/rfcomm/core.c +++ b/net/bluetooth/rfcomm/core.c | |||
@@ -279,9 +279,7 @@ struct rfcomm_dlc *rfcomm_dlc_alloc(gfp_t prio) | |||
279 | if (!d) | 279 | if (!d) |
280 | return NULL; | 280 | return NULL; |
281 | 281 | ||
282 | init_timer(&d->timer); | 282 | setup_timer(&d->timer, rfcomm_dlc_timeout, (unsigned long)d); |
283 | d->timer.function = rfcomm_dlc_timeout; | ||
284 | d->timer.data = (unsigned long) d; | ||
285 | 283 | ||
286 | skb_queue_head_init(&d->tx_queue); | 284 | skb_queue_head_init(&d->tx_queue); |
287 | spin_lock_init(&d->lock); | 285 | spin_lock_init(&d->lock); |
diff --git a/net/bluetooth/sco.c b/net/bluetooth/sco.c index 93ad1aae3f38..b91d3c81a73c 100644 --- a/net/bluetooth/sco.c +++ b/net/bluetooth/sco.c | |||
@@ -97,13 +97,6 @@ static void sco_sock_clear_timer(struct sock *sk) | |||
97 | sk_stop_timer(sk, &sk->sk_timer); | 97 | sk_stop_timer(sk, &sk->sk_timer); |
98 | } | 98 | } |
99 | 99 | ||
100 | static void sco_sock_init_timer(struct sock *sk) | ||
101 | { | ||
102 | init_timer(&sk->sk_timer); | ||
103 | sk->sk_timer.function = sco_sock_timeout; | ||
104 | sk->sk_timer.data = (unsigned long)sk; | ||
105 | } | ||
106 | |||
107 | /* ---- SCO connections ---- */ | 100 | /* ---- SCO connections ---- */ |
108 | static struct sco_conn *sco_conn_add(struct hci_conn *hcon, __u8 status) | 101 | static struct sco_conn *sco_conn_add(struct hci_conn *hcon, __u8 status) |
109 | { | 102 | { |
@@ -436,7 +429,7 @@ static struct sock *sco_sock_alloc(struct net *net, struct socket *sock, int pro | |||
436 | sk->sk_protocol = proto; | 429 | sk->sk_protocol = proto; |
437 | sk->sk_state = BT_OPEN; | 430 | sk->sk_state = BT_OPEN; |
438 | 431 | ||
439 | sco_sock_init_timer(sk); | 432 | setup_timer(&sk->sk_timer, sco_sock_timeout, (unsigned long)sk); |
440 | 433 | ||
441 | bt_sock_link(&sco_sk_list, sk); | 434 | bt_sock_link(&sco_sk_list, sk); |
442 | return sk; | 435 | return sk; |