Re: [B.A.T.M.A.N.] [PATCH] batman-adv: a delayed_work has to be initialised once

[Marek Lindner <lindner_marek@yahoo.de>]

On Saturday, December 15, 2012 21:10:56 Antonio Quartulli wrote:
> -	batadv_start_purge_timer(bat_priv);
> +	INIT_DELAYED_WORK(&bat_priv->orig_work, batadv_purge_orig);
> +	queue_delayed_work(batadv_event_workqueue,
> +			   &bat_priv->orig_work, msecs_to_jiffies(1000));
> +
>  	return 0;
> 
>  err:
> @@ -396,7 +392,8 @@ static void batadv_purge_orig(struct work_struct *work)
>  	delayed_work = container_of(work, struct delayed_work, work);
>  	bat_priv = container_of(delayed_work, struct batadv_priv, orig_work);
>  	_batadv_purge_orig(bat_priv);
> -	batadv_start_purge_timer(bat_priv);
> +	queue_delayed_work(batadv_event_workqueue,
> +			   &bat_priv->orig_work, msecs_to_jiffies(1000));
>  }

If the same numeric value is used multiple times we should have a define.


> @@ -2136,7 +2129,9 @@ int batadv_tt_init(struct batadv_priv *bat_priv)
>  	if (ret < 0)
>  		return ret;
> 
> -	batadv_tt_start_timer(bat_priv);
> +	INIT_DELAYED_WORK(&bat_priv->tt.work, batadv_tt_purge);
> +	queue_delayed_work(batadv_event_workqueue, &bat_priv->tt.work,
> +			   msecs_to_jiffies(5000));
> 
>  	return 1;
>  }
> @@ -2286,7 +2281,8 @@ static void batadv_tt_purge(struct work_struct *work)
>  	batadv_tt_req_purge(bat_priv);
>  	batadv_tt_roam_purge(bat_priv);
> 
> -	batadv_tt_start_timer(bat_priv);
> +	queue_delayed_work(batadv_event_workqueue, &bat_priv->tt.work,
> +			   msecs_to_jiffies(5000));
>  }

Same here.

Cheers,
Marek