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.

Sign up for GitHub

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

Jump to bottom

[Bug]: 2405 MemmapAndMatFunctionalTestApp Fails on Q35 Release ClangPDB #1017

Closed
1 task done
os-d opened this issue Aug 9, 2024 · 0 comments · Fixed by #1018
Closed
1 task done

[Bug]: 2405 MemmapAndMatFunctionalTestApp Fails on Q35 Release ClangPDB #1017

os-d opened this issue Aug 9, 2024 · 0 comments · Fixed by #1018
Assignees
@kuqin12
Labels
state:needs-triage Needs to triaged to determine next steps type:bug Something isn't working urgency:low Little to no impact

Comments

@os-d
Copy link
Contributor

os-d commented Aug 9, 2024

Is there an existing issue for this?

  • I have searched existing issues

Current Behavior

Currently all toolchains except release clangpdb have success on 2405 on the MemmapAndMatFunctionalTestapp. Release clangpdb the test succeeds if it is the only test run, but if other tests are fun, it fails to produce an XML.

Expected Behavior

The test should pass.

Steps To Reproduce

Run all tests on release clangpdb on 2405 Q35.

Build Environment

- OS(s): Windows 11
- Tool Chain(s): CLANGPDB
- Targets Impacted: RELEASE

Version Information

release/20245 based repo

Urgency

Low

Are you going to fix this?

Someone else needs to fix it

Do you need maintainer feedback?

No maintainer feedback needed

Anything else?

No response
@os-d os-d added type:bug Something isn't working state:needs-triage Needs to triaged to determine next steps labels Aug 9, 2024
@github-actions github-actions bot added state:needs-owner Needs an issue owner to be assigned urgency:low Little to no impact and removed state:needs-owner Needs an issue owner to be assigned labels Aug 9, 2024
@kuqin12 kuqin12 mentioned this issue Aug 9, 2024
Merged
5 tasks
kuqin12 added a commit to microsoft/mu_plus that referenced this issue Aug 11, 2024
@kuqin12
Updated logic to allocate more data once it needs to query memory map (…
cf4b663 
…#547)

## Description

This change updates the logic to allocate 1KB extra data for the
subsequent GetMemoryMap function call to avoid once and done failure in
the test app.

Fixes microsoft/mu_tiano_platforms#1017.

- [x] Impacts functionality?
- [ ] Impacts security?
- [ ] Breaking change?
- [x] Includes tests?
- [ ] Includes documentation?

## How This Was Tested

This was tested on QEMU Q35 and verified fixed the original issue.

## Integration Instructions

N/A
@kuqin12 kuqin12 mentioned this issue Aug 11, 2024
Merged
5 tasks
kuqin12 added a commit that referenced this issue Aug 12, 2024
@kuqin12
Add back memory map test app (#1018)
5ce7af5 
## Description

This change re-enables the memory map test app fix by picking up the
latest MU_PLUS and MU_SILICON_ARM.

Resolves #1017.

For details on how to complete to complete these options and their
meaning refer to
[CONTRIBUTING.md](https://github.com/microsoft/mu/blob/HEAD/CONTRIBUTING.md).

- [x] Impacts functionality?
- [ ] Impacts security?
- [ ] Breaking change?
- [x] Includes tests?
- [ ] Includes documentation?

## How This Was Tested

This change is tested on QEMU Q35.

## Integration Instructions

N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kuqin12 kuqin12

Labels
state:needs-triage Needs to triaged to determine next steps type:bug Something isn't working urgency:low Little to no impact
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add back memory map test app kuqin12/mu_tiano_platforms
2 participants
@os-d @kuqin12