Re: [ovs-dev] [PATCH 25/41] ofp-util: Fix OF1.4+ version of ofputil_decode_set_async_config().

2016-01-19 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Jan 18, 2016, at 11:27 PM, Ben Pfaff wrote: > > 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

[ovs-dev] [PATCH 25/41] ofp-util: Fix OF1.4+ version of ofputil_decode_set_async_config().

2016-01-18 Thread Ben Pfaff
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