On Fri, 7 Oct 2016 12:59:08 -0700, Pravin Shelar wrote: > On Fri, Oct 7, 2016 at 9:07 AM, Jiri Benc <jb...@redhat.com> wrote: > > Always keep the first vlan tag "accelerated", i.e. in skb->vlan_tci. > > > > Unfortunately, with all the changes since v2, this patchset no longer has > > the nice deletions > insertions diffstat. I still think it's worth it, as it > > makes things more consistent overall. > > > After looking at the changes, I am not sure about the value. These > patches are making code bit complicated by processing vlan header > twice rather than once in current code.
Yes, this is a trade-off. A bit more complexity on packet ingress, less complexity on packet processing. My main motivation was L3 packets where the code in packet_length became more complicated than I'd like to to cover all possible cases. Normalizing the vlan tags looked as a pretty obvious improvement. But I'm not that thrilled with what it evolved into. I think it's slightly better than what we have now but I can understand how you may think opposite. I'll rip the fixes (patches 3 and 6) off this patchset and send them separately. > As far as patch 6 is concerned I think we could do MTU checks similar > to the rest of networking stack (for example is_skb_forwardable()). > That would simplify things here. Fixing the current code is not that hard. The real problem is the added complexity with L3 packets. I'll look more into the possible solutions for the L3 patchset. Jiri
