On Fri, Jul 24, 2015 at 07:52:32PM +0000, Jarno Rajahalme wrote: > Could add to the commit message: > > Keep actset_output as OFPP_UNSET if there is a group action. > > Acked-by: Jarno Rajahalme <[email protected]>
You're right, the commit message is under-descriptive. I changed it to: commit 9055ca9a21f6667d33ea0cd20961a8e2cace059e Author: Ben Pfaff <[email protected]> Date: Thu Jul 23 15:45:05 2015 -0700 ofproto-dpif-xlate: Fix handling of write_actions(group(5),output(10)). This keeps actset_output as OFPP_UNSET in the special case where a single write_actions() contains a group action followed by an output action. The presumably more common case where instructions in separate action lists contained output and group actions was already properly handled. Without the fix added by this commit, the test case as updated by this commit fails. Found by inspection. Signed-off-by: Ben Pfaff <[email protected]> Acked-by: Jarno Rajahalme <[email protected]> and applied this to master and branch-2.4. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
