[ovs-discuss] Adding a new OpenFlow action to ovs, ctx->xin->packet is null

[Ben Pfaff]

On Sat, May 10, 2014 at 05:57:41PM +0200, Christian Stigen Larsen wrote:
> Ben Pfaff <blp at nicira.com>:
> >> Is do_xlate_actions ("translate actions"?) the wrong place for this kind of
> >> stuff?  How can I get a pointer to the raw packet data?
> > 
> > Usually, ofproto-dpif-xlate should not actually implement the OpenFlow
> > actions.  Rather, it should output a set of datapath (kernel) actions
> > that implement the OpenFlow actions.  Otherwise, every packet in the
> > flow has to go from the kernel to userspace, which is slow.
> 
> Yes, I figured this was not the place.  So should I add a new action to
> ovs_action_attr instead, then add it as a datapath action during
> translation?

Yes, that's the way to do it.