.@hpe.com
TÂ +506 2560-4430 x 4011
Hewlett-Packard Company
Paseo de las Flores
Heredia
Costa Rica
Please print thoughtfully
-Original Message-----
From: Lutz, Arnoldo
Sent: Monday, June 13, 2016 10:07 AM
To: 'dev@openvswitch.org'
Subject: Subject: [ovsdb-idl 1/1] Fix issues dete
Hi.
We found some issues affecting Partial Map Update feature included in master
branch.
This patch fixes a memory leak due to lack of freeing datum allocated in the
process of
requesting a change to a map.
It also fix an error produced when NDEBUG flag is not set that causes an
assertion when
Added delta calculations for PMUs
Added additional checks to make sure that only one update operation is
performed on each key for each transaction. If several update operations are
issued by the client, these are resolved and only one is applied on commit.
Delayed operation validity checks until
Modified ovsdb_idl_row structure to include information on partial map updates,
which will be used to create mutate operations on map columns when the
transaction is committed.
Also filled out generic (skeleton) functions to populate partial map update
information on the row when a map is partial
Several memory management errors were fixed. Also, some wrong uses of pointers
that produced segmentation faults were fixed.
Refactored code into separate files
Code for Partial Map Updates and Partial Map Update Lists was extracted into
separate files, and the inner contents of structs pmu and p
Code inserted that autogenerates corresponding map functions to set and
delete elements in map columns.
Inserts description to the functions that are autogenerated.
Changes on autogenerated functions
Signed-off-by: arnoldo.lutz.guev...@hpe.com
---
ovsdb/ovsdb-idlc.in | 69 +++
---
tests/test-ovsdb.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/tests/test-ovsdb.c b/tests/test-ovsdb.c
index 0ad03bd..8c7aedd 100644
--- a/tests/test-ovsdb.c
+++ b/tests/test-ovsdb.c
@@ -197,7 +197,11 @@ usage(void)
"connect to SERVER and dump the c
Add file PARTIAL-UPDATE-MAP-COLUMN.md with a description on the way to
use this feature.
Modifies Makefile.am to include documentation in distribution list
Signed-off-by: arnoldo.lutz.guev...@hpe.com
---
Makefile.am | 1 +
PARTIAL-UPDATE-MAP-COLUMN.md | 64 +
In the current implementation, every time an element of either a map or
set column has to be modified, the entire content of the column is
sent to the server to be updated. This is not a major problem if the
information contained in the column for the corresponding row is small,
but there are cases
This commit adds some definitions and skeleton functions to start working.
Ovsdb tests run and pass ok.
Signed-off-by: arnoldo.lutz.guev...@hpe.com
---
2 0 lib/ovsdb-idl-provider.h
31 3 lib/ovsdb-idl.c
7 0 lib/ovsdb-idl.h
lib/ovsdb-idl-provider.h | 2 ++
lib/o
10 matches
Mail list logo