On Wed, Feb 24, 2016 at 2:44 PM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Feb 22, 2016 at 04:28:14PM -0800, Andy Zhou wrote: > > Avoid calling ovsdb_monitor_get_update() monitor has been flushed > > already. > > > > Reported-by: Liran Schour <lir...@il.ibm.com> > > Signed-off-by: Andy Zhou <az...@ovn.org> > > > > ---- > > v1 -> v2: no change. > > What are the visible consequences of this change? That is, does it > suppress sending something to clients, or does it just suppress work in > the monitor, or...? This information is probably suitable to include in > the commit message. >
Make sense, how about the following? Optimizing ovsdb_jsonrpc_mintor_flush_all() by avoiding calling ovsdb_monitor_get_update() on monitors that do not have any unflushed updates. This change saves CPU cycles on ovsdb-server's main loop, but should not introduce any client visible changes. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev