netdev - Re: [PATCH net] net: sched: reduce amount of log messages in act

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20200402.180417.804204103829966415.davem@davemloft.net>
Date:   Thu, 02 Apr 2020 18:04:17 -0700 (PDT)
From:   David Miller <davem@...emloft.net>
To:     marcelo.leitner@...il.com
Cc:     netdev@...r.kernel.org, jhs@...atatu.com, xiyou.wangcong@...il.com,
        jiri@...nulli.us
Subject: Re: [PATCH net] net: sched: reduce amount of log messages in
 act_mirred

From: Marcelo Ricardo Leitner <marcelo.leitner@...il.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.