[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 6/7] meson: document, why we don't generate trace events f
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v6 6/7] meson: document, why we don't generate trace events for tests/ and qga/ |
Date: |
Thu, 27 Jan 2022 07:50:12 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> writes:
> Making trace generation work for tests/ and qga/ would involve some
> Meson hackery to ensure we generate the trace-events files before
> trace-tool uses them. Since we don't actually support tracing there
> anyway, we bypass that problem.
>
> Let's add corresponding comments.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
> ---
> qga/meson.build | 7 +++++++
> tests/meson.build | 7 +++++++
> 2 files changed, 14 insertions(+)
>
> diff --git a/qga/meson.build b/qga/meson.build
> index cfb1fbc085..724d5a667b 100644
> --- a/qga/meson.build
> +++ b/qga/meson.build
> @@ -15,6 +15,13 @@ qga_qapi_outputs = [
> 'qga-qapi-visit.h',
> ]
>
> +# Problem: to generate trace events, we'd have to add the .trace-events
> +# file to qapi_trace_events like we do in qapi/meson.build. Since
> +# qapi_trace_events is used by trace/meson.build, we'd have to move
> +# subdir('qga') above subdir('trace') in the top-level meson.build.
> +# Can't, because it would break the dependency of qga on qemuutil (which
> +# depends on trace_ss). Not worth solving now; simply suppress trace
> +# event generation instead.
> qga_qapi_files = custom_target('QGA QAPI files',
> output: qga_qapi_outputs,
> input: 'qapi-schema.json',
> diff --git a/tests/meson.build b/tests/meson.build
> index 3f3882748a..af8f5b1746 100644
> --- a/tests/meson.build
> +++ b/tests/meson.build
> @@ -31,6 +31,13 @@ test_qapi_outputs = [
> 'test-qapi-visit.h',
> ]
>
> +# Problem: to generate trace events, we'd have to add the .trace-events
> +# file to qapi_trace_events like we do in qapi/meson.build. Since
> +# qapi_trace_events is used by trace/meson.build, we'd have to move
> +# subdir('qga') above subdir('trace') in the top-level meson.build.
Shouldn't this be subdir('tests')?
> +# Can't, because it would break the dependency of qga on qemuutil (which
> +# depends on trace_ss). Not worth solving now; simply suppress trace
> +# event generation instead.
> test_qapi_files = custom_target('Test QAPI files',
> output: test_qapi_outputs,
> input:
> files('qapi-schema/qapi-schema-test.json',
- [PATCH v6 1/7] qapi/gen: Add FOO.trace-events output module, (continued)
- [PATCH v6 1/7] qapi/gen: Add FOO.trace-events output module, Vladimir Sementsov-Ogievskiy, 2022/01/26
- [PATCH v6 2/7] qapi/commands: refactor error handling code, Vladimir Sementsov-Ogievskiy, 2022/01/26
- [PATCH v6 3/7] qapi/commands: Optionally generate trace for QMP commands, Vladimir Sementsov-Ogievskiy, 2022/01/26
- [PATCH v6 7/7] qapi: generate trace events by default, Vladimir Sementsov-Ogievskiy, 2022/01/26
- [PATCH v6 4/7] meson: generate trace events for qmp commands, Vladimir Sementsov-Ogievskiy, 2022/01/26
- [PATCH v6 5/7] docs/qapi-code-gen: update to cover trace events code generation, Vladimir Sementsov-Ogievskiy, 2022/01/26
- [PATCH v6 6/7] meson: document, why we don't generate trace events for tests/ and qga/, Vladimir Sementsov-Ogievskiy, 2022/01/26
- Re: [PATCH v6 0/7] trace qmp commands, Markus Armbruster, 2022/01/27
- Re: [PATCH v6 0/7] trace qmp commands, Stefan Hajnoczi, 2022/01/27