Skip to content

Commit

Permalink
memory: add case for hugepage mount path
Browse files Browse the repository at this point in the history
  VIRT-297104: Huge page mount path
Signed-off-by: nanli <[email protected]>
  • Loading branch information
nanli1 committed Oct 8, 2023
1 parent 7c1c7b6 commit c45bab7
Show file tree
Hide file tree
Showing 2 changed files with 225 additions and 0 deletions.
43 changes: 43 additions & 0 deletions libvirt/tests/cfg/memory/memory_backing/hugepage_mount_path.cfg
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
- memory.backing.mount_path:
type = hugepage_mount_path
mem_value = 2097152
set_pagesize_1 = "1048576"
set_pagenum_1 = "2"
set_pagesize_2 = "2048"
set_pagenum_2 = "3072"
security_context = "drwxr-xr-x. 3 root root system_u:object_r:hugetlbfs_t:s0"
log_path = "/var/log/libvirt/qemu/%s.log"
guest_log_error1 = "hugepage is not mounted"
guest_log_error2 = "unable to create backing store for hugepages: Permission denied"
page_unit = "KiB"
page_size = "2048"
numa_size = 1048576
mem_unit = "KiB"
mem_value = 2097152
current_mem = 2097152
max_mem = 15242880
memory_backing_dict = "'mb': {'hugepages': {'pages': [{'unit': '${page_unit}', 'size': '${page_size}'}]}}"
numa_cpu = {'numa_cell': [{'id': '0', 'cpus': '0-1', 'memory': '${numa_size}', 'unit': 'KiB'}, {'id': '1', 'cpus': '2-3', 'memory': '${numa_size}', 'unit': 'KiB'}]}
vm_attrs = {${memory_backing_dict},"cpu":${numa_cpu},'vcpu': 4,'max_mem_rt_slots': 16,'memory_unit':"${mem_unit}",'memory':${mem_value},'current_mem':${current_mem},'current_mem_unit':'KiB','max_mem_rt': ${max_mem}, 'max_mem_rt_unit': "KiB"}
start_vm = "no"
consume_value = 204800
variants case:
- default:
path = "/dev/hugepages"
vm_hugepage_mountpoint = "/dev/hugepages1G"
mount_pagesize = "1G"
expect_state = "active"
check_qemu = ['"mem-path":"${path}/libvirt/qemu/','"prealloc":true']
attach_dict = {'mem_model': 'dimm', 'source': {"pagesize":1048576, "pagesize_unit":"KiB"},'target': {'size': 1048576, 'size_unit': 'KiB','node':0}}
check_security_cmd = "ls -l -Z ${path}/libvirt/qemu/ -d"
- disabled:
hugetlbfs_mount = '""'
expect_state = "failed"
- customized:
default_path = "/dev/hugepages/"
path = "/dev/hugetest"
vm_hugepage_mountpoint = "${path}"
hugetlbfs_mount = "${path}"
expect_state = "active"
check_security_cmd = "ls -l -Z ${path}/libvirt/qemu -d"
check_qemu = ['"mem-path":"${path}/libvirt/qemu/','"prealloc":true']
182 changes: 182 additions & 0 deletions libvirt/tests/src/memory/memory_backing/hugepage_mount_path.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,182 @@
# ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#
# Copyright Redhat
#
# SPDX-License-Identifier: GPL-2.0

# Author: Nan Li <[email protected]>
#
# ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
import re

from avocado.utils import process

from virttest import utils_libvirtd
from virttest import utils_misc
from virttest import virsh
from virttest import test_setup

from virttest.libvirt_xml import vm_xml
from virttest.libvirt_xml.devices import memory
from virttest.utils_test import libvirt
from virttest.utils_libvirt import libvirt_memory


def run(test, params, env):
"""
Verify the mount path config takes effect
Verify huge page path mount doesn't impact running guest vm
"""
def setup_test():
"""
Set hugepage on host and prepare guest
"""
test.log.info("TEST_SETUP: Set hugepage on host and prepare guest")

hp_cfg = test_setup.HugePageConfig(params)
if case == "customized":
process.run("umount %s" % default_path, ignore_status=True)
hp_cfg.mount_hugepage_fs()
hp_cfg.set_kernel_hugepages(set_pagesize_1, set_pagenum_1)
hp_cfg.set_kernel_hugepages(set_pagesize_2, set_pagenum_2)

vmxml = vm_xml.VMXML.new_from_inactive_dumpxml(vm_name)
vmxml.setup_attrs(**vm_attrs)
virsh.define(vmxml.xml, debug=True, ignore_status=False)
test.log.debug("Define xml by :\n%s ", vmxml)

def run_test():
"""
Define guest, start guest and check mem.
"""
test.log.info("TEST_STEP1: Modify the huge page mount path config ")
if case != "default":
qemu_obj = libvirt.customize_libvirt_config(
{"hugetlbfs_mount": '"%s"' % hugetlbfs_mount}, "qemu",
restart_libvirt=False)
params.update({"qemu_conf_obj": qemu_obj})
process.run("cat /etc/libvirt/qemu.conf", shell=True)

