Commit 2f55c437 authored by Ying Xue's avatar Ying Xue Committed by David S. Miller
Browse files

tipc: remove unnecessary wrapper functions of kernel timer APIs 



Not only some wrapper function like k_term_timer() is empty, but also
some others including k_start_timer() and k_cancel_timer() don't return
back any value to its caller, what's more, there is no any component
in the kernel world to do such thing. Therefore, these timer interfaces
defined in tipc module should be purged.

Signed-off-by: default avatarYing Xue <ying.xue@windriver.com>
Tested-by: default avatarTero Aho <Tero.Aho@coriant.com>
Reviewed-by: default avatarJon Maloy <jon.maloy@ericsson.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 6b8326ed

net/tipc/core.h

+0 −67
Original line number Diff line number Diff line
@@ -106,73 +106,6 @@ void tipc_unregister_sysctl(void); 
#define tipc_unregister_sysctl()

#endif



/*

 * TIPC timer code

 */

typedef void (*Handler) (unsigned long);



/**

 * k_init_timer - initialize a timer

 * @timer: pointer to timer structure

 * @routine: pointer to routine to invoke when timer expires

 * @argument: value to pass to routine when timer expires

 *

 * Timer must be initialized before use (and terminated when no longer needed).

 */

static inline void k_init_timer(struct timer_list *timer, Handler routine,

				unsigned long argument)

{

	setup_timer(timer, routine, argument);

}



/**

 * k_start_timer - start a timer

 * @timer: pointer to timer structure

 * @msec: time to delay (in ms)

 *

 * Schedules a previously initialized timer for later execution.

 * If timer is already running, the new timeout overrides the previous request.

 *

 * To ensure the timer doesn't expire before the specified delay elapses,

 * the amount of delay is rounded up when converting to the jiffies

 * then an additional jiffy is added to account for the fact that

 * the starting time may be in the middle of the current jiffy.

 */

static inline void k_start_timer(struct timer_list *timer, unsigned long msec)

{

	mod_timer(timer, jiffies + msecs_to_jiffies(msec) + 1);

}



/**

 * k_cancel_timer - cancel a timer

 * @timer: pointer to timer structure

 *

 * Cancels a previously initialized timer.

 * Can be called safely even if the timer is already inactive.

 *

 * WARNING: Must not be called when holding locks required by the timer's

 *          timeout routine, otherwise deadlock can occur on SMP systems!

 */

static inline void k_cancel_timer(struct timer_list *timer)

{

	del_timer_sync(timer);

}



/**

 * k_term_timer - terminate a timer

 * @timer: pointer to timer structure

 *

 * Prevents further use of a previously initialized timer.

 *

 * WARNING: Caller must ensure timer isn't currently running.

 *

 * (Do not "enhance" this routine to automatically cancel an active timer,

 * otherwise deadlock can arise when a timeout routine calls k_term_timer.)

 */

static inline void k_term_timer(struct timer_list *timer)

{

}



/*

 * TIPC message buffer code

 *

net/tipc/discover.c

+18 −14
Original line number Diff line number Diff line
@@ -38,10 +38,14 @@ 
#include "link.h"

#include "discover.h"



#define TIPC_LINK_REQ_INIT	125	/* min delay during bearer start up */

#define TIPC_LINK_REQ_FAST	1000	/* max delay if bearer has no links */

#define TIPC_LINK_REQ_SLOW	60000	/* max delay if bearer has links */

#define TIPC_LINK_REQ_INACTIVE	0xffffffff /* indicates no timer in use */

/* min delay during bearer start up */

#define TIPC_LINK_REQ_INIT	msecs_to_jiffies(125)

/* max delay if bearer has no links */

#define TIPC_LINK_REQ_FAST	msecs_to_jiffies(1000)

/* max delay if bearer has links */

#define TIPC_LINK_REQ_SLOW	msecs_to_jiffies(60000)

/* indicates no timer in use */

#define TIPC_LINK_REQ_INACTIVE	0xffffffff





