Skip to content

Commit 5a3a896

Browse files
keesdavem330
authored andcommitted
net: vxge: Convert timers to use timer_setup()
In preparation for unconditionally passing the struct timer_list pointer to all timer callbacks, switch to using the new timer_setup() and from_timer() to pass the timer pointer explicitly. Cc: Jon Mason <[email protected]> Cc: "David S. Miller" <[email protected]> Cc: Miroslav Lichvar <[email protected]> Cc: Jarod Wilson <[email protected]> Cc: Eric Dumazet <[email protected]> Cc: stephen hemminger <[email protected]> Cc: [email protected] Signed-off-by: Kees Cook <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent d3e99b2 commit 5a3a896

File tree

2 files changed

+9
-11
lines changed

2 files changed

+9
-11
lines changed

drivers/net/ethernet/neterion/vxge/vxge-main.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2597,9 +2597,9 @@ static int vxge_add_isr(struct vxgedev *vdev)
25972597
return VXGE_HW_OK;
25982598
}
25992599

2600-
static void vxge_poll_vp_reset(unsigned long data)
2600+
static void vxge_poll_vp_reset(struct timer_list *t)
26012601
{
2602-
struct vxgedev *vdev = (struct vxgedev *)data;
2602+
struct vxgedev *vdev = from_timer(vdev, t, vp_reset_timer);
26032603
int i, j = 0;
26042604

26052605
for (i = 0; i < vdev->no_of_vpath; i++) {
@@ -2616,9 +2616,9 @@ static void vxge_poll_vp_reset(unsigned long data)
26162616
mod_timer(&vdev->vp_reset_timer, jiffies + HZ / 2);
26172617
}
26182618

2619-
static void vxge_poll_vp_lockup(unsigned long data)
2619+
static void vxge_poll_vp_lockup(struct timer_list *t)
26202620
{
2621-
struct vxgedev *vdev = (struct vxgedev *)data;
2621+
struct vxgedev *vdev = from_timer(vdev, t, vp_lockup_timer);
26222622
enum vxge_hw_status status = VXGE_HW_OK;
26232623
struct vxge_vpath *vpath;
26242624
struct vxge_ring *ring;
@@ -2858,12 +2858,12 @@ static int vxge_open(struct net_device *dev)
28582858
vdev->config.rx_pause_enable);
28592859

28602860
if (vdev->vp_reset_timer.function == NULL)
2861-
vxge_os_timer(&vdev->vp_reset_timer, vxge_poll_vp_reset, vdev,
2861+
vxge_os_timer(&vdev->vp_reset_timer, vxge_poll_vp_reset,
28622862
HZ / 2);
28632863

28642864
/* There is no need to check for RxD leak and RxD lookup on Titan1A */
28652865
if (vdev->titan1 && vdev->vp_lockup_timer.function == NULL)
2866-
vxge_os_timer(&vdev->vp_lockup_timer, vxge_poll_vp_lockup, vdev,
2866+
vxge_os_timer(&vdev->vp_lockup_timer, vxge_poll_vp_lockup,
28672867
HZ / 2);
28682868

28692869
set_bit(__VXGE_STATE_CARD_UP, &vdev->state);

drivers/net/ethernet/neterion/vxge/vxge-main.h

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -417,12 +417,10 @@ struct vxge_tx_priv {
417417
module_param(p, int, 0)
418418

419419
static inline
420-
void vxge_os_timer(struct timer_list *timer, void (*func)(unsigned long data),
421-
struct vxgedev *vdev, unsigned long timeout)
420+
void vxge_os_timer(struct timer_list *timer, void (*func)(struct timer_list *),
421+
unsigned long timeout)
422422
{
423-
init_timer(timer);
424-
timer->function = func;
425-
timer->data = (unsigned long)vdev;
423+
timer_setup(timer, func, 0);
426424
mod_timer(timer, jiffies + timeout);
427425
}
428426

0 commit comments

Comments
 (0)