On Tue, May 31, 2016 at 01:52:55PM -0700, Ben Pfaff wrote: > On Wed, May 04, 2016 at 04:34:23PM +0900, Simon Horman wrote: > > From: Lorand Jakab <[email protected]> > > > > This is needed for sending a packet back to the datapath after a miss > > upcall was processed. The presence of a layer 2 packet is signaled by > > adding OVS_KEY_ATTR_ETHERNET to the packet metadata sent with the > > ovs_packet netlink message. Layer 3 packets need to be accompanied by > > OVS_KEY_ATTR_PACKET_ETHERTYPE to indicate network protocol. > > > > Signed-off-by: Lorand Jakab <[email protected]> > > Signed-off-by: Simon Horman <[email protected]> > > I am not sure it makes sense to do a masked set of a packet Ethertype > (in odp_execute_masked_set_action()).
Thanks. I'm not sure that we need to allow set of packet Ethernet at all. It does seem that its not necessary to do something useful with l3 tunnels at this time. So I've removed it for now. > Acked-by: Ben Pfaff <[email protected]> Thanks! _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
