[ovs-dev] [PATCH] ofp-util: Add OFPUTIL_P_OF12 and NXFF_OPENFLOW12

[Ben Pfaff]

On Fri, Jul 20, 2012 at 10:29:25AM +0900, Simon Horman wrote:
> Add OFPUTIL_P_OF12 and NXFF_OPENFLOW12 for Open Flow 1.2
> 
> OFPUTIL_P_OF12_TID and in turn OFPUTIL_P_OF12_ANY is not provided as
> OFPUTIL_P_OF12 supports the use of table ids in modify flow messages.
> 
> Signed-off-by: Simon Horman <horms at verge.net.au>

With this applied:

hda:~# gdb --args ovs-ofctl -F openflow12 add-flow br0 actions=drop
GNU gdb (GDB) 7.2-debian
Copyright (C) 2010 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "i486-linux-gnu".
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>...
Reading symbols from /mnt/_build/utilities/ovs-ofctl...done.
(gdb) r
Starting program: /mnt/_build/utilities/ovs-ofctl -F openflow12 add-flow br0 actions=drop
[Thread debugging using libthread_db enabled]

Program received signal SIGABRT, Aborted.
0xffffe416 in __kernel_vsyscall ()
(gdb) bt
#0  0xffffe416 in __kernel_vsyscall ()
#1  0xb7cef911 in raise (sig=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:64
#2  0xb7cf2d42 in abort () at abort.c:92
#3  0x0807399d in ofputil_encode_flow_mod (fm=0xbffff9e4, 
    protocol=OFPUTIL_P_OF12) at ../lib/ofp-util.c:1847
#4  0x0804dfc7 in ofctl_flow_mod__ (remote=<value optimized out>, 
    fms=<value optimized out>, n_fms=<value optimized out>)
    at ../utilities/ovs-ofctl.c:1044
#5  0x0804e13d in ofctl_flow_mod (argc=<value optimized out>, argv=0xbffffc90, 
    command=0) at ../utilities/ovs-ofctl.c:1069
#6  0x08052afd in run_command (argc=3, argv=0xbffffc90, commands=0x80a5ae0)
    at ../lib/command-line.c:78
#7  0x0804f53a in main (argc=6, argv=0xbffffc84)
    at ../utilities/ovs-ofctl.c:113
(gdb)