Thanks Ben for point that out, I'd very much like to review it.
On Mon, Aug 5, 2013 at 10:37 AM, Ben Pfaff <b...@nicira.com> wrote: > On Sun, Aug 04, 2013 at 08:14:31PM -0700, Alex Wang wrote: > > This commit adds a new commmand "ovs-ofctl parse-ofpraw" to ovs. > > It allows user to parse undecoded raw OpenFlow messages into human > > readable form. It can parse the output log of "ovs-appctl log/controller" > > command. > > > > Signed-off-by: Alex Wang <al...@nicira.com> > > It's not possible to reasonably break binary data into lines with > new-lines, as this patch appears to do, because binary data can itself > contain new-line characters. > > OpenFlow is "self-delimiting" because each message starts with a > header that says how long the message is, so we can simply read > messages one at a time from a file without any special delimiters. I > posted a patch a few weeks ago that adds a command to do this: > http://patchwork.openvswitch.org/patch/723/ > Will you review it, please? > > Thanks, > > Ben. >
_______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
