[ovs-discuss] Set-field instruction

[Ben Pfaff]

On Fri, Aug 30, 2013 at 10:22:16AM +0100, John Hurley wrote:
> I have been playing about with the latest OVS and writing an
> ofproto-provider module. However, when dealing with some OF 1.3 actions I
> am finding issues. The conversion of the 'ofpacts' struct back to openflow
> is crashing when 1.2+ actions like 'set-field' are used. The approach works
> fine for 1.1 actions.
> 
> I am using the ofpacts_put_openflow11_instructions() function to do this.
> 
> Digging into the ovs code a bit I see that the set-field action is handled
> when packaging the actions in ofpact_from_openflow11() but is not extracted
> again in the above (actually causes ovs to seg fault in
> ofpact_to_openflow11() with no break after matching type OFPACT_REG_LOAD).
> 
> Is this action simply not supported at the current time (should we expect
> an ofpacts_put_openflow13_instructions() in the future) or am I missing
> something that allows such actions to be handled in OVS as it currently
> stands?

This sounds like a bug, but I don't see an obvious problem in the
code.  Can you explain how to trigger the issue or where there's a
problem in the code?

Thanks,

Ben.