[libvirt] [PATCH] Support configuration of huge pages in guests

john cooper john.cooper at redhat.com
Thu Sep 3 05:17:56 UTC 2009


Daniel P. Berrange wrote:
> * configure.in: Add check for mntent.h
> * qemud/libvirtd_qemu.aug, qemud/test_libvirtd_qemu.aug, src/qemu.conf
>   Add 'hugetlbfs_mount' config parameter
> * src/qemu_conf.c, src/qemu_conf.h: Check for -mem-path flag in QEMU,
>   and pass it when hugepages are requested.
>   Load hugetlbfs_mount config parameter, search for mount if not given.
> * src/qemu_driver.c: Free hugetlbfs_mount/path parameter in driver shutdown.
>   Create directory for QEMU hugepage usage, chowning if required.
> * docs/formatdomain.html.in: Document memoryBacking/hugepages elements
> * docs/schemas/domain.rng: Add memoryBacking/hugepages elements to schema
> * src/util.c, src/util.h, src/libvirt_private.syms: Add virFileFindMountPoint
>   helper API
> * tests/qemuhelptest.c: Add -mem-path constants
> * tests/qemuxml2argvtest.c, tests/qemuxml2xmltest.c: Add tests for hugepage
>   handling
> * tests/qemuxml2argvdata/qemuxml2argv-hugepages.xml,
>   tests/qemuxml2argvdata/qemuxml2argv-hugepages.args: Data files for
>   hugepage tests
> ---
>  configure.in                                       |    2 +-
>  docs/formatdomain.html                             |    8 +++-
>  docs/formatdomain.html.in                          |    8 +++
>  docs/schemas/domain.rng                            |    9 ++++
>  qemud/libvirtd_qemu.aug                            |    1 +
>  qemud/test_libvirtd_qemu.aug                       |    4 ++
>  src/domain_conf.c                                  |   10 ++++-
>  src/domain_conf.h                                  |    1 +
>  src/libvirt_private.syms                           |    1 +
>  src/qemu.conf                                      |   13 +++++
>  src/qemu_conf.c                                    |   49 ++++++++++++++++++++
>  src/qemu_conf.h                                    |    3 +
>  src/qemu_driver.c                                  |   34 ++++++++++++++
>  src/util.c                                         |   37 ++++++++++++++-
>  src/util.h                                         |    4 ++
>  tests/qemuhelptest.c                               |    6 ++-
>  tests/qemuxml2argvdata/qemuxml2argv-hugepages.args |    1 +
>  tests/qemuxml2argvdata/qemuxml2argv-hugepages.xml  |   25 ++++++++++
>  tests/qemuxml2argvtest.c                           |    7 ++-
>  tests/qemuxml2xmltest.c                            |    1 +
>  20 files changed, 217 insertions(+), 7 deletions(-)
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-hugepages.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-hugepages.xml

I did a fetch on libvirt.git before reviewing and
it appears there is some code motion relative
to the version this patch was against.  Although
AFAICT nothing which appears to result in more than
patch bounce.

Looks good to me.  ACK.

-john

-- 
john.cooper at redhat.com




More information about the libvir-list mailing list