diff options
| author | Dean Michael Berris <dberris@google.com> | 2016-11-16 02:18:23 +0000 |
|---|---|---|
| committer | Dean Michael Berris <dberris@google.com> | 2016-11-16 02:18:23 +0000 |
| commit | 30d8d0833f6c76322e726c09f60c8110bb816e92 (patch) | |
| tree | c35737370c8355b5c95f0cb095fedd35aa4701cf /docs/XRay.rst | |
| parent | 8934fe31f3cbfa1543b98dc101005a2142bb3fe0 (diff) | |
[XRay][docs] Define requirements on installed log handlers.
Summary:
We update the documentation to define what the requirements are for the
provided XRay log handler. This is to make it clear that the function
pointer provided must do internal synchronisation and that there are no
guarantees provided by XRay on when the function shall be invoked once
it has been installed as a log handler.
Reviewers: rSerge, rengolin
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D26651
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@287073 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'docs/XRay.rst')
| -rw-r--r-- | docs/XRay.rst | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/docs/XRay.rst b/docs/XRay.rst index 17dde1234837..222cc8f2e049 100644 --- a/docs/XRay.rst +++ b/docs/XRay.rst @@ -162,6 +162,19 @@ functions we list below: - ``__xray_unpatch()``: Unpatch the instrumentation points defined in the binary. +There are some requirements on the logging handler to be installed for the +thread-safety of operations to be performed by the XRay runtime library: + +- The function should be thread-safe, as multiple threads may be invoking the + function at the same time. If the logging function needs to do + synchronisation, it must do so internally as XRay does not provide any + synchronisation guarantees outside from the atomicity of updates to the + pointer. +- The pointer provided to ``__xray_set_handler(...)`` must be live even after + calls to ``__xray_remove_handler()`` and ``__xray_unpatch()`` have succeeded. + XRay cannot guarantee that all threads that have ever gotten a copy of the + pointer will not invoke the function. + Trace Analysis Tools -------------------- |