Acked-by: Christian Lindig <christian.lin...@citrix.com> I am providing feedback on https://github.com/edwintorok/xen/pull/1.
In general: this is a large patch and therefore difficult to review for correctness. However: * It comes with a lot of testing and was fuzz-tested * It improves building OCaml xenstore * It improves functional and performance problems I welcome this work and think it ends a period of stagnation of this code base. This applies not just to this patch but the other patches sent out by Edwin in the context of this. ________________________________________ From: Edwin Török <edvin.to...@citrix.com> Sent: 15 January 2021 22:28 To: xen-devel@lists.xenproject.org Cc: Edwin Torok; Christian Lindig; David Scott; Ian Jackson; Wei Liu Subject: [PATCH v4 0/4] tools/ocaml/xenstored: optimizations Various speed optimizations that have already been posted, but committing them was delayed to avoid conflicts with XSAs. The XSAs are out, so these are ready to go now. The switch to Maps may expose bugs in certain xenstored clients, which previously relied on iteration order of the DIRECTORY response. In our testing we found one such case, which turned out to be a bug in a testsuite (it always dropped the 1st xenstore key). For convenience here is a tree with all patch series applied: https://github.com/edwintorok/xen/pull/1 Edwin Török (4): tools/ocaml/xenstored: replace hand rolled GC with weak GC references tools/ocaml/xenstored: backport find_opt/update from 4.06 tools/ocaml/xenstored: use more efficient node trees tools/ocaml/xenstored: use more efficient tries tools/ocaml/xenstored/connection.ml | 3 -- tools/ocaml/xenstored/connections.ml | 2 +- tools/ocaml/xenstored/history.ml | 14 ------ tools/ocaml/xenstored/stdext.ml | 19 ++++++++ tools/ocaml/xenstored/store.ml | 51 +++++++++----------- tools/ocaml/xenstored/symbol.ml | 70 +++++++--------------------- tools/ocaml/xenstored/symbol.mli | 22 +++------ tools/ocaml/xenstored/trie.ml | 61 +++++++++++------------- tools/ocaml/xenstored/trie.mli | 26 +++++------ tools/ocaml/xenstored/xenstored.ml | 16 +------ 10 files changed, 109 insertions(+), 175 deletions(-) -- 2.29.2
