From: "Alex Bennée" <alex.bennee@linaro.org>
To: Luis Fernando Fujita Pires <luis.pires@eldorado.org.br>
Cc: "farosas@linux.ibm.com" <farosas@linux.ibm.com>,
"richard.henderson@linaro.org" <richard.henderson@linaro.org>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
Lucas Mateus Martins Araujo e Castro
<lucas.araujo@eldorado.org.br>,
Fernando Eckhardt Valle <fernando.valle@eldorado.org.br>,
"qemu-ppc@nongnu.org" <qemu-ppc@nongnu.org>,
Matheus Kowalczuk Ferst <matheus.ferst@eldorado.org.br>,
"david@gibson.dropbear.id.au" <david@gibson.dropbear.id.au>
Subject: Re: [PATCH 0/5] stop collection of instruction usage statistics
Date: Thu, 27 May 2021 15:56:02 +0100 [thread overview]
Message-ID: <87sg281axu.fsf@linaro.org> (raw)
In-Reply-To: <CP2PR80MB36689194A6DF4B3848702C7FDA239@CP2PR80MB3668.lamprd80.prod.outlook.com>
Luis Fernando Fujita Pires <luis.pires@eldorado.org.br> writes:
> 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.
Ahh OK. If you wanted to you could probably re-create that information
using the howvec plugin (see contrib/plugins/howvec) if the decode
tables where added for PPC.
--
Alex Bennée
next prev parent reply other threads:[~2021-05-27 14:59 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-26 20:20 [PATCH 0/5] stop collection of instruction usage statistics Bruno Larsen (billionai)
2021-05-26 20:21 ` [PATCH 1/5] target/ppc: fixed GEN_OPCODE behavior when PPC_DUMP_CPU is set Bruno Larsen (billionai)
2021-05-26 21:13 ` Luis Fernando Fujita Pires
2021-05-26 21:24 ` Richard Henderson
2021-05-27 1:18 ` david
2021-05-26 20:21 ` [PATCH 2/5] target/ppc: remove ppc_cpu_dump_statistics Bruno Larsen (billionai)
2021-05-26 21:25 ` Richard Henderson
2021-05-26 21:27 ` Luis Fernando Fujita Pires
2021-05-27 1:20 ` David Gibson
2021-05-27 4:35 ` David Gibson
2021-05-27 13:22 ` Bruno Piazera Larsen
2021-05-27 14:01 ` Greg Kurz
2021-05-29 5:46 ` David Gibson
2021-05-27 6:01 ` Greg Kurz
2021-05-29 5:47 ` David Gibson
2021-05-31 14:26 ` Bruno Piazera Larsen
2021-05-26 20:21 ` [PATCH 3/5] target/ppc: removed mentions to DO_PPC_STATISTICS Bruno Larsen (billionai)
2021-05-26 21:26 ` Richard Henderson
2021-05-26 21:35 ` Luis Fernando Fujita Pires
2021-05-27 4:37 ` David Gibson
2021-05-26 20:21 ` [PATCH 4/5] monitor: removed cpustats command Bruno Larsen (billionai)
2021-05-26 21:28 ` Richard Henderson
2021-05-27 4:40 ` David Gibson
2021-05-26 21:35 ` Luis Fernando Fujita Pires
2021-05-27 6:40 ` Greg Kurz
2021-05-27 8:09 ` Dr. David Alan Gilbert
2021-05-27 8:30 ` Greg Kurz
2021-05-27 11:24 ` Bruno Piazera Larsen
2021-05-27 14:57 ` Greg Kurz
2021-06-08 15:10 ` Markus Armbruster
2021-06-08 15:15 ` Greg Kurz
2021-05-27 8:08 ` Dr. David Alan Gilbert
2021-05-26 20:21 ` [PATCH 5/5] hw/core/cpu: removed cpu_dump_statistics function Bruno Larsen (billionai)
2021-05-26 21:29 ` Richard Henderson
2021-05-26 21:35 ` Luis Fernando Fujita Pires
2021-05-27 1:21 ` David Gibson
2021-05-27 21:05 ` Eduardo Habkost
2021-05-27 13:57 ` [PATCH 0/5] stop collection of instruction usage statistics Alex Bennée
2021-05-27 14:23 ` Bruno Piazera Larsen
2021-05-27 14:25 ` Luis Fernando Fujita Pires
2021-05-27 14:56 ` Alex Bennée [this message]
2021-05-27 15:39 ` Luis Fernando Fujita Pires
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87sg281axu.fsf@linaro.org \
--to=alex.bennee@linaro.org \
--cc=david@gibson.dropbear.id.au \
--cc=farosas@linux.ibm.com \
--cc=fernando.valle@eldorado.org.br \
--cc=lucas.araujo@eldorado.org.br \
--cc=luis.pires@eldorado.org.br \
--cc=matheus.ferst@eldorado.org.br \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=richard.henderson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.