On Tue, Nov 05, 2019 at 04:27:51PM +0200, Lorenzo Bianconi wrote: > Allow jsonrpc clients (e.g. ovn-controller) to specify jsonrpc input > buffer size in order to reduce overhead when downloading huge db size > since current value is 512B. The user can specify rpc buffer size using > ovsdb_idl_set_remote routine passing requested value > > Signed-off-by: Lorenzo Bianconi <[email protected]>
Hmm, I thought that we had decided to just try a 4096-byte buffer to start. That would be a much smaller patch. The marginal benefits of buffers larger than that probably decline a lot since the processing cost of 4096 bytes of JSON is probably a lot more than the overhead of a system call. I imagine that any further benefit is probably from being able to process more JSON per trip through the main loop. Those same benefits could also be obtained by increasing the 'for' loop limit in jsonrpc_recv() from 50 to some larger number. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
