From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <martin@hundeboll.net>
Date: Fri, 24 Aug 2012 19:18:04 +0200
From: Martin =?UTF-8?B?SHVuZGViw7hsbA==?= <martin@hundeboll.net>
Message-ID: <20120824191804.232524cc@hundeboll.net>
In-Reply-To: <1345825951-24041-1-git-send-email-ordex@autistici.org>
References: <1345825951-24041-1-git-send-email-ordex@autistici.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Subject: Re: [B.A.T.M.A.N.] [PATCHv2] batman-adv: print packets re-routing
 on DBG_TT and ratelimit it
Reply-To: The list for a Better Approach To Mobile Ad-hoc Networking
 <b.a.t.m.a.n@lists.open-mesh.org>
List-Id: The list for a Better Approach To Mobile Ad-hoc Networking
 <b.a.t.m.a.n.lists.open-mesh.org>
List-Unsubscribe: <https://lists.open-mesh.org/mm/options/b.a.t.m.a.n>,
 <mailto:b.a.t.m.a.n-request@lists.open-mesh.org?subject=unsubscribe>
List-Archive: <http://lists.open-mesh.org/pipermail/b.a.t.m.a.n/>
List-Post: <mailto:b.a.t.m.a.n@lists.open-mesh.org>
List-Help: <mailto:b.a.t.m.a.n-request@lists.open-mesh.org?subject=help>
List-Subscribe: <https://lists.open-mesh.org/mm/listinfo/b.a.t.m.a.n>,
 <mailto:b.a.t.m.a.n-request@lists.open-mesh.org?subject=subscribe>
To: The list for a Better Approach To Mobile Ad-hoc Networking <b.a.t.m.a.n@lists.open-mesh.org>

Hi Antonio,

On Fri, 24 Aug 2012 18:32:31 +0200
Antonio Quartulli <ordex@autistici.org> wrote:

> diff --git a/routing.c b/routing.c
> index e7a4e25..9b8ff61 100644
> --- a/routing.c
> +++ b/routing.c
> @@ -972,10 +972,10 @@ static int batadv_check_unicast_ttvn(struct
> batadv_priv *bat_priv, batadv_orig_node_free_ref(orig_node);
>  		}
>  
> -		batadv_dbg(BATADV_DBG_ROUTES, bat_priv,
> -			   "TTVN mismatch (old_ttvn %u new_ttvn %u)!
> Rerouting unicast packet (for %pM) to %pM\n",
> -			   unicast_packet->ttvn, curr_ttvn,
> ethhdr->h_dest,
> -			   unicast_packet->dest);
> +		net_ratelimited_function(batadv_dbg, BATADV_DBG_TT,
> bat_priv,

Please add net_ratelimited_function() to compat.h, since it didn't make
its way into mainline before 3.5.

// Martin