Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TC 903 : ATS Functionality Check #413

Open
f4bjh opened this issue Dec 10, 2024 · 1 comment
Open

TC 903 : ATS Functionality Check #413

f4bjh opened this issue Dec 10, 2024 · 1 comment

Comments

@f4bjh
Copy link

f4bjh commented Dec 10, 2024

Hi,

For TC 903 of SBSA test suite, boundvariable is used in ALIGN_MEMORY : https://github.com/ARM-software/bsa-acs/blob/main/val/sbsa/src/sbsa_acs_iovirt.c#L206

But this variable has antagonist declaration :

extern uint32_t bound;

uint64_t bound = 0x01;

static uint64_t bound = 0x01;

Shouldn't it be uint64_t bound everywhere ?

Note that I did not test TARGET_BM_BOOT neabled, but it should not have impact here.

Thanks in advance for your help,
Best regards
Fabrice DECROP LONGET

@SrikarJosyula
Copy link
Contributor

Hi @f4bjh,
Thank you for pointing out the issue. We have now isolated the memory access alignment only to Baremetal environment.
Can you please pick the binary with updated changes and try it out from your end?
Binary at - Bsa_ACPI_target.efi

Thanks,
ACS Team.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants