From: Taehee Yoo <ap420073@gmail.com>
Date: Wed, 6 May 2020 15:47:45 +0000 (+0000)
Subject: hsr: remove WARN_ONCE() in hsr_fill_frame_info()
X-Git-Tag: Ubuntu-5.10.0-12.13~2734^2~257^2~1
X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=f96e87178bb819a2ee25623a6935fa850c2defdd;p=mirror_ubuntu-hirsute-kernel.git

hsr: remove WARN_ONCE() in hsr_fill_frame_info()

When VLAN frame is being sent, hsr calls WARN_ONCE() because hsr doesn't
support VLAN. But using WARN_ONCE() is overdoing.
Using netdev_warn_once() is enough.

Signed-off-by: Taehee Yoo <ap420073@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
---

diff --git a/net/hsr/hsr_forward.c b/net/hsr/hsr_forward.c
index ddd9605bad04..ed13760463de 100644
--- a/net/hsr/hsr_forward.c
+++ b/net/hsr/hsr_forward.c
@@ -321,7 +321,7 @@ static int hsr_fill_frame_info(struct hsr_frame_info *frame,
 	if (ethhdr->h_proto == htons(ETH_P_8021Q)) {
 		frame->is_vlan = true;
 		/* FIXME: */
-		WARN_ONCE(1, "HSR: VLAN not yet supported");
+		netdev_warn_once(skb->dev, "VLAN not yet supported");
 	}
 	if (ethhdr->h_proto == htons(ETH_P_PRP) ||
 	    ethhdr->h_proto == htons(ETH_P_HSR)) {