> On Nov 30, 2016, at 5:51 AM, Jiri Benc <jb...@redhat.com> wrote: > > On Tue, 29 Nov 2016 15:30:52 -0800, Jarno Rajahalme wrote: >> @@ -504,11 +485,20 @@ void ovs_vport_send(struct vport *vport, struct >> sk_buff *skb, u8 mac_proto) >> goto drop; >> } >> >> - if (unlikely(packet_length(skb, vport->dev) > mtu && >> - !skb_is_gso(skb))) { >> - net_warn_ratelimited("%s: dropped over-mtu packet: %d > %d\n", >> - vport->dev->name, >> - packet_length(skb, vport->dev), mtu); >> + if (unlikely(!is_skb_forwardable(vport->dev, skb))) { > > How does this work when the vlan tag is accelerated? Then we can be > over MTU, yet the check will pass. >
I’ll check how other call sites use this. Jarno > Jiri