Skip to content

Commit

Permalink
test: hugepages: Use wait_for_up in EPT violation test
Browse files Browse the repository at this point in the history
We were doing a `wait_for_up` after snapshot restore (which is
nonsense), but not after boot. Move it to be after boot.

Signed-off-by: Patrick Roy <roypat@amazon.co.uk>
  • Loading branch information
roypat committed Sep 12, 2024
1 parent c873765 commit b4c4e18
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/integration_tests/performance/test_huge_pages.py
Original file line number Diff line number Diff line change
Expand Up @@ -174,6 +174,7 @@ def test_ept_violation_count(
vm.basic_config(huge_pages=huge_pages, mem_size_mib=256)
vm.add_net_iface()
vm.start()
vm.wait_for_up()

metrics.set_dimensions(
{
Expand Down Expand Up @@ -211,7 +212,6 @@ def test_ept_violation_count(

with ftrace_events("kvm:*"):
vm.restore_from_snapshot(snapshot, resume=True, uffd_path=SOCKET_PATH)
vm.wait_for_up()

# Verify if guest can run commands, and also wake up the fast page fault helper to trigger page faults.
vm.ssh.check_output(f"kill -s {signal.SIGUSR1} {pid}")
Expand Down

0 comments on commit b4c4e18

Please sign in to comment.