On Wed, Aug 27, 2025 at 3:12 AM Gustavo Romero <gustavo.rom...@linaro.org>
wrote:
>
> This commit fixes the asset precaching in the reverse_debugging test on
> aarch64.
>
> QemuBaseTest.main() precaches assets (kernel, rootfs, DT blobs, etc.)
> that are defined in variables with the ASSET_ prefix. This works because
> it ultimately calls Asset.precache_test(), which relies on introspection
> to locate these variables.
>
> If an asset variable is not named with the ASSET_ prefix, precache_test
> cannot find the asset and precaching silently fails. Hence, fix the
> asset precaching by fixing the asset variable name.
>
> Signed-off-by: Gustavo Romero <gustavo.rom...@linaro.org>
> ---

Reviewed-by: Manos Pitsidianakis <manos.pitsidiana...@linaro.org>

>  tests/functional/test_aarch64_reverse_debug.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/functional/test_aarch64_reverse_debug.py
b/tests/functional/test_aarch64_reverse_debug.py
> index 58d4532835..8bc91ccfde 100755
> --- a/tests/functional/test_aarch64_reverse_debug.py
> +++ b/tests/functional/test_aarch64_reverse_debug.py
> @@ -21,7 +21,7 @@ class ReverseDebugging_AArch64(ReverseDebugging):
>
>      REG_PC = 32
>
> -    KERNEL_ASSET = Asset(
> +    ASSET_KERNEL = Asset(
>          ('https://archives.fedoraproject.org/pub/archive/fedora/linux/'
>           'releases/29/Everything/aarch64/os/images/pxeboot/vmlinuz'),
>
 '7e1430b81c26bdd0da025eeb8fbd77b5dc961da4364af26e771bd39f379cbbf7')
> @@ -30,7 +30,7 @@ class ReverseDebugging_AArch64(ReverseDebugging):
>      def test_aarch64_virt(self):
>          self.set_machine('virt')
>          self.cpu = 'cortex-a53'
> -        kernel_path = self.KERNEL_ASSET.fetch()
> +        kernel_path = self.ASSET_KERNEL.fetch()
>          self.reverse_debugging(args=('-kernel', kernel_path))
>
>
> --
> 2.34.1
>
>

Reply via email to