[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 6/6] new script/analyse-tlb-flushes-simpletra
From: |
Pranith Kumar |
Subject: |
Re: [Qemu-devel] [PATCH v2 6/6] new script/analyse-tlb-flushes-simpletrace.py |
Date: |
Tue, 30 May 2017 19:33:23 -0400 |
Hi Alex,
Please find some comments and questions below:
On Wed, May 17, 2017 at 10:52 AM, Alex Bennée <address@hidden> wrote:
> This is a simple helper script to extract TLB flush stats from the a
> simpletrace file and plot the results.
>
> Signed-off-by: Alex Bennée <address@hidden>
>
> ---
> v2
> - re-factored for new trace events
> - added time and latency graphs
> ---
> scripts/analyse-tlb-flushes-simpletrace.py | 144
> +++++++++++++++++++++++++++++
> 1 file changed, 144 insertions(+)
> create mode 100755 scripts/analyse-tlb-flushes-simpletrace.py
>
> diff --git a/scripts/analyse-tlb-flushes-simpletrace.py
> b/scripts/analyse-tlb-flushes-simpletrace.py
> new file mode 100755
> index 0000000000..03fab8c86b
> --- /dev/null
> +++ b/scripts/analyse-tlb-flushes-simpletrace.py
<snip>
> +
> +def get_args():
> + "Grab options"
> + parser = argparse.ArgumentParser()
> + parser.add_argument("--output", "-o", type=str, help="Render plot to
> file")
> + parser.add_argument("--vcpus", type=int, help="Number of vCPUS")
It is not really clear what this argument is for. I guess you are
saying how many cpus the guest from which trace file was generated
had? What happens if we pass in less number of vcpus than used for
generation?
> + parser.add_argument("--graph", choices=['time', 'latency'],
> default='time')
What does latency here indicate? I tried this argument on a sample
trace file I generated, and it had three empty boxes.
> + parser.add_argument("events", type=str, help='trace file read from')
> + parser.add_argument("tracefile", type=str, help='trace file read from')
The help text for 'events' file here should be something like 'the
trace events file'.
Thanks,
--
Pranith
- [Qemu-devel] [PATCH v2 0/6] gdb updates and cputlb traces, Alex Bennée, 2017/05/17
- [Qemu-devel] [PATCH v2 2/6] scripts/qemu-gdb/timers.py: new helper to dump timer state, Alex Bennée, 2017/05/17
- [Qemu-devel] [PATCH v2 3/6] scripts/qemu-gdb/tcg: new helper to dump tcg state, Alex Bennée, 2017/05/17
- [Qemu-devel] [PATCH v2 4/6] cputlb: remove tlb_flush_count, Alex Bennée, 2017/05/17
- [Qemu-devel] [PATCH v2 1/6] scripts/replay-dump.py: replay log dumper, Alex Bennée, 2017/05/17
- [Qemu-devel] [PATCH v2 5/6] cputlb: add trace events, Alex Bennée, 2017/05/17
- [Qemu-devel] [PATCH v2 6/6] new script/analyse-tlb-flushes-simpletrace.py, Alex Bennée, 2017/05/17
- Re: [Qemu-devel] [PATCH v2 6/6] new script/analyse-tlb-flushes-simpletrace.py,
Pranith Kumar <=
- Re: [Qemu-devel] [PATCH v2 0/6] gdb updates and cputlb traces, Stefan Hajnoczi, 2017/05/18