This purpose of this patchset is to provide initial multipart request
support and provide a basis for discussion of how such support should be
implemented.
There are 4 patches in this series:
1. ofproto: Further factor out handling of port requests
A clean up for ofproto port request handling code.
I would like this considered as an enhancement as is.
It is also a dependency of subsequent changes.
2. ofp-print: Merge ofp_print_stats_{reply,request}()
This facilitates printing of the flags or multipart requests.
I would also like this considered as an enhancement as is.
It is also a dependency of subsequent changes.
3. connmgr: buffer multipart requests
This is the body of multipart request support.
I would like this considered as an RFC as I am sure
there are other ways of implementing this feature.
If the approach I have taken is broadly acceptable
I believe it should be enhanced to somehow limit the number
of requests that may be buffered. I simple hard limit seems
like a good place to start to me.
4. This adds support for multipart port stats and port desc stats requests
The primary purpose of this patch is to exercise the previous patches
an bring things together in a useful though limited form.
I have not extended it to other messages as first I would like
to reach some consensus on the approach to be taken when
implementing support for multipart requests. In particular patches
3 and 4.
Simon Horman (4):
ofproto: Further factor out handling of port requests
ofp-print: Merge ofp_print_stats_{reply,request}()
connmgr: buffer multipart requests
ofproto: Support multipart port stats and port desc stats requests
OPENFLOW-1.1+ | 6 +-
lib/ofp-print.c | 55 +++++++---------
ofproto/connmgr.c | 162 ++++++++++++++++++++++++++++++++++++++++++++++++
ofproto/ofproto.c | 85 +++++++++++++++++--------
tests/ofproto-macros.at | 1 +
tests/ofproto.at | 109 ++++++++++++++++++++++++++++++++
6 files changed, 357 insertions(+), 61 deletions(-)
--
1.8.5.2
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
