On Mon, Sep 12, 2016 at 01:52:42PM -0700, Jarno Rajahalme wrote:
> Refactor handle_packet_out() to prepare for bundle support for packet
> outs in a later patch.
> 
> Two new callbacks are introduced in ofproto-provider class:
> ->packet_xlate() and ->packet_execute().  ->packet_xlate() translates
> the packet using the flow and actions provided by the caller, but
> defers all OpenFlow-visible side-effects (stats, learn actions, actual
> packet output, etc.) to be explicitly executed with the
> ->packet_execute() call.
> 
> Adds a new ofproto_rule_reduce_timeouts__() that must be called with
> 'ofproto_mutex' held.  This is used in the next patch.
> 
> Signed-off-by: Jarno Rajahalme <ja...@ovn.org>
> ---
> v3: Removed layer violations by making ofproto-dpif initialize and
>     manage its private 'aux' member in struct ofproto_packet_out.

It appears that an xlate_cache always allocates at least 512 bytes of
space.  (It's not a change but I hadn't noticed before.)  Is that right?
It seems like a lot, since I guess that the ordinary occupancy is much
smaller than that.

Acked-by: Ben Pfaff <b...@ovn.org>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to