test.log.info("TEST_STEP2: Restart libvirt service")
libvirtd = utils_libvirtd.Libvirtd()
process.run("systemctl restart %s" % libvirtd.service_name, shell=True)

def _compare_state():
state = process.run("systemctl is-active %s" % libvirtd.service_name,
shell=True, ignore_status=True).stdout_text.strip()
return state == expect_state

if not utils_misc.wait_for(lambda: _compare_state(), timeout=20):
test.fail("Expect libvirt service state: '%s'" % expect_state)
else:
test.log.debug("Get expected '%s' libvirt service", expect_state)
if case == "disabled":
return

test.log.info("TEST_STEP3: Start guest")
vm.start()
vmxml = vm_xml.VMXML.new_from_dumpxml(vm_name)
test.log.debug("Current vmxml is:\n %s", vmxml)

test.log.info("TEST_STEP4: Check the qemu cmd line")
libvirt.check_qemu_cmd_line(check_qemu)

test.log.info("TEST_STEP5: Check security context")
cmd_result = process.run(check_security_cmd, ignore_status=True,
shell=True).stdout_text
if not re.search(security_context, cmd_result):
test.fail("Expect get %s in %s" % (security_context, cmd_result))

test.log.info("TEST_STEP6: Check the guest log")
libvirt.check_logfile(err_1, log_path % vm_name, str_in_log=False)
libvirt.check_logfile(err_2, log_path % vm_name, str_in_log=False)

if case == "customized":
return

test.log.info("TEST_STEP7: Stop services")
utils_libvirtd.Libvirtd().stop()

test.log.info("TEST_STEP8: Mount hugepage path")
hp_cfg = test_setup.HugePageConfig(params)
hp_cfg.hugepage_size = mount_pagesize
hp_cfg.mount_hugepage_fs()

test.log.info("TEST_STEP9: Do virsh list to check guest status")
output = virsh.dom_list("--all", debug=True).stdout.strip()
result = re.search(r".*%s.*%s.*" % (vm_name, "running"), output)
if not result:
test.fail("VM '{}' with the state running is not found".format(vm_name))

test.log.info("TEST_STEP10: Attach memory device")
mem_dev = memory.Memory()
mem_dev.setup_attrs(**attach_dict)
virsh.attach_device(vm.name, mem_dev.xml, debug=True,
ignore_status=False)

test.log.info("TEST_STEP11: Check mem")
session = vm.wait_for_login()
libvirt_memory.consume_vm_freememory(session, consume_value)
session.close()

test.log.info("TEST_STEP12: Destroy the guest")
virsh.destroy(vm_name, ignore_status=False)

test.log.info("TEST_STEP13: Restart service")
process.run("systemctl restart %s" % libvirtd.service_name, shell=True)
if not utils_misc.wait_for(lambda: _compare_state(), timeout=20):
test.fail("Expect libvirt service state: '%s'" % expect_state)
else:
test.log.debug("Get expected '%s' libvirt service", expect_state)

test.log.info("TEST_STEP14: Start the guest")
virsh.start(vm_name, ignore_status=False)

def teardown_test():
"""
Clean data.
"""
test.log.info("TEST_TEARDOWN: Clean up env.")
libvirt.customize_libvirt_config(
None, "qemu", config_object=params.get("qemu_conf_obj"),
is_recover=True)
bkxml.sync()

hp_cfg = test_setup.HugePageConfig(params)
hp_cfg.cleanup()
if default_path:
hp_cfg.hugepage_path = default_path
hp_cfg.mount_hugepage_fs()

vm_name = params.get("main_vm")
vm = env.get_vm(vm_name)
vmxml = vm_xml.VMXML.new_from_inactive_dumpxml(vm_name)
bkxml = vmxml.copy()

log_path = params.get("log_path")
err_1 = params.get("guest_log_error1")
err_2 = params.get("guest_log_error2")
hugetlbfs_mount = params.get("hugetlbfs_mount")
default_path = params.get("default_path")
consume_value = int(params.get("consume_value"))

mount_pagesize = params.get("mount_pagesize", '{}')
set_pagesize_1 = params.get("set_pagesize_1")
set_pagenum_1 = params.get("set_pagenum_1")
set_pagesize_2 = params.get("set_pagesize_2")
set_pagenum_2 = params.get("set_pagenum_2")

case = params.get("case")
check_security_cmd = params.get("check_security_cmd")
check_qemu = eval(params.get("check_qemu", '{}'))
security_context = params.get("security_context")
expect_state = params.get("expect_state")
attach_dict = eval(params.get("attach_dict", "{}"))
vm_attrs = eval(params.get("vm_attrs", "{}"))

try:
setup_test()
run_test()

finally:
teardown_test()

0 comments on commit c45bab7

Please sign in to comment.