Looks fine.
As an idea for later, you could actually move all of these definitions that are now in lacp.h into lacp.c, if you did two more things: made lacp_process_pdu() take the whole frame and parse it itself, and made the lacp_run() callback take a const void * and a length instead of a structure. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
