On Wed, 11 May 2016 12:28:14 +0900, Simon Horman wrote: > I think that at this stage I would prefer to prohibit push_eth() acting > on a packet which already has an ethernet header. Indeed that is what > my patch-set already does in its modifications of __ovs_nla_copy_actions(). > > The reason that I lean towards prohibiting this is that I do not > have an easy way to exercise this case within the current patch-set. > And thus this extra complexity seems well suited to being handled handled > incrementally as further work.
Works for me. I don't see any real usage for multiple Ethernet headers. Thanks! Jiri
