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

[Feature]: DXE Should Only Allocate EFI_MEMORY_SP Ranges As Last Resort #885

Closed
os-d opened this issue Jun 3, 2024 · 1 comment
Closed
Labels
state:needs-triage Needs to triaged to determine next steps type:feature-request A new feature proposal urgency:medium Important with a moderate impact

Comments

@os-d
Copy link
Contributor

os-d commented Jun 3, 2024

Feature Overview

EFI_MEMORY_SP indicates special memory that may have performance issues or be unreliable or have other special characteristics. Today, DXE will allocate this memory as part of its normal pool.

Solution Overview

DXE should treat EFI_MEMORY_SP like untested memory, where it is not used unless we run out of standard system memory, in which case it gets promoted to be allocatable.

Alternatives Considered

No response

Urgency

Medium

Are you going to implement the feature request?

I will implement the feature

Do you need maintainer feedback?

No maintainer feedback needed

Anything else?

No response

@os-d os-d added state:needs-triage Needs to triaged to determine next steps type:feature-request A new feature proposal labels Jun 3, 2024
@github-actions github-actions bot added the urgency:medium Important with a moderate impact label Jun 3, 2024
@os-d
Copy link
Contributor Author

os-d commented Jun 24, 2024

Closing for now as edk2 has implemented the simpler case of UEFI never allocating SP memory. If this needs to be revisted later, it will be.

@os-d os-d closed this as completed Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state:needs-triage Needs to triaged to determine next steps type:feature-request A new feature proposal urgency:medium Important with a moderate impact
Projects
None yet
Development

No branches or pull requests

1 participant