file->last_compact is initialized from time_msec() so the difference has
to be computed relative to that clock, not against wall clock time.

This only affected the log message, not the decision about when to
compact.

Signed-off-by: Ben Pfaff <b...@ovn.org>
Reported-by: Daniel Alvarez Sanchez <dalva...@redhat.com>
Reported-at: 
https://mail.openvswitch.org/pipermail/ovs-discuss/2018-March/046316.html
Signed-off-by: Ben Pfaff <b...@ovn.org>
---
 ovsdb/file.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ovsdb/file.c b/ovsdb/file.c
index 90c2b9d20a9a..4b7ad52ab52a 100644
--- a/ovsdb/file.c
+++ b/ovsdb/file.c
@@ -655,7 +655,7 @@ ovsdb_file_compact(struct ovsdb_file *file)
 
     comment = xasprintf("compacting database online "
                         "(%.3f seconds old, %u transactions)",
-                        (time_wall_msec() - file->last_compact) / 1000.0,
+                        (time_msec() - file->last_compact) / 1000.0,
                         file->n_transactions);
     VLOG_INFO("%s: %s", file->file_name, comment);
 
-- 
2.16.1

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to