Minki/linux
1
0
Fork 1
mirror of https://github.com/torvalds/linux.git synced 2024-12-03 17:41:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

perf cs-etm: Fix indentation in function cs_etm__process_decoder_queue()

Browse Source 
Fixing wrong indentation of the while() loop - no change of
functionality.

Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
Tested-by: Leo Yan <leo.yan@linaro.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Suzuki Poulouse <suzuki.poulose@arm.com>
Cc: coresight@lists.linaro.org
Cc: linux-arm-kernel@lists.infradead.org
Fixes: 3fa0e83e29 ("perf cs-etm: Modularize main packet processing loop")
Link: http://lkml.kernel.org/r/20190524173508.29044-9-mathieu.poirier@linaro.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
This commit is contained in:
Mathieu Poirier 2019-05-24 11:34:59 -06:00 committed by Arnaldo Carvalho de Melo
parent 5f7cb03555
commit 882f4874ad
1 changed files with 54 additions and 54 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

108
tools/perf/util/cs-etm.c
View File

@ -1578,64 +1578,64 @@ static int cs_etm__process_decoder_queue(struct cs_etm_queue *etmq)
packet_queue = cs_etm__etmq_get_packet_queue(etmq); packet_queue = cs_etm__etmq_get_packet_queue(etmq);
/* Process each packet in this chunk */ /* Process each packet in this chunk */
while (1) { while (1) {
ret = cs_etm_decoder__get_packet(packet_queue, ret = cs_etm_decoder__get_packet(packet_queue,
etmq->packet); etmq->packet);
if (ret <= 0) if (ret <= 0)
/*
* Stop processing this chunk on
* end of data or error
*/
break;
/* /*
* Since packet addresses are swapped in packet * Stop processing this chunk on
* handling within below switch() statements, * end of data or error
* thus setting sample flags must be called
* prior to switch() statement to use address
* information before packets swapping.
*/ */
ret = cs_etm__set_sample_flags(etmq); break;
if (ret < 0)
break;
switch (etmq->packet->sample_type) { /*
case CS_ETM_RANGE: * Since packet addresses are swapped in packet
/* * handling within below switch() statements,
* If the packet contains an instruction * thus setting sample flags must be called
* range, generate instruction sequence * prior to switch() statement to use address
* events. * information before packets swapping.
*/ */
cs_etm__sample(etmq); ret = cs_etm__set_sample_flags(etmq);
break; if (ret < 0)
case CS_ETM_EXCEPTION: break;
case CS_ETM_EXCEPTION_RET:
/* switch (etmq->packet->sample_type) {
* If the exception packet is coming, case CS_ETM_RANGE:
* make sure the previous instruction /*
* range packet to be handled properly. * If the packet contains an instruction
*/ * range, generate instruction sequence
cs_etm__exception(etmq); * events.
break; */
case CS_ETM_DISCONTINUITY: cs_etm__sample(etmq);
/* break;
* Discontinuity in trace, flush case CS_ETM_EXCEPTION:
* previous branch stack case CS_ETM_EXCEPTION_RET:
*/ /*
cs_etm__flush(etmq); * If the exception packet is coming,
break; * make sure the previous instruction
case CS_ETM_EMPTY: * range packet to be handled properly.
/* */
* Should not receive empty packet, cs_etm__exception(etmq);
* report error. break;
*/ case CS_ETM_DISCONTINUITY:
pr_err("CS ETM Trace: empty packet\n"); /*
return -EINVAL; * Discontinuity in trace, flush
default: * previous branch stack
break; */
} cs_etm__flush(etmq);
break;
case CS_ETM_EMPTY:
/*
* Should not receive empty packet,
* report error.
*/
pr_err("CS ETM Trace: empty packet\n");
return -EINVAL;
default:
break;
} }
}
return ret; return ret;
} }