coresight: tmc-etf: Fix snapshot mode update function

When working in snapshot mode function perf_aux_output_begin()
does not set the handle->size because the size is expected to be
deduced by the placement of the "head" and "old" pointers in user
space.  As such there is no point in trying to adjust the amount
of data to copy to the ring buffer.

Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
Reviewed-by: Leo Yan <leo.yan@linaro.org>
Reviewed-by: Suzuki K Poulose <suzuki.poulose@arm.com>
Tested-by: Leo Yan <leo.yan@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Mathieu Poirier 2019-06-19 11:29:08 -06:00 committed by Greg Kroah-Hartman
parent 0402f75efe
commit 99f81eb9c5
1 changed files with 4 additions and 2 deletions

View File

@ -477,9 +477,11 @@ static unsigned long tmc_update_etf_buffer(struct coresight_device *csdev,
/* /*
* The TMC RAM buffer may be bigger than the space available in the * The TMC RAM buffer may be bigger than the space available in the
* perf ring buffer (handle->size). If so advance the RRP so that we * perf ring buffer (handle->size). If so advance the RRP so that we
* get the latest trace data. * get the latest trace data. In snapshot mode none of that matters
* since we are expected to clobber stale data in favour of the latest
* traces.
*/ */
if (to_read > handle->size) { if (!buf->snapshot && to_read > handle->size) {
u32 mask = 0; u32 mask = 0;
/* /*