On 4/8/25 9:57 AM, Martin Morgenstern via dev wrote: > Hi, > > this is v2 of the patch set originally proposed and discussed at [1], > incorporating the comments and suggestions by Ilya. > > v1 --> v2: > * Dropped patch 1 because gratuitous replies can mask fencing issues. > * Dropped patches 4, 5 and 6: they become a separate patch set and > will be generalized to a proper timing API. > * Renamed the coverage counter to "jsonrpc_recv_incomplete". > * Renamed byteq_optimize() to byteq_fast_forward() and added tests. > * Fixed typos, improved comments and wording. > > [1] https://mail.openvswitch.org/pipermail/ovs-dev/2025-March/421708.html > > Martin Morgenstern (2): > jsonrpc: Add coverage for incomplete receives. > jsonrpc: Maximize receive buffer size. > > lib/byteq.c | 17 +++++++++++++++++ > lib/byteq.h | 1 + > lib/jsonrpc.c | 8 ++++++++ > tests/library.at | 2 +- > tests/test-byteq.c | 32 ++++++++++++++++++++++++++++++++ > 5 files changed, 59 insertions(+), 1 deletion(-) >
Thanks, Martin and Felix! I moved the 'q->tail = q->head;' in the second patch under the 'if' statement to avoid extra assignment when the value didn't change. With that, applied to main. Looking forward for the second set with a proper timing API. Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
