perf cs-etm: Fix indentation in function cs_etm__process_decoder_queue()
authorMathieu Poirier <mathieu.poirier@linaro.org>
Fri, 24 May 2019 17:34:59 +0000 (11:34 -0600)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 10 Jun 2019 18:50:02 +0000 (15:50 -0300)
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: 3fa0e83e2948 ("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>
tools/perf/util/cs-etm.c

index a74c53a..68fec6f 100644 (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);
 
-               /* Process each packet in this chunk */
-               while (1) {
-                       ret = cs_etm_decoder__get_packet(packet_queue,
-                                                        etmq->packet);
-                       if (ret <= 0)
-                               /*
-                                * Stop processing this chunk on
-                                * end of data or error
-                                */
-                               break;
+       /* Process each packet in this chunk */
+       while (1) {
+               ret = cs_etm_decoder__get_packet(packet_queue,
+                                                etmq->packet);
+               if (ret <= 0)
+                       /*
+                        * Stop processing this chunk on
+                        * end of data or error
+                        */
+                       break;
+
+               /*
+                * Since packet addresses are swapped in packet
+                * handling within below switch() statements,
+                * 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);
+               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,
-                        * thus setting sample flags must be called
-                        * prior to switch() statement to use address
-                        * information before packets swapping.
+                        * If the packet contains an instruction
+                        * range, generate instruction sequence
+                        * events.
                         */
-                       ret = cs_etm__set_sample_flags(etmq);
-                       if (ret < 0)
-                               break;
-
-                       switch (etmq->packet->sample_type) {
-                       case CS_ETM_RANGE:
-                               /*
-                                * If the packet contains an instruction
-                                * range, generate instruction sequence
-                                * events.
-                                */
-                               cs_etm__sample(etmq);
-                               break;
-                       case CS_ETM_EXCEPTION:
-                       case CS_ETM_EXCEPTION_RET:
-                               /*
-                                * If the exception packet is coming,
-                                * make sure the previous instruction
-                                * range packet to be handled properly.
-                                */
-                               cs_etm__exception(etmq);
-                               break;
-                       case CS_ETM_DISCONTINUITY:
-                               /*
-                                * Discontinuity in trace, flush
-                                * previous branch stack
-                                */
-                               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;
-                       }
+                       cs_etm__sample(etmq);
+                       break;
+               case CS_ETM_EXCEPTION:
+               case CS_ETM_EXCEPTION_RET:
+                       /*
+                        * If the exception packet is coming,
+                        * make sure the previous instruction
+                        * range packet to be handled properly.
+                        */
+                       cs_etm__exception(etmq);
+                       break;
+               case CS_ETM_DISCONTINUITY:
+                       /*
+                        * Discontinuity in trace, flush
+                        * previous branch stack
+                        */
+                       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;
 }