On Sat, Jan 07, 2012 at 11:11:46AM -0800, Ethan Jackson wrote: > In future patches, directly using a void * pointer in the > pktbuf_save() definition will simplify the code. > > Signed-off-by: Ethan Jackson <et...@nicira.com>
In pktbuf_save(), you can use ofpbuf_clone_data_with_headroom() instead of open-coding it. I think there's an existing incorrect statement in the comment on pktbuf_save(): 'in_port' should be an OpenFlow port number these days, I think. Otherwise it looks good. Thank you. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev