From: RYAN D. MOATS <rmo...@us.ibm.com>

Currently changes are added to the front of the track list, so
they are looped through in LIFO order. Incremental processing
is more efficient with a FIFO presentation, so add changes to the
back of the track list.
---
 lib/ovsdb-idl.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c
index 4cb1c81..c973d37 100644
--- a/lib/ovsdb-idl.c
+++ b/lib/ovsdb-idl.c
@@ -1351,8 +1351,8 @@ ovsdb_idl_row_change__(struct ovsdb_idl_row *row, const 
struct json *row_json,
                         = row->table->idl->change_seqno + 1;
                     if (table->modes[column_idx] & OVSDB_IDL_TRACK) {
                         if (list_is_empty(&row->track_node)) {
-                            list_push_front(&row->table->track_list,
-                                            &row->track_node);
+                            list_push_back(&row->table->track_list,
+                                           &row->track_node);
                         }
                         if (!row->updated) {
                             row->updated = bitmap_allocate(class->n_columns);
@@ -1572,7 +1572,7 @@ ovsdb_idl_row_destroy(struct ovsdb_idl_row *row)
                 = row->table->idl->change_seqno + 1;
         }
         if (list_is_empty(&row->track_node)) {
-            list_push_front(&row->table->track_list, &row->track_node);
+            list_push_back(&row->table->track_list, &row->track_node);
         }
     }
 }
-- 
1.7.1

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to