I spent some time looking at and playing with patch 4/5 "v2: ofproto: Implement OpenFlow 1.3 meter table." and decided that there was a fair bit that I wanted to tweak. This series represent what I came up with:
IMPORTANT: In this series, I intend to squash all the patches after "Implement OpenFlow 1.3 meter table" into that patch before pushing. Jarno, what do you think? Patches: ofp-actions: Give ovs_instruction funcs names that start with that prefix. ofpact: New function ovs_instruction_type_from_ofpact_type(). ofproto: Implement OpenFlow 1.3 meter table. ofproto: Change 'const uint32_t *meter_id' to just 'uint32_t meter_id'. ofproto: Use another method to find meter_id. ofproto: Allocate meter bands separately from the meters themselves. ofproto: Meter-related comment updates. ofproto: Get rid of 'last_meter'. ofproto: Break handle_meter_mod() into sub-functions. lib/ofp-actions.c | 123 +++++++++--- lib/ofp-actions.h | 17 +-- lib/ofp-parse.c | 2 +- lib/rconn.c | 14 +- ofproto/ofproto-dpif.c | 4 + ofproto/ofproto-provider.h | 57 ++++++- ofproto/ofproto.c | 452 +++++++++++++++++++++++++++++++++++++++++--- ofproto/ofproto.h | 3 + 8 files changed, 590 insertions(+), 82 deletions(-) -- 1.7.2.5 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev