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

oem-ibm: Change in hb_predictive_mem_guard bios attribute #612

Open
wants to merge 1 commit into
base: 1060
Choose a base branch
from

Conversation

riyadixitagra
Copy link
Contributor

This commit enhances the displayName and helpText of hb_predictive_mem_guard and hb_predictive_mem_guard_current bios enum attribute.

This commit enhances the displayName and helpText of
hb_predictive_mem_guard and hb_predictive_mem_guard_current bios enum
attribute.

Signed-off-by: Riya Dixit <riyadixitagra@gmail.com>
Copy link
Contributor

@jaypadath jaypadath left a comment

Choose a reason for hiding this comment

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

Pls change oem-ibm: Change in hb_predictive_mem_guard bios attribute to "oem-ibm: Changes in memory deallocation bios attribute".
bios enum attribute --> bios enum attributes
Also add the Testing field

@@ -64,15 +64,15 @@
"attribute_name": "hb_predictive_mem_guard",
"possible_values": ["Enabled", "Disabled"],
"default_values": ["Enabled"],
"helpText": "Enable or Disable Predictive Guard for Memory Errors.",
"displayName": "Predictive Memory Guard"
"helpText": "The Hypervisor dynamically removes memory pages from memory that detected predictive failures. If this option is disabled, dynamic memory deallocation will not occur for the predictive memory failures.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Would it be better to write "The hypervisor dynamically removes the memory pages which were identified as predictive failures" ?

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.

3 participants