diff mbox

[PATCHv9,35/35] doc: implementers-guide: adding helper recommendations

Message ID 1463761851-37121-36-git-send-email-christophe.milard@linaro.org
State Accepted
Commit 35a8bdf5b13436094886c85e1abc08845c332502
Headers show

Commit Message

Christophe Milard May 20, 2016, 4:30 p.m. UTC
The implementation guide is updated with recommendations regarding
the usage of helpers and some of its functions.

Signed-off-by: Christophe Milard <christophe.milard@linaro.org>
---
 doc/implementers-guide/implementers-guide.adoc | 32 ++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)
diff mbox

Patch

diff --git a/doc/implementers-guide/implementers-guide.adoc b/doc/implementers-guide/implementers-guide.adoc
index 8863004..abd49c0 100644
--- a/doc/implementers-guide/implementers-guide.adoc
+++ b/doc/implementers-guide/implementers-guide.adoc
@@ -506,5 +506,37 @@  from ODP_TEST_INFO_INACTIVE to ODP_TEST_INFO_CONDITIONAL:
 ------------------
 =================
 
+==== helper usage ====
+
+The tests (both platform agnostic and platform dependent tests) make use of
+a set of functions defined in a helper library. The helper library tries to
+abstract and regroup common actions that applications may perform but
+which are not part of the ODP API (i.e. mostly OS system calls).
+Using these functions is recommended, as running the tests on a different OS
+could (hopefully) be as simple as changing the OS related helper lib.
+
+In the linux helper, two functions are given to create and join ODP threads:
+
+`odph_odpthreads_create()`
+
+`odph_odpthreads_join()`
+
+These two functions abstract what an ODP thread really is and their usage
+is recommended as they would be implemented in other OS`s helper lib.
+
+Five older functions exist to tie and ODP thread to a specific implementation:
+
+`odph_linux_pthread_create()`
+
+`odph_linux_pthread_join()`
+
+`odph_linux_process_fork_n()`
+
+`odph_linux_process_fork()`
+
+`odph_linux_process_wait_n()`
+
+The usage of these functions should not occur within ODP examples nor tests.
+The usage of these functions in other application is not recommended.
 
 include::../glossary.adoc[]