> 在 2019年1月16日，23:00，Dr. David Alan Gilbert 写道：
>
> * Philippe Mathieu-Daudé (phi...@redhat.com) wrote:
>>> On 1/16/19 5:50 AM, Fei Li wrote:
>>> Hi all,
>>>
>>> Kindly ping. :)
>>>
>>> As my v10 of qemu_thread_create partl
> 在 2019年1月8日，01:18，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> qemu_thread_create() abort()s on error. Not nice. Give it a return
>> value and an Error ** argument, so it can return success/failure.
>>
>> Considering qemu_thread_create() is quite wid
> 在 2019年1月8日，00:50，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> Always call migrate_set_error() to set the error state without relying
>> on whether multifd_save_cleanup() succeeds. As the passed &local_err
>> is never used in multifd_save_cleanup(), r
> 在 2019年1月8日，01:21，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> Utilize the existed errp to propagate the error instead of the
>> temporary &error_abort.
>>
>> Cc: Markus Armbruster
>> Cc: Marc-André Lureau
>> Signed-off-by: Fei L
> 在 2019年1月8日，01:50，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> For iothread_complete: utilize the existed errp to propagate the
>> error and do the corresponding cleanup to replace the temporary
>> &error_abort.
>>
>> For qemu_signalfd_
> 在 2019年1月8日，01:54，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> Supplement the error handling for vnc_thread_worker_thread: add
>> an Error parameter for it to propagate the error to its caller to
>> handle in case it fails, and make it return a Boolean to i
> 在 2019年1月8日，01:55，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> To avoid the segmentation fault in qemu_thread_join(), just directly
>> return when the QemuThread *thread failed to be created in either
>> qemu-thread-posix.c or qemu-thread-win32.c.
>>
> 在 2019年1月9日，22:36，Markus Armbruster 写道：
>
> Fei Li writes:
>
>>> 在 2019/1/9 上午1:07, Markus Armbruster 写道:
>>> fei writes:
>>>
>>>>> 在 2019年1月8日，01:18，Markus Armbruster 写道：
>>>>>
>>>>> Fei Li wri
> 在 2019年1月9日，23:24，Markus Armbruster 写道：
>
> Fei Li writes:
>
>>> 在 2019/1/9 上午1:29, Markus Armbruster 写道:
>>> fei writes:
>>>
>>>>> 在 2019年1月8日，01:55，Markus Armbruster 写道：
>>>>>
>>>>> Fei Li writes
> 在 2019年1月9日，23:26，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> Update qemu_thread_create()'s callers by
>> - setting an error on qemu_thread_create() failure for callers that
>> set an error on failure;
>> - reporting the error and returning fai
> 在 2019年1月8日，02:13，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> Supplement the error handling for touch_all_pages: add an Error
>> parameter for it to propagate the error to its caller to do the
>> handling in case it fails.
>>
>> Cc: M
Sorry for the late reply.
Yes, so sorry that this gets stuck for a long while. Actually I needs to send a
v14 with a little update from v13. I will send a new version ASAP.
发自我的 iPhone
> 在 2019年7月12日，13:57，Markus Armbruster 写道：
>
> Did this get stuck?
>
> Fei Li w
> 在 2019年2月1日，20:58，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> From: Fei Li
>>
>> Utilize the existed errp to propagate the error and do the
>> corresponding cleanup to replace the temporary &error_abort.
>>
>> Cc: Markus Arm
> 在 2019年2月1日，20:33，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> From: Fei Li
>>
>> The callers of qemu_init_vcpu() already passed the **errp to handle
>> errors. In view of this, add a new Error parameter to qemu_init_vcpu()
>> and a
> 在 2019年2月1日，22:03，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> From: Fei Li
>>
>> For iothread_complete: utilize the existed errp to propagate the
>
> "For iothread_complete" is redundant, isn't it?
Emm, right. Will remove it in the
> 在 2019年2月1日，22:13，Markus Armbruster 写道：
>
> Fei Li writes:
>
>> From: Fei Li
>>
>> For qemu_signalfd_compat: set errno, do some cleanup, and return
>
> "For iothread_complete" is redundant, isn't it?
Yep, will remove it. Thank
n x86,
and these (APIC/IOAPIC) live right below 4GiB.
So just split the RAM range into two portions:
- 1 GiB range from 0x8000 to 0xc000.
- The remainder at 0x1
...leaving a hole between the ranges.
Signed-off-by: Andrei Warkentin
Signed-off-by: Fei Wu
---
hw/riscv/virt.c
On 8/1/2023 6:46 AM, Daniel Henrique Barboza wrote:
>
>
> On 7/30/23 22:53, Fei Wu wrote:
>> riscv virt platform's memory started at 0x8000 and
>> straddled the 4GiB boundary. Curiously enough, this choice
>> of a memory layout will prevent from launching a VM
On 8/1/2023 6:46 AM, Daniel Henrique Barboza wrote:
>
>
> On 7/30/23 22:53, Fei Wu wrote:
>> riscv virt platform's memory started at 0x8000 and
>> straddled the 4GiB boundary. Curiously enough, this choice
>> of a memory layout will prevent from launching a VM
On 8/3/2023 11:07 PM, Andrew Jones wrote:
> On Mon, Jul 31, 2023 at 09:53:17AM +0800, Fei Wu wrote:
>> riscv virt platform's memory started at 0x8000 and
>> straddled the 4GiB boundary. Curiously enough, this choice
>> of a memory layout will prevent from launching a
-II platform (WIP) is needed to provide related ACPI
tables.
For testing purposes only, we used a workaround to generate the ACPI
tables in Qemu with a dedicated downstream patch.
[1] https://github.com/riscv-non-isa/riscv-server-platform
[2] https://github.com/tianocore/edk2.git
Fei Wu (2):
hw
- PCIe AHCI
- PCIe NIC
- No virtio device
- No fw_cfg device
- No ACPI table provided
- Only minimal device tree nodes
[1] https://github.com/riscv-non-isa/riscv-server-platform
Signed-off-by: Fei Wu
---
configs/devices/riscv64-softmmu/default.mak |1 +
hw/riscv/Kconfig
/server_platform_requirements.adoc
Signed-off-by: Fei Wu
---
hw/riscv/server_platform_ref.c | 6 +++-
target/riscv/cpu-qom.h | 1 +
target/riscv/cpu.c | 62 ++
3 files changed, 68 insertions(+), 1 deletion(-)
diff --git a/hw/riscv/server_platform_ref.c b/hw/riscv
On 3/5/2024 3:35 AM, Daniel Henrique Barboza wrote:
>
>
> On 3/4/24 07:25, Fei Wu wrote:
>> The RISC-V Server Platform specification[1] defines a standardized set
>> of hardware and software capabilities, that portable system software,
>> such as OS and hypervisors can
On 3/5/2024 3:43 AM, Daniel Henrique Barboza wrote:
>
>
> On 3/4/24 07:25, Fei Wu wrote:
>> The harts requirements of RISC-V server platform [1] require RVA23 ISA
>> profile support, plus Sv48, Svadu, H, Sscofmpf etc. This patch provides
>> a virt CPU type (rvsp-re
Hi Conor,
On 3/6/2024 8:27 AM, Conor Dooley wrote:
> On Mon, Mar 04, 2024 at 06:25:39PM +0800, Fei Wu wrote:
>
>> +name = riscv_isa_string(cpu_ptr);
>> +qemu_fdt_setprop_string(ms->fdt, cpu_name, "riscv,isa", name);
>> +g_free(name)
On 3/6/2024 8:19 AM, Alistair Francis wrote:
> On Mon, Mar 4, 2024 at 8:28 PM Fei Wu wrote:
>>
>> The RISC-V Server Platform specification[1] defines a standardized set
>> of hardware and software capabilities, that portable system software,
>> such as OS and hypervisor
On 3/5/2024 1:58 PM, Wu, Fei wrote:
> On 3/5/2024 3:43 AM, Daniel Henrique Barboza wrote:
>>
>>
>> On 3/4/24 07:25, Fei Wu wrote:
>>> The harts requirements of RISC-V server platform [1] require RVA23 ISA
>>> profile support, plus Sv48, Svadu, H, Sscofmpf
On 3/7/2024 8:48 AM, Alistair Francis wrote:
> On Thu, Mar 7, 2024 at 5:13 AM Atish Kumar Patra wrote:
>>
>> On Wed, Mar 6, 2024 at 4:56 AM Wu, Fei wrote:
>>>
>>> On 3/6/2024 8:19 AM, Alistair Francis wrote:
>>>> On Mon, Mar 4, 2024 at 8:28 PM
On 3/6/2024 9:26 PM, Wu, Fei wrote:
> On 3/5/2024 1:58 PM, Wu, Fei wrote:
>> On 3/5/2024 3:43 AM, Daniel Henrique Barboza wrote:
>>>
>>>
>>> On 3/4/24 07:25, Fei Wu wrote:
>>>> The harts requirements of RISC-V server platform [1] require RVA
On 3/23/2024 3:14 AM, Atish Kumar Patra wrote:
> On Tue, Mar 12, 2024 at 6:53 AM Fei Wu wrote:
>>
>> The RISC-V Server Platform specification[1] defines a standardized set
>> of hardware and software capabilities, that portable system software,
>> such as OS and hy
On 3/8/2024 5:20 PM, Andrew Jones wrote:
> On Thu, Mar 07, 2024 at 02:26:18PM +0800, Wu, Fei wrote:
>> On 3/7/2024 8:48 AM, Alistair Francis wrote:
>>> On Thu, Mar 7, 2024 at 5:13 AM Atish Kumar Patra
>>> wrote:
>>>>
>>>> On Wed, Mar 6, 2024 a
On 3/8/2024 6:15 AM, Marcin Juszkiewicz wrote:
> W dniu 4.03.2024 o 11:25, Fei Wu pisze:
>
>> The RISC-V Server Platform specification[1] defines a standardized
>> set of hardware and software capabilities, that portable system
>> software, such as OS and hypervisors can
On 3/8/2024 3:15 AM, Daniel Henrique Barboza wrote:
>
>
> On 3/7/24 04:36, Wu, Fei wrote:
>> On 3/6/2024 9:26 PM, Wu, Fei wrote:
>>> On 3/5/2024 1:58 PM, Wu, Fei wrote:
>>>> On 3/5/2024 3:43 AM, Daniel Henrique Barboza wrote:
>>>>>
>>
definition the 1st patch
- replace sifive_test with power reset syscon
- add versioning scheme to fdt
- integrate recent changes in virt.c
- use g_autofree
- create_fdt/finalize_fdt
- riscv_isa_write_fdt
- some cleanups
Fei Wu (2):
target/riscv: Add server platform reference cpu
hw/riscv
/server_platform_requirements.adoc
Signed-off-by: Fei Wu
---
target/riscv/cpu-qom.h | 1 +
target/riscv/cpu.c | 61 ++
2 files changed, 62 insertions(+)
diff --git a/target/riscv/cpu-qom.h b/target/riscv/cpu-qom.h
index 3670cfe6d9..adb934d19e 100644
--- a/target/riscv/cpu-qom.h
- PCIe AHCI
- PCIe NIC
- No virtio device
- No fw_cfg device
- No ACPI table provided
- Only minimal device tree nodes
[1] https://github.com/riscv-non-isa/riscv-server-platform
Signed-off-by: Fei Wu
---
configs/devices/riscv64-softmmu/default.mak |1 +
hw/riscv/Kconfig
On 3/23/2023 2:59 PM, LIU Zhiwei wrote:
>
> On 2023/3/23 14:00, Wu, Fei wrote:
>> On 3/23/2023 1:37 PM, LIU Zhiwei wrote:
>>> On 2023/3/23 10:44, Fei Wu wrote:
>>>> Currently it's assumed the 2 low bits of mmu_idx map to privilege mode,
>>>> this
On 3/23/2023 8:38 AM, Wu, Fei wrote:
> On 3/22/2023 9:19 PM, Richard Henderson wrote:
>> On 3/22/23 05:12, Fei Wu wrote:
>>> Kernel needs to access user mode memory e.g. during syscalls, the window
>>> is usually opened up for a very limited time through MSTATUS.SUM, t
On 3/23/2023 11:53 PM, Richard Henderson wrote:
> On 3/22/23 19:44, Fei Wu wrote:
>> Currently it's assumed the 2 low bits of mmu_idx map to privilege mode,
>> this assumption won't last as we are about to add more mmu_idx.
>>
>> Signed-off-by: F
On 3/24/2023 12:07 AM, Richard Henderson wrote:
> On 3/22/23 23:00, Wu, Fei wrote:
>>>> + ctx->priv = env->priv;
>>>
>>> This is not right. You should put env->priv into tb flags before you use
>>> it in translation.
>>>
>> I s
On 3/24/2023 9:02 AM, Wu, Fei wrote:
> On 3/23/2023 11:53 PM, Richard Henderson wrote:
>> On 3/22/23 19:44, Fei Wu wrote:
>>> Currently it's assumed the 2 low bits of mmu_idx map to privilege mode,
>>> this assumption won't last as we are about to add more m
On 3/24/2023 10:37 AM, Richard Henderson wrote:
> On 3/23/23 18:20, Wu, Fei wrote:
>> I lack some background here, why should tb_flags be preferred if env has
>> the same info? Then for reading from tb_flags, we need to add it to
>> tb_flags first.
>
> We read from tb
Currently it's assumed the 2 low bits of mmu_idx map to privilege mode,
this assumption won't last as we are about to add more mmu_idx. Here an
individual priv field is added into TB_FLAGS.
Signed-off-by: Fei Wu
---
target/riscv/cpu.h | 2 +-
ta
v4 -> v5:
* add priv to tb_flags
v3 -> v4:
* seperate priv from mmu_idx
* use index 2 for S+SUM mmu_idx
* no tlb_flush for MPRV / MPP changes
Fei Wu (2):
target/riscv: separate priv from mmu_idx
target/riscv: reduce overhead of MSTATUS_SUM change
target/riscv
On 3/24/2023 9:02 AM, Wu, Fei wrote:
> On 3/23/2023 11:53 PM, Richard Henderson wrote:
>> On 3/22/23 19:44, Fei Wu wrote:
>>> Currently it's assumed the 2 low bits of mmu_idx map to privilege mode,
>>> this assumption won't last as we are about to add more m
ry to flush tlb anymore when SUM changes. This is similar to how
ARM handles Privileged Access Never (PAN).
Result of 'pipe 10' from unixbench boosts from 223656 to 1705006. Many
other syscalls benefit a lot from this too.
Signed-off-by: Fei Wu
---
target/riscv/cpu.h
On 3/28/2023 12:43 AM, Daniel Henrique Barboza wrote:
>
>
> On 3/25/23 07:54, Richard Henderson wrote:
>> This builds on Fei and Zhiwei's SUM and TB_FLAGS changes.
>>
>> * Reclaim 5 TB_FLAGS bits, since we nearly ran out.
>>
>> * Using cpu_mmu_i
ted.
.global _start
.text
_start:
csrr t3, 0x4
Thanks,
Fei.
On 3/25/2023 6:54 PM, Richard Henderson wrote:
> This builds on Fei and Zhiwei's SUM and TB_FLAGS changes.
>
> * Reclaim 5 TB_FLAGS bits, since we nearly ran out.
>
> * Using cpu_mmu_index(env, true) is insufficient to implement
> HLVX properly. While that choo
On 4/4/2023 3:11 PM, LIU Zhiwei wrote:
>
> On 2023/4/4 14:42, Wu, Fei wrote:
>> On 3/25/2023 6:54 PM, Richard Henderson wrote:
>>> This builds on Fei and Zhiwei's SUM and TB_FLAGS changes.
>>>
>>> * Reclaim 5 TB_FLAGS bits, since we nearly ran out
146, 135
0x8041abd0, 1, 1, 46032689, 123
0x8041ab3c, 1, 1, 46021650, 123
0x8045ffe8, 1, 5, 40927215, 328
Fei Wu (2):
accel/tcg/plugin: export host insn size
plugins/hotblocks: add host insn size
accel/tcg/plugin-gen.c | 1 +
contrib/plugins/hotblocks
It's only valid when inline=false, otherwise it's default to 0.
Signed-off-by: Fei Wu
---
contrib/plugins/hotblocks.c | 24 +++-
1 file changed, 15 insertions(+), 9 deletions(-)
diff --git a/contrib/plugins/hotblocks.c b/contrib/plugins/hotblocks.c
index
fixed, instruction size is still a valid estimation.
Signed-off-by: Fei Wu
---
accel/tcg/plugin-gen.c | 1 +
include/qemu/plugin.h| 2 ++
include/qemu/qemu-plugin.h | 8
plugins/api.c| 5 +
plugins/qemu-plugins.symbols | 1 +
5 files changed, 17 inser
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
linaro.org>
>
> The series ended up moving all the useful bits of CONFIG_PROFILER into
> tb stats which was dynamically controlled on a per TB basis. Now that
> the perf integration stuff was merged maybe there is a simpler series to
> be picked out of the remains?
>
> Fei
On 5/3/2023 4:12 PM, Richard Henderson wrote:
> On 4/21/23 14:24, Fei Wu wrote:
>> From: "Vanderson M. do Rosario"
>>
>> To store statistics for each TB, we created a TBStatistics structure
>> which is linked with the TBs. TBStatistics can stay alive a
On 5/3/2023 4:28 PM, Richard Henderson wrote:
> On 4/21/23 14:24, Fei Wu wrote:
>> From: "Vanderson M. do Rosario"
>>
>> If a TB has a TBS (TBStatistics) with the TB_EXEC_STATS
>> enabled, then we instrument the start code of this TB
>> to atomical
On 4/22/2023 12:42 AM, Alex Bennée wrote:
>
> Fei Wu writes:
>
>> This patch series were done by Vanderson and Alex originally in 2019, I
>> (Fei Wu) rebased them on latest upstream from:
>> https://github.com/stsquad/qemu/tree/tcg/tbstats-and-perf-v10
>> an
On 5/12/2023 4:42 PM, Alex Bennée wrote:
>
> "Wu, Fei" writes:
>
>> On 4/22/2023 12:42 AM, Alex Bennée wrote:
>>>
>>> Fei Wu writes:
>>>
>>>> This patch series were done by Vanderson and Alex originally in 2019, I
>>>>
struction after applying this patch.
Thanks,
Fei.
> This actually reverts commit 0ee342256af9205e7388efdf193a6d8f1ba1a617,
> Some comments are also added to indicate that predicate() must be
> provided for an implemented CSR.
>
> Reported-by: Fei Wu
> Signed-off-by: Bin
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
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:
>>>>>
>>>
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
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:
>>>>>
>>>
This patch series were done by Vanderson and Alex originally in 2019, I
(Fei Wu) rebased them on latest upstream from:
https://github.com/stsquad/qemu/tree/tcg/tbstats-and-perf-v10
and send out this review per Alex's request, I will continue to address
any future review comments here. As
From: "Vanderson M. do Rosario"
If a TB has a TBS (TBStatistics) with the TB_EXEC_STATS
enabled, then we instrument the start code of this TB
to atomically count the number of times it is executed.
We count both the number of "normal" executions and atomic
executions of a TB.
The execution count
From: "Vanderson M. do Rosario"
To store statistics for each TB, we created a TBStatistics structure
which is linked with the TBs. TBStatistics can stay alive after
tb_flush and be relinked to a regenerated TB. So the statistics can
be accumulated even through flushes.
The goal is to have all pr
From: "Vanderson M. do Rosario"
If a TB has a TBS (TBStatistics) with the TB_JIT_STATS enabled then we
collect statistics of its translation processes and code translation.
To help with collection we include the TCGProfile structure
unconditionally. It will have further alterations in future com
From: Alex Bennée
This shouldn't live in the monitor code anyway. While we are at it
make it an uint64_t as we won't be dealing in negative numbers.
Signed-off-by: Alex Bennée
---
accel/tcg/monitor.c | 2 --
accel/tcg/tb-stats.c | 2 ++
include/qemu/timer.h | 2 +-
3 files changed, 3 insertio
From: "Vanderson M. do Rosario"
Replace all others CONFIG_PROFILER statistics and migrate it to
TBStatistics system. However, TCGProfiler still exists and can
be use to store global statistics and times. All TB related
statistics goes to TBStatistics.
Signed-off-by: Vanderson M. do Rosario
Mess
From: Alex Bennée
This starts the conversion of CONFIG_PROFILER data collection to under
the TBStats system. We introduce a new flag TB_JIT_TIME and start
tracking how much time is spent restoring execution state from a given
TB.
Signed-off-by: Alex Bennée
---
accel/tcg/translate-all.c | 23 ++
From: Alex Bennée
We continue the conversion of CONFIG_PROFILER data to TBStats by
measuring the time it takes to generate code. Instead of calculating
elapsed time as we do we simply store key timestamps in the profiler
structure and then calculate the totals and add them to TBStats under
lock.
From: "Vanderson M. do Rosario"
Dump the hottest TBs if -d tb_stats,dump_limit=N is used.
Example of output for the 3 hottest TBs:
TB id:1 | phys:0x34d54 virt:0x00034d54 flags:0xf0
| exec:4828932/0 guest inst cov:16.38%
| trans:1 ints: g:3 op:82 op_opt:34 spills:3
From: "Vanderson M. do Rosario"
Adding tb_stats [start|pause|stop|filter] command to hmp.
This allows controlling the collection of statistics.
It is also possible to set the level of collection:
all, jit, or exec.
tb_stats filter allow to only collect statistics for the TB
in the last_search li
From: "Vanderson M. do Rosario"
We add some of the statistics collected in the TCGProfiler into the
TBStats, having the statistics not only for the whole emulation but
for each TB. Then, we removed these stats from TCGProfiler and
reconstruct the information for the "info jit" using the sum of al
From: Alex Bennée
We keep track of translations but can only do so up until the
translation cache is flushed. At that point we really have no idea if
we can re-create a translation because all the active tracking
information has been reset.
Signed-off-by: Alex Bennée
Reviewed-by: Richard Hender
From: "Vanderson M. do Rosario"
-d tb_stats[[,level=(+all+jit+exec+time)][,dump_limit=]]
"dump_limit" is used to limit the number of dumped TBStats in
linux-user mode.
[all+jit+exec+time] control the profilling level used
by the TBStats. Can be used as follow:
-d tb_stats
-d tb_stats,level=ji
From: "Vanderson M. do Rosario"
These commands allow the exploration of TBs generated by the TCG.
Understand which one hotter, with more guest/host instructions... and
examine their guest, host and IR code.
The goal of this command is to allow the dynamic exploration of TCG
behavior and code qua
From: Alex Bennée
Signed-off-by: Alex Bennée
---
meson.build | 2 --
meson_options.txt | 2 --
scripts/meson-buildoptions.sh | 3 ---
3 files changed, 7 deletions(-)
diff --git a/meson.build b/meson.build
index 29f8644d6d..27627199d7 100644
--- a/meson.build
+++ b
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
On 4/21/2023 9:24 PM, Fei Wu wrote:
> From: "Vanderson M. do Rosario"
>
> These commands allow the exploration of TBs generated by the TCG.
> Understand which one hotter, with more guest/host instructions... and
> examine their guest, host and IR code.
>
> The g
v12
---
* fix some CI issues, including make do-meson-check and build
* move part of patch 9 to 1
* fix inverse_sort_tbs
* use normal ++ instead of qatomic_inc for stats->executions.normal
* add how to for tb-stats
* append Fei Wu to Signed-off-by
Alex Bennée (5):
accel/tcg: move profi
a TB.
The execution count of the TB is stored in its respective
TBS.
All TBStatistics are created by default with the flags from
default_tbstats_flag.
Signed-off-by: Vanderson M. do Rosario
Message-Id: <20190829173437.5926-3-vanderson...@gmail.com>
[AJB: Fix author]
Signed-off-by: Alex Ben
From: Alex Bennée
This shouldn't live in the monitor code anyway. While we are at it
make it an uint64_t as we won't be dealing in negative numbers.
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
accel/tcg/monitor.c | 2 --
accel/tcg/tb-stats.c | 2 ++
include/qemu/timer.h
From: Alex Bennée
This starts the conversion of CONFIG_PROFILER data collection to under
the TBStats system. We introduce a new flag TB_JIT_TIME and start
tracking how much time is spent restoring execution state from a given
TB.
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
accel/tcg
do Rosario
Message-Id: <20190829173437.5926-6-vanderson...@gmail.com>
[AJB: fix authorship]
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
accel/tcg/monitor.c| 25 --
accel/tcg/tb-stats.c | 78 +
accel/tcg/tcg-accel-ops.c | 15 +++---
-
Acked-by: Dr. David Alan Gilbert
Signed-off-by: Vanderson M. do Rosario
Message-Id: <20190829173437.5926-10-vanderson...@gmail.com>
[AJB: fix authorship, dropped coverset]
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
accel/tcg/monitor.c | 54 ++
.
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
accel/tcg/tb-stats.c | 33 ---
accel/tcg/translate-all.c | 69 ++-
include/exec/tb-stats.h | 7
include/tcg/tcg.h | 14
tcg/tcg.c | 17
ts
-d tb_stats,level=jit+time
-d tb_stats,dump_limit=15
...
Signed-off-by: Vanderson M. do Rosario
Message-Id: <20190829173437.5926-7-vanderson...@gmail.com>
[AJB: fix authorship, reword title]
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
accel/tcg/tb-stats.c | 5 +
6-4-vanderson...@gmail.com>
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
accel/tcg/translate-all.c | 29 +++--
accel/tcg/translator.c| 3 +++
include/exec/tb-stats-flags.h | 1 +
include/exec/tb-stats.h | 23 +++
i
Henderson
Signed-off-by: Fei Wu
---
accel/tcg/tb-maint.c| 1 +
accel/tcg/tb-stats.c| 19 +++
include/exec/tb-stats.h | 8
3 files changed, 28 insertions(+)
diff --git a/accel/tcg/tb-maint.c b/accel/tcg/tb-maint.c
index 0980fca358..11ff0ddd90 100644
--- a/accel/tcg
Signed-off-by: Fei Wu
---
docs/tb-stats.txt | 116 ++
1 file changed, 116 insertions(+)
create mode 100644 docs/tb-stats.txt
diff --git a/docs/tb-stats.txt b/docs/tb-stats.txt
new file mode 100644
index 00..2f96ebb741
--- /dev/null
+++ b/docs
odeQuality#Overheads.
Reviewed-by: Alex Bennée
Signed-off-by: Vanderson M. do Rosario
Message-Id: <20190829173437.5926-5-vanderson...@gmail.com>
[AJB: fix authorship, use prof structure]
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
accel/tcg/tb-stats.c | 99 ++
o have all present and future qemu/tcg statistics and
meta-data stored in this new structure.
Reviewed-by: Alex Bennée
Signed-off-by: Vanderson M. do Rosario
Message-Id: <20190829173437.5926-2-vanderson...@gmail.com>
[AJB: fix git author, review comments]
Signed-off-by: Alex Bennée
Signe
000ec1cb (id:13)
Signed-off-by: Vanderson M. do Rosario
Message-Id: <20190829173437.5926-12-vanderson...@gmail.com>
[AJB: fix authorship, ad softmmu support]
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
accel/tcg/tb-stats.c | 21 +
include/e
...@gmail.com>
Message-Id: <20190829173437.5926-9-vanderson...@gmail.com>
[AJB: fix authorship]
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
accel/tcg/monitor.c | 45 ++
accel/tcg/tb-stats.c | 112 ++
hmp-commands.
From: Alex Bennée
Signed-off-by: Alex Bennée
Signed-off-by: Fei Wu
---
meson.build | 2 --
meson_options.txt | 2 --
scripts/meson-buildoptions.sh | 3 ---
3 files changed, 7 deletions(-)
diff --git a/meson.build b/meson.build
index 4dddccb890..4be21c9d57 100644
On 5/18/2023 9:57 PM, Fei Wu wrote:
> From: "Vanderson M. do Rosario"
>
> Adding tb_stats [start|pause|stop|filter] command to hmp.
> This allows controlling the collection of statistics.
> It is also possible to set the level of collection:
> all, jit, or exec.
&
On 5/18/2023 9:57 PM, Fei Wu wrote:
> From: "Vanderson M. do Rosario"
>
> To store statistics for each TB, we created a TBStatistics structure
> which is linked with the TBs. TBStatistics can stay alive after
> tb_flush and be relinked to a regenerated TB. So the statis
1 - 100 of 488 matches
Mail list logo
