diff options
| author | Taehee Yoo <ap420073@gmail.com> | 2020-05-06 15:47:45 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-05-07 17:40:02 -0700 |
| commit | f96e87178bb819a2ee25623a6935fa850c2defdd (patch) | |
| tree | a7ab0189e1005d020f90577ba77b1b079ef10c10 | |
| parent | 7596ac9d19a9df25707ecaac0675881f62dd8c18 (diff) | |
| download | linux-f96e87178bb819a2ee25623a6935fa850c2defdd.tar.gz linux-f96e87178bb819a2ee25623a6935fa850c2defdd.tar.bz2 linux-f96e87178bb819a2ee25623a6935fa850c2defdd.zip | |
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>
| -rw-r--r-- | net/hsr/hsr_forward.c | 2 |
1 files changed, 1 insertions, 1 deletions
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)) { |