Age | Commit message (Collapse) | Author |
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/2397212
Change-Id: I2eef649581113157f94e5de5699bacf70992d575
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/2397212
Change-Id: I420f772a7d66ce8ca371bddff08550f1b954909f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/2397212
Change-Id: I03eeab2e465d8e1b1628015d8e54459ddecc5f5f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
This project was upgraded with external_updater.
Usage: tools/external_updater/updater.sh update OpenCSD
For more info, check https://cs.android.com/android/platform/superproject/+/master:tools/external_updater/README.md
Bug: 263556196
Test: TreeHugger
Change-Id: I8683cfcc0d1283feab4e84c447f0887dc9797e0a
|
|
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Adds in support for cores with FEAT_ITE (arch v9.4, ETE p1v3), which allows a
PE to inject instumentation data directly into the ETE trace stream.
This new packet is decoded and passed through to the decoder.
This generates a new generic packet type OCSD_GEN_TRC_ELEM_INSTRUMENTATION
containing the instrumentation payload.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Mispredict handler was removing unneeded addresses between the mispredict
packet and the atom packet, without deleting the item afterwards.
Erase now also deletes the unwanted item.
Fixes issue #52 from github
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
ocsdlib has ASAN enabled for this build - follow suit with test progs to
allow use of ASAN checks in debug.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/2285438
Change-Id: I9ace7b16bcd435a03b219d8764d17fb7b00046ea
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/2285438
Change-Id: Ib9b61e3c8d27ca377f8412f6b5b3fb18d72de063
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/2285438
Change-Id: Idab0e6a4222f55bd42d65644092f4e0f8cb8a2fc
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Test: make & run simpleperf_unit_test
Change-Id: Id84ea0d8b492d08285d757aa677d6de6a67e09b2
|
|
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
The handling for trace streams using HSYNC and FSYNC packets was not
correctly accounting for an odd number of HSYNC packets in a stream,
which could result in read past end of buffer.
Routine changed to ensure handling of 2 byte aligned end of buffer,
and FSYNC across a pair of input buffers.
Reported-by: yabinc <yabinc@google.com> (github issue #50)
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
The reset on 4xFSYNC for frame aligned input data blocks was
not correctly throwing an error on bad input, resulting in
subsequent block overruns
Additionally, a 4XFSYNC at the end of block could also result in
an read past end of block.
Reported-by: yabinc <yabinc@google.com> (github issue #49)
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Currently setting incorrect configuration flags on initialising the decode
tree for the demux formatter does not result in error messages being seen
Fix adjusts the demux module to split init and configure to allow attachement
of logger before configure.
Also reversed the priority of HSYNC and FSYNC alignment values.
Always 2 byte multiples on HSYNC or HSYNC + FSYNC, 4 byte if FSYNC only
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Pathalogical case where 7 ID changes in a frame. all with data associated with previous ID,
may cause buffer overrun issue. Fix by adding in additional buffer for final data element.
Highly unlikely / impossible to occur in valid system generated frame data but could be
triggered by corrupt / invalid input data.
Reported-by: yabinc <yabinc@google.com> (github issue #51)
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
A series of tests to ensure that the Trace Demux module is working correctly.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
issue #48)
Indexing the address bytes misses the last byte of 4 continuation bytes resulting in
incorrect decode. This patch fixes the indexing.
Reported-by vimalraj-rajasekharan (github issue #48)
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Add class to provide a printable string for all ocsd_datapath_resp_t values.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Convert VS2015 build in a new directory for VS2022.
Ensure all variants - debug, release for win32, win64 work.
Fix up DLL example demo build for c_api_pkt_print_test program
with library built as DLL.
Fixup .gitignore for new generated files in VS2022 that do not need to
appear in the repo.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/2270111
Change-Id: I76199779d0894aa6c385d8e7a0a93c814d4a3915
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/2270111
Change-Id: Ic2383aa2328dce58dee5d9b178c61dbace46037a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/2270111
Change-Id: I917984c57aa12d6c781721c5c3abb9def52cf45f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Test: make & run simpleperf_unit_test
Change-Id: I76175e22ac99e8da96af4e7153645b0661bd3e0f
|
|
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
For 64 bit timestamp values with the top bit (63) set, this was being incorrectly masked to 1b0 by the extraction routine.
Handling of 9th (final) byte in the value extraction changed to fix this issue.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
fix a typo in annotation : compplete -> complete
Reported-by: <nothatDinger <52913141+nothatDinger@users.noreply.github.com>
|
|
std::lexicographical_compare usage in snapshot_parser_util.h
requires <algorithm>.
This fixes a build breakage when using ToT libc++ which
no longer transitively include <algorithm>.
Signed-off-by: Manoj Gupta <manojgupta@google.com>
|
|
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
In arch 8.8 and 9.3 a BC instruction is added, which is a direct branch generating an atom element.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Instruction analyser adjusted to recognise the new BC <cond> instruction as
a direct branch which is associated with atom elements.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
FEAT_RME (realms) adds a NSE bit to context packets to add security states root and realm.
This adds support for this new bit.
Assumes that the NSE bit is guaranteed to RAZ for any device that does not have the feature.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
immediately.
Handling for speculative trace added caching of trace elements to be committed cancelled.
This results in elements being output in blocks on commit.
This introduced an issue where a context element could change a context, but not be output to the client,
before subsequent atoms were processed, and memory accesses to the client could occur without the new
context being seen. This could result in the incorrect memory being served from the client to the decoder,
resulting in decode issues.
|
|
Add an interface function to implement a call to the memory accessor that invalidates any
caching in the accessor object.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
Adds interface to allow decoders to invalidate any cacheing in the memory accessor objects.
Typically used when a context trace element may have changed the memory map.
Signed-off-by: Mike Leach <mike.leach@linaro.org>
|
|
a15ec4bb20 am: 0bc6549445 am: b4fdf28724
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/1856139
Change-Id: Id3cdd24096c7d5737a945a2d6793742a4317b416
|
|
a15ec4bb20 am: 0bc6549445
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/1856139
Change-Id: I05b09ad98b017fa6115878600a78d84df4b9a7bf
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/1856139
Change-Id: I494e75e0945fbae91caa8ac9a937503711f153f6
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/1856139
Change-Id: I111d0f27a24f5c356e40c7930bb6820ac2f9b99d
|
|
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/1856139
Change-Id: I18478ee7ba5c58545a8dbdd2f6a3fe81a5ab54f2
|
|
|
|
a3ae22590f am: 5d330c3cd9 am: ecdd4e08d6
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/1854148
Change-Id: If8949696baad31750c6733f083143a85581f10d9
|
|
a3ae22590f am: 5d330c3cd9
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/1854148
Change-Id: Idcd61c80cca6a1650e89c9e02bdb420580346973
|
|
a3ae22590f
Original change: https://android-review.googlesource.com/c/platform/external/OpenCSD/+/1854148
Change-Id: I2a51a76fffdc95f9c78b942aee2a843dc0d497e0
|