Re: [ovs-dev] [PATCH] ofp-util: Fix length field of bundle messages

2015-04-08 Thread YAMAMOTO Takashi
hi, thank you for review. you are right. i didn't notice the code in ofconn_send. thank you. i tend to think it's nicer to have ofputil_encode_xxx generate the complete packet, though. background: my code [1] uses libofproto to generate on-wire packet and write it out to a file. thus ofconn_s

Re: [ovs-dev] [PATCH] ofp-util: Fix length field of bundle messages

2015-04-07 Thread Jarno Rajahalme
Yamamoto, ofconn_send(), which will eventually be called for these, already does ofpmsg_update_length() before passing the message to rconn_send(). Given this, it seems unnecessary to make this change. Jarno > On Apr 6, 2015, at 11:43 PM, YAMAMOTO Takashi wrote: > > Signed-off-by: YAMAMOTO

[ovs-dev] [PATCH] ofp-util: Fix length field of bundle messages

2015-04-06 Thread YAMAMOTO Takashi
Signed-off-by: YAMAMOTO Takashi --- lib/ofp-util.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/ofp-util.c b/lib/ofp-util.c index a0da289..7035c8c 100644 --- a/lib/ofp-util.c +++ b/lib/ofp-util.c @@ -8760,6 +8760,7 @@ ofputil_encode_bundle_ctrl_reply(const struct ofp_header *oh,