diff options
author | Ben Pfaff <blp@ovn.org> | 2016-01-18 16:00:34 -0800 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2016-01-20 09:48:59 -0800 |
commit | 5876b4dbf6887a9c6c419c130a8ef9996ac52012 (patch) | |
tree | eebf6ea6998889d5ae23d855021fbdb395e14268 /OPENFLOW-1.1+.md | |
parent | 8fd0bb602e49bff8b5b6db594083572bb5604025 (diff) | |
download | openvswitch-5876b4dbf6887a9c6c419c130a8ef9996ac52012.tar.gz |
ofp-util: Fix OF1.4+ version of ofputil_decode_set_async_config().
The OF1.0 through OF1.3 "set async config" set the whole configuration,
OF1.4+ only update parts of it piecemeal, but the decoding function always
set the whole configuration. This commit fixes the problem by changing the
interface to require the caller to provide an initial state. (It would
be possible to simply make it mutate existing state in-place, but that
interface seems a little more error-prone.)
Found by inspection.
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Jarno Rajahalme <jarno@ovn.org>
Diffstat (limited to 'OPENFLOW-1.1+.md')
-rw-r--r-- | OPENFLOW-1.1+.md | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/OPENFLOW-1.1+.md b/OPENFLOW-1.1+.md index 537f660b9..62ebddcd6 100644 --- a/OPENFLOW-1.1+.md +++ b/OPENFLOW-1.1+.md @@ -192,10 +192,7 @@ OpenFlow 1.4 features are listed in the previous section. * More extensible wire protocol Many on-wire structures got TLVs. - Already implemented: port desc properties, port mod properties, - port stats properties, table mod properties, - queue stats, unified property errors, queue desc. - Remaining required: set-async + All required features are now supported. Remaining optional: table desc, table-status [EXT-262] [required for OF1.4+] |