Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-21 Thread Wu, Fei
On 4/17/2023 9:01 PM, Wu, Fei wrote: > On 4/17/2023 8:11 PM, Alex Bennée wrote: >> >> "Wu, Fei" writes: >> >>> On 4/11/2023 3:27 PM, Alex Bennée wrote: "Wu, Fei" writes: > On 4/10/2023 6:36 PM, Alex Bennée wrote: >> >> Richard Henderson writes: >> >>> On 4/6/23

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-17 Thread Wu, Fei
On 4/17/2023 8:11 PM, Alex Bennée wrote: > > "Wu, Fei" writes: > >> On 4/11/2023 3:27 PM, Alex Bennée wrote: >>> >>> "Wu, Fei" writes: >>> On 4/10/2023 6:36 PM, Alex Bennée wrote: > > Richard Henderson writes: > >> On 4/6/23 00:46, Alex Bennée wrote: >>> If your aim is

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-17 Thread Alex Bennée
"Wu, Fei" writes: > On 4/11/2023 3:27 PM, Alex Bennée wrote: >> >> "Wu, Fei" writes: >> >>> On 4/10/2023 6:36 PM, Alex Bennée wrote: Richard Henderson writes: > On 4/6/23 00:46, Alex Bennée wrote: >> If your aim is to examine JIT efficiency what is wrong with the curr

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-17 Thread Wu, Fei
On 4/11/2023 3:27 PM, Alex Bennée wrote: > > "Wu, Fei" writes: > >> On 4/10/2023 6:36 PM, Alex Bennée wrote: >>> >>> Richard Henderson writes: >>> On 4/6/23 00:46, Alex Bennée wrote: > If your aim is to examine JIT efficiency what is wrong with the current > "info jit" that you can

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-12 Thread Wu, Fei
On 4/12/2023 9:28 PM, Alex Bennée wrote: > > "Wu, Fei" writes: > >> On 4/11/2023 3:27 PM, Alex Bennée wrote: >>> >>> "Wu, Fei" writes: >>> On 4/10/2023 6:36 PM, Alex Bennée wrote: > > Richard Henderson writes: > >> On 4/6/23 00:46, Alex Bennée wrote: >>> If your aim is

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-12 Thread Alex Bennée
"Wu, Fei" writes: > On 4/11/2023 3:27 PM, Alex Bennée wrote: >> >> "Wu, Fei" writes: >> >>> On 4/10/2023 6:36 PM, Alex Bennée wrote: Richard Henderson writes: > On 4/6/23 00:46, Alex Bennée wrote: >> If your aim is to examine JIT efficiency what is wrong with the curr

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-12 Thread Wu, Fei
On 4/11/2023 3:27 PM, Alex Bennée wrote: > > "Wu, Fei" writes: > >> On 4/10/2023 6:36 PM, Alex Bennée wrote: >>> >>> Richard Henderson writes: >>> On 4/6/23 00:46, Alex Bennée wrote: > If your aim is to examine JIT efficiency what is wrong with the current > "info jit" that you can

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-11 Thread Alex Bennée
"Wu, Fei" writes: > On 4/10/2023 6:36 PM, Alex Bennée wrote: >> >> Richard Henderson writes: >> >>> On 4/6/23 00:46, Alex Bennée wrote: If your aim is to examine JIT efficiency what is wrong with the current "info jit" that you can access via the HMP? Also I'm wondering if its

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-10 Thread Wu, Fei
On 4/10/2023 6:36 PM, Alex Bennée wrote: > > Richard Henderson writes: > >> On 4/6/23 00:46, Alex Bennée wrote: >>> If your aim is to examine JIT efficiency what is wrong with the current >>> "info jit" that you can access via the HMP? Also I'm wondering if its >>> time to remove the #ifdefs fro

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-10 Thread Alex Bennée
"Wu, Fei" writes: > On 4/6/2023 3:46 PM, Alex Bennée wrote: >> >> Fei Wu writes: >> >>> The translation ratio of host to guest instruction count is one of the >>> key performance factor of binary translation. TCG doesn't collect host >>> instruction count at present, it does collect host ins

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-10 Thread Alex Bennée
Richard Henderson writes: > On 4/6/23 00:46, Alex Bennée wrote: >> If your aim is to examine JIT efficiency what is wrong with the current >> "info jit" that you can access via the HMP? Also I'm wondering if its >> time to remove the #ifdefs from CONFIG_PROFILER because I doubt the >> extra dat

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-07 Thread Richard Henderson
On 4/6/23 00:46, Alex Bennée wrote: If your aim is to examine JIT efficiency what is wrong with the current "info jit" that you can access via the HMP? Also I'm wondering if its time to remove the #ifdefs from CONFIG_PROFILER because I doubt the extra data it collects is that expensive. Richard,

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-06 Thread Wu, Fei
On 4/6/2023 3:46 PM, Alex Bennée wrote: > > Fei Wu writes: > >> The translation ratio of host to guest instruction count is one of the >> key performance factor of binary translation. TCG doesn't collect host >> instruction count at present, it does collect host instruction size >> instead, alth

Re: [PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-06 Thread Alex Bennée
Fei Wu writes: > The translation ratio of host to guest instruction count is one of the > key performance factor of binary translation. TCG doesn't collect host > instruction count at present, it does collect host instruction size > instead, although they are not the same thing as instruction s

[PATCH 1/2] accel/tcg/plugin: export host insn size

2023-04-05 Thread Fei Wu
The translation ratio of host to guest instruction count is one of the key performance factor of binary translation. TCG doesn't collect host instruction count at present, it does collect host instruction size instead, although they are not the same thing as instruction size might not be fixed, ins