/**
@@ -63,7 +67,7 @@ struct tipc_link_req { 
	spinlock_t lock;

	struct sk_buff *buf;

	struct timer_list timer;

	unsigned int timer_intv;

	unsigned long timer_intv;

};



/**
@@ -265,7 +269,7 @@ static void disc_update(struct tipc_link_req *req) 
		if ((req->timer_intv == TIPC_LINK_REQ_INACTIVE) ||

		    (req->timer_intv > TIPC_LINK_REQ_FAST)) {

			req->timer_intv = TIPC_LINK_REQ_INIT;

			k_start_timer(&req->timer, req->timer_intv);

			mod_timer(&req->timer, jiffies + req->timer_intv);

		}

	}

}
@@ -295,12 +299,13 @@ void tipc_disc_remove_dest(struct tipc_link_req *req) 


/**

 * disc_timeout - send a periodic link setup request

 * @req: ptr to link request structure

 * @data: ptr to link request structure

 *

 * Called whenever a link setup request timer associated with a bearer expires.

 */

static void disc_timeout(struct tipc_link_req *req)

static void disc_timeout(unsigned long data)

{

	struct tipc_link_req *req = (struct tipc_link_req *)data;

	int max_delay;



	spin_lock_bh(&req->lock);
@@ -329,7 +334,7 @@ static void disc_timeout(struct tipc_link_req *req) 
	if (req->timer_intv > max_delay)

		req->timer_intv = max_delay;



	k_start_timer(&req->timer, req->timer_intv);

	mod_timer(&req->timer, jiffies + req->timer_intv);

exit:

	spin_unlock_bh(&req->lock);

}
@@ -363,8 +368,8 @@ int tipc_disc_create(struct tipc_bearer *b_ptr, struct tipc_media_addr *dest) 
	req->num_nodes = 0;

	req->timer_intv = TIPC_LINK_REQ_INIT;

	spin_lock_init(&req->lock);

	k_init_timer(&req->timer, (Handler)disc_timeout, (unsigned long)req);

	k_start_timer(&req->timer, req->timer_intv);

	setup_timer(&req->timer, disc_timeout, (unsigned long)req);

	mod_timer(&req->timer, jiffies + req->timer_intv);

	b_ptr->link_req = req;

	tipc_bearer_send(req->bearer_id, req->buf, &req->dest);

	return 0;
@@ -376,8 +381,7 @@ int tipc_disc_create(struct tipc_bearer *b_ptr, struct tipc_media_addr *dest) 
 */

void tipc_disc_delete(struct tipc_link_req *req)

{

	k_cancel_timer(&req->timer);

	k_term_timer(&req->timer);

	del_timer_sync(&req->timer);

	kfree_skb(req->buf);

	kfree(req);

}
@@ -397,7 +401,7 @@ void tipc_disc_reset(struct tipc_bearer *b_ptr) 
	req->domain = b_ptr->domain;

	req->num_nodes = 0;

	req->timer_intv = TIPC_LINK_REQ_INIT;

	k_start_timer(&req->timer, req->timer_intv);

	mod_timer(&req->timer, jiffies + req->timer_intv);

	tipc_bearer_send(req->bearer_id, req->buf, &req->dest);

	spin_unlock_bh(&req->lock);

}

net/tipc/link.c

