On Thu, May 09, 2013 at 09:22:07AM -0700, Ben Pfaff wrote: > On Thu, May 02, 2013 at 06:06:46PM +0900, Simon Horman wrote: > > Update handling of the following actions to use the dl_type set by MPLS > > push and pop actions if it differs from the original dl_type. This is > > consistent with the existing checking of load actions and allows > > their existing checks to enforce dl_type pre-requisites correctly. > > > > output_reg > > bundle > > reg_move > > stack_push > > stack_pop > > learn > > multipath > > > > In order to avoid the verbosity of updating the flow for each applicable > > action the update is treated as a common case and performed at the top of > > ofpact_check__(). This was suggested by Jesse Gross. > > > > Cc: Jesse Gross <je...@nicira.com> > > Signed-off-by: Simon Horman <ho...@verge.net.au> > > Jesse, it looks like you suggested this, do you want to review it? > > I think that there may be a related issue though: it looks to me like > a pop_mpls action has no effect on dl_type if there is no MPLS header, > but I think that the logic for checking actions assumes that pop_mpls > always updates dl_type.
Thanks, I'll look into that. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
