From: Han Zhou <[email protected]> In the else condition, it is already ensured that index >= idl->min_index. So the MAX() is confusing and misleading here.
Signed-off-by: Han Zhou <[email protected]> --- lib/ovsdb-idl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c index 29cb80e..5ae86f7 100644 --- a/lib/ovsdb-idl.c +++ b/lib/ovsdb-idl.c @@ -1888,7 +1888,7 @@ ovsdb_idl_check_server_db(struct ovsdb_idl *idl) VLOG_WARN("%s: clustered database server has stale data; " "trying another server", server_name); } else { - idl->min_index = MAX(idl->min_index, index); + idl->min_index = index; ok = true; } } else { -- 2.1.0 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
