[ovs-dev] [PATCH 2/2] feature: Add ovs-ofctl parse-ofpraw command.

Alex Wang alexw at nicira.com
Mon Aug 5 17:42:49 UTC 2013


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 <blp at 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 <alexw at 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.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20130805/47b32217/attachment-0003.html>


More information about the dev mailing list