Hi Sughosh,
On Sat, 5 Aug 2023 at 05:36, Sughosh Ganu <sughosh.g...@linaro.org> wrote:
>
> When running the trace test on the sandbox platform, the current size
> of 16MiB is no longer large enough for capturing the entire trace
> history, and results in truncation. Use a size of 32MiB for the trace
> buffer on the sandbox platform while running the trace test.
>
> Signed-off-by: Sughosh Ganu <sughosh.g...@linaro.org>
> ---
> Changes since V6:
> * New patch for fixing CI trace test failure.
>
> .azure-pipelines.yml | 2 +-
> .gitlab-ci.yml | 2 +-
> test/py/tests/test_trace.py | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
Reviewed-by: Simon Glass <s...@chromium.org>
Do you know what has made it jump up?
> diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml
> index 2678e5ae05..2ef4db5318 100644
> --- a/.azure-pipelines.yml
> +++ b/.azure-pipelines.yml
> @@ -275,7 +275,7 @@ stages:
> TEST_PY_BD: "sandbox"
> BUILD_ENV: "FTRACE=1 NO_LTO=1"
> TEST_PY_TEST_SPEC: "trace"
> - OVERRIDE: "-a CONFIG_TRACE=y -a CONFIG_TRACE_EARLY=y -a
> CONFIG_TRACE_EARLY_SIZE=0x01000000"
> + OVERRIDE: "-a CONFIG_TRACE=y -a CONFIG_TRACE_EARLY=y -a
> CONFIG_TRACE_EARLY_SIZE=0x01000000 -a CONFIG_TRACE_BUFFER_SIZE=0x02000000"
> coreboot:
> TEST_PY_BD: "coreboot"
> TEST_PY_ID: "--id qemu"
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 8010afae95..3e41299658 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -315,7 +315,7 @@ sandbox trace_test.py:
> TEST_PY_BD: "sandbox"
> BUILD_ENV: "FTRACE=1 NO_LTO=1"
> TEST_PY_TEST_SPEC: "trace"
> - OVERRIDE: "-a CONFIG_TRACE=y -a CONFIG_TRACE_EARLY=y -a
> CONFIG_TRACE_EARLY_SIZE=0x01000000"
> + OVERRIDE: "-a CONFIG_TRACE=y -a CONFIG_TRACE_EARLY=y -a
> CONFIG_TRACE_EARLY_SIZE=0x01000000 -a CONFIG_TRACE_BUFFER_SIZE=0x02000000"
> <<: *buildman_and_testpy_dfn
>
> evb-ast2500 test.py:
> diff --git a/test/py/tests/test_trace.py b/test/py/tests/test_trace.py
> index ac3e95925e..ad2250920d 100644
> --- a/test/py/tests/test_trace.py
> +++ b/test/py/tests/test_trace.py
> @@ -61,7 +61,7 @@ def collect_trace(cons):
>
> # Read out the trace data
> addr = 0x02000000
> - size = 0x01000000
> + size = 0x02000000
> out = cons.run_command(f'trace calls {addr:x} {size:x}')
> print(out)
> fname = os.path.join(TMPDIR, 'trace')
> --
> 2.34.1
>
Regards,
Simon
