[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH 0/5] stop collection of instruction usage statistics
From: |
Luis Fernando Fujita Pires |
Subject: |
RE: [PATCH 0/5] stop collection of instruction usage statistics |
Date: |
Thu, 27 May 2021 14:25:47 +0000 |
From: Alex Bennée <alex.bennee@linaro.org>
> I have no particular comment to make about the PPC stuff but with the common
> translator loop we have hooks across all converted front ends to identify the
> start of each instruction. It's needed for the TCG plugin instrumentation and
> we
> could in theory use it for more integrated stats across the board.
>
> Out of interest what was the main aim of this code - a view of total executed
> instructions or something more detailed like a breakdown of types and ops?
The legacy instruction decoding logic in the PPC implementation uses a table
that maps opcode patterns (based on different parts of the instructions) to
handlers that actually implement each instruction.
The code that is being removed would list how many times each specific handler
was invoked, so it had more information than just the total count of executed
instructions.
That being said, the code probably wasn't being used for a while now, as it
didn't even compile.
--
Luis Pires
Instituto de Pesquisas ELDORADO <http://www.eldorado.org.br>
Departamento de Computação Embarcada
Aviso Legal - Disclaimer <https://www.eldorado.org.br/disclaimer.html>
- Re: [PATCH 4/5] monitor: removed cpustats command, (continued)
Re: [PATCH 4/5] monitor: removed cpustats command, Dr. David Alan Gilbert, 2021/05/27
[PATCH 5/5] hw/core/cpu: removed cpu_dump_statistics function, Bruno Larsen (billionai), 2021/05/26
Re: [PATCH 0/5] stop collection of instruction usage statistics, Alex Bennée, 2021/05/27