+14 −13
Original line number Diff line number Diff line
@@ -106,7 +106,7 @@ static void link_handle_out_of_seq_msg(struct tipc_link *l_ptr, 
static void tipc_link_proto_rcv(struct tipc_link *l_ptr, struct sk_buff *buf);

static int  tipc_link_tunnel_rcv(struct tipc_node *n_ptr,

				 struct sk_buff **buf);

static void link_set_supervision_props(struct tipc_link *l_ptr, u32 tolerance);

static void link_set_supervision_props(struct tipc_link *l_ptr, u32 tol);

static void link_state_event(struct tipc_link *l_ptr, u32 event);

static void link_reset_statistics(struct tipc_link *l_ptr);

static void link_print(struct tipc_link *l_ptr, const char *str);
@@ -169,8 +169,9 @@ int tipc_link_is_active(struct tipc_link *l_ptr) 
 * link_timeout - handle expiration of link timer

 * @l_ptr: pointer to link

 */

static void link_timeout(struct tipc_link *l_ptr)

static void link_timeout(unsigned long data)

{

	struct tipc_link *l_ptr = (struct tipc_link *)data;

	struct sk_buff *skb;



	tipc_node_lock(l_ptr->owner);
@@ -217,9 +218,9 @@ static void link_timeout(struct tipc_link *l_ptr) 
	tipc_node_unlock(l_ptr->owner);

}



static void link_set_timer(struct tipc_link *l_ptr, u32 time)

static void link_set_timer(struct tipc_link *link, unsigned long time)

{

	k_start_timer(&l_ptr->timer, time);

	mod_timer(&link->timer, jiffies + time);

}



/**
@@ -299,8 +300,7 @@ struct tipc_link *tipc_link_create(struct tipc_node *n_ptr, 


	tipc_node_attach_link(n_ptr, l_ptr);



	k_init_timer(&l_ptr->timer, (Handler)link_timeout,

		     (unsigned long)l_ptr);

	setup_timer(&l_ptr->timer, link_timeout, (unsigned long)l_ptr);



	link_state_event(l_ptr, STARTING_EVT);
@@ -479,7 +479,7 @@ static void link_activate(struct tipc_link *l_ptr) 
static void link_state_event(struct tipc_link *l_ptr, unsigned int event)

{

	struct tipc_link *other;

	u32 cont_intv = l_ptr->continuity_interval;

	unsigned long cont_intv = l_ptr->cont_intv;



	if (l_ptr->flags & LINK_STOPPED)

		return;
@@ -1880,15 +1880,16 @@ void tipc_link_bundle_rcv(struct sk_buff *buf) 
	kfree_skb(buf);

}



static void link_set_supervision_props(struct tipc_link *l_ptr, u32 tolerance)

static void link_set_supervision_props(struct tipc_link *l_ptr, u32 tol)

{

	if ((tolerance < TIPC_MIN_LINK_TOL) || (tolerance > TIPC_MAX_LINK_TOL))

	unsigned long intv = ((tol / 4) > 500) ? 500 : tol / 4;



	if ((tol < TIPC_MIN_LINK_TOL) || (tol > TIPC_MAX_LINK_TOL))

		return;



	l_ptr->tolerance = tolerance;

	l_ptr->continuity_interval =

		((tolerance / 4) > 500) ? 500 : tolerance / 4;

	l_ptr->abort_limit = tolerance / (l_ptr->continuity_interval / 4);

	l_ptr->tolerance = tol;

	l_ptr->cont_intv = msecs_to_jiffies(intv);

	l_ptr->abort_limit = tol / (jiffies_to_msecs(l_ptr->cont_intv) / 4);

}



void tipc_link_set_queue_limits(struct tipc_link *l_ptr, u32 window)

net/tipc/link.h

+2 −2
Original line number Diff line number Diff line
@@ -105,7 +105,7 @@ struct tipc_stats { 
 * @peer_bearer_id: bearer id used by link's peer endpoint

 * @bearer_id: local bearer id used by link

 * @tolerance: minimum link continuity loss needed to reset link [in ms]

 * @continuity_interval: link continuity testing interval [in ms]

 * @cont_intv: link continuity testing interval

 * @abort_limit: # of unacknowledged continuity probes needed to reset link

 * @state: current state of link FSM

 * @fsm_msg_cnt: # of protocol messages link FSM has sent in current state
@@ -146,7 +146,7 @@ struct tipc_link { 
	u32 peer_bearer_id;

	u32 bearer_id;

	u32 tolerance;

	u32 continuity_interval;

	unsigned long cont_intv;

	u32 abort_limit;

	int state;

	u32 fsm_msg_cnt;

net/tipc/msg.h

+0 −2
Original line number Diff line number Diff line
@@ -77,12 +77,10 @@ 


#define TIPC_MEDIA_ADDR_OFFSET	5





struct tipc_msg {

	__be32 hdr[15];

};





static inline u32 msg_word(struct tipc_msg *m, u32 pos)

{

	return ntohl(m->hdr[pos]);

CRA Git | Maintained and supported by SUSTech CRA and CCSE