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

Several Residency Manager improvements/fixes from UE5/The Coalition #837

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

wallisc
Copy link
Contributor

@wallisc wallisc commented Jul 31, 2023

This adds several new functions to the residency manager as well as a bug fix:

New Functionality:

  1. The ability to query the amount of resident and evicted memory from the residency manager. The ability to view evicted memory has also been added to the sample.
  2. The ability to specify a custom local memory budget. In UE5, this is can be used with value 0 when the app is detected to be out of focus so that it can trigger extremely aggressive memory eviction. This is particularly important for when using the UE5 editor and you're launching the game as a separate process.

Bug Fix:

  1. Fixing the parameters for EnqueueMakeResident, according to the Epic change history this fix came from Microsoft originally so unclear why this isn't reflected in the github repo

This adds several new functions to the residency manager as well as a bug fix:

New Functionality:
 1.  The ability to query the amount of resident and evicted memory from the residency manager. The ability to view evicted memory has also been added to the sample.
2. The ability to specify a custom local memory budget. In UE5, this is can be used with value 0 when the app is detected to be out of focus so that it can trigger extremely aggressive memory eviction. This is particularly important for when using the UE5 editor and you're launching the game as a separate process.

Bug Fix:
1. Fixing the parameters for EnqueueMakeResident, according to the Epic change history this fix came from Microsoft originally so unclear why this isn't reflected in the github repo
Copy link
Member

@jenatali jenatali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Successfully merging this pull request may close these issues.

2 participants