[pve-devel] [PATCH] fix #2402: allow 1GB hugepages if 2MB is unavailable

[Stefan Reiter]

As reported in bug #2402, a system started with "default_hugepagesz=1G
hugepagesz=1G" does not have a /sys/kernel/mm/hugepages/hugepages-2048kB
directory.

To still allow 1GB hugepages, ignore the missing directory in hugepages_mount
(since it's not needed anyway), and correctly check if the requested hugepage
size is available in hugepages_size instead.

Signed-off-by: Stefan Reiter <s.rei...@proxmox.com>
---
 PVE/QemuServer/Memory.pm | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/PVE/QemuServer/Memory.pm b/PVE/QemuServer/Memory.pm
index b579955..f311dce 100644
--- a/PVE/QemuServer/Memory.pm
+++ b/PVE/QemuServer/Memory.pm
@@ -358,7 +358,7 @@ sub hugepages_mount {
    my $mountdata = PVE::ProcFSTools::parse_proc_mounts();
 
    foreach my $size (qw(2048 1048576)) {
-       return if (! -d "/sys/kernel/mm/hugepages/hugepages-${size}kB");
+       next if (! -d "/sys/kernel/mm/hugepages/hugepages-${size}kB");
 
        my $path = "/run/hugepages/kvm/${size}kB";
 
@@ -400,18 +400,20 @@ sub hugepages_size {
        #try to use 1GB if available && memory size is matching
        if (-d "/sys/kernel/mm/hugepages/hugepages-1048576kB" && ($size % 1024 
== 0)) {
            return 1024;
-       } else {
+       } elsif (-d "/sys/kernel/mm/hugepages/hugepages-2048kB") {
            return 2;
        }
 
+       die "your system doesn't support hugepages\n";
+
    } else {
 
        my $hugepagesize = $conf->{hugepages} * 1024 . "kB";
 
        if (! -d "/sys/kernel/mm/hugepages/hugepages-$hugepagesize") {
-               die "your system doesn't support hugepages of $hugepagesize";
+               die "your system doesn't support hugepages of $hugepagesize\n";
        }
-       die "Memory size $size is not a multiple of the requested hugepages 
size $hugepagesize" if ($size % $conf->{hugepages}) != 0;
+       die "Memory size $size is not a multiple of the requested hugepages 
size $hugepagesize\n" if ($size % $conf->{hugepages}) != 0;
        return $conf->{hugepages};
    }
 
-- 
2.20.1


_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel