[PATCH 1/2] qemu: Separate out hugepages basedir making
Erik Skultety
eskultet at redhat.com
Fri Oct 14 14:38:11 UTC 2022
On Wed, Oct 12, 2022 at 12:37:58PM +0200, Michal Privoznik wrote:
> During its initialization, the QEMU driver iterates over
> hugetlbfs mount points, creating the driver specific path in each
> of them ($prefix/libvirt/qemu). This path is created with very
> wide mode (0777) because per-domain directories are then created
> under it.
>
> Separate this code into a function so that it can be re-used.
>
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
> src/qemu/qemu_conf.c | 27 +++++++++++++++++++++++++++
> src/qemu/qemu_conf.h | 3 +++
> src/qemu/qemu_driver.c | 17 +----------------
> 3 files changed, 31 insertions(+), 16 deletions(-)
>
> diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
> index 4f59e5fb07..c20fec26ba 100644
> --- a/src/qemu/qemu_conf.c
> +++ b/src/qemu/qemu_conf.c
> @@ -1576,3 +1576,30 @@ qemuGetMemoryBackingPath(virQEMUDriver *driver,
> *memPath = g_strdup_printf("%s/%s", domainPath, alias);
> return 0;
> }
> +
> +
> +int
> +qemuMkdirBaseHugepage(virQEMUDriver *driver,
> + virHugeTLBFS *hugepage)
Apart from virDomainHugepage we don't have another naming precedent Hugepage
functions, hence I'd like to propose qemuHugepageMakeBasedir instead.
Reviewed-by: Erik Skultety <eskultet at redhat.com>
More information about the libvir-list
mailing list