[ovs-dev] [cfm 3/6] packets: Generalize compose_lacp_packet()

[Ben Pfaff]

On Tue, Mar 22, 2011 at 06:32:55PM -0700, Ethan Jackson wrote:
> Creates a new function compose_packet() which replaces
> compose_lacp_packet(). The CFM code uses this function as well.

It seems reasonable to have a function like this.  I'm not as happy
about compose_packet() doing a memory copy.  Not that it would
normally matter, but it seems unnecessary.  Would it be reasonable for
it to just take a size and return a pointer to the start of that
space?  The function would look a lot like make_openflow() but it
would insert an Ethernet header instead of an OpenFlow header.