Re: [PATCH net] net: sched: reduce amount of log messages in act_mirred

[Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>]

On Thu, Apr 02, 2020 at 06:04:17PM -0700, David Miller wrote:
> From: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
> Date: Thu,  2 Apr 2020 19:26:12 -0300
> 
> > @@ -245,8 +245,8 @@ static int tcf_mirred_act(struct sk_buff *skb, const struct tc_action *a,
> >  	}
> >  
> >  	if (unlikely(!(dev->flags & IFF_UP))) {
> > -		net_notice_ratelimited("tc mirred to Houston: device %s is down\n",
> > -				       dev->name);
> > +		pr_notice_once("tc mirred: device %s is down\n",
> > +			       dev->name);
> 
> This reduction is too extreme.
> 
> If someone causes this problem, reconfigures everything thinking that the
> problem will be fixed, they won't see this message the second time and
> mistakenly think it's working.

Fair point. Then what about removing it entirely? printk's are not the
best way to debug packet drops anyway and the action already registers
the drops in its stats.

Or perhaps a marker in the message, stating that it is logged only
once per boot. I'm leaning to the one above, to just remove it.