Skip to content

ch: disable LISA hugepage setup in perf-stable mode#4352

Merged
vyadavmsft merged 1 commit intomainfrom
vyadav_march_fix4
Mar 19, 2026
Merged

ch: disable LISA hugepage setup in perf-stable mode#4352
vyadavmsft merged 1 commit intomainfrom
vyadav_march_fix4

Conversation

@vyadavmsft
Copy link
Copy Markdown
Collaborator

Disable LISA-side hugepage configuration for Cloud Hypervisor perf runs and leave hugepage setup to the Cloud Hypervisor harness.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Disables LISA-side hugepage reservation during Cloud Hypervisor perf-stable runs, delegating hugepage setup to the Cloud Hypervisor harness to avoid duplicated/competing host configuration.

Changes:

  • Removed hugepage reservation (1GB with 2MB fallback) from perf-stable host setup in CloudHypervisorTests._setup_host_perf_policies().
  • Updated the perf-stable policy description to no longer list hugepage reservation as a LISA-managed policy.

Comment thread lisa/microsoft/testsuites/cloud_hypervisor/ch_tests_tool.py
Copy link
Copy Markdown
Collaborator

@anirudhrb anirudhrb left a comment

Choose a reason for hiding this comment

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

Just curious, what's the reason for disabling? Was it conflicting with some Cloud Hypervisor tests or was it not giving us much gains?

@vyadavmsft
Copy link
Copy Markdown
Collaborator Author

@anirudhrb it was not giving gains , CH test were setting hugepage and LISA was also setting. so its better CH manage hugepage and we dont intefer .

@vyadavmsft vyadavmsft merged commit f5162fe into main Mar 19, 2026
62 checks passed
@vyadavmsft vyadavmsft deleted the vyadav_march_fix4 branch March 19, 2026 22:51
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