• Wanpeng Li's avatar
    KVM: kvmclock: Fix vCPUs > 64 can't be online/hotpluged · d7eb79c6
    Wanpeng Li authored
    # lscpu
    Architecture:          x86_64
    CPU op-mode(s):        32-bit, 64-bit
    Byte Order:            Little Endian
    CPU(s):                88
    On-line CPU(s) list:   0-63
    Off-line CPU(s) list:  64-87
    
    # cat /proc/cmdline
    BOOT_IMAGE=/vmlinuz-5.10.0-rc3-tlinux2-0050+ root=/dev/mapper/cl-root ro
    rd.lvm.lv=cl/root rhgb quiet console=ttyS0 LANG=en_US .UTF-8 no-kvmclock-vsyscall
    
    # echo 1 > /sys/devices/system/cpu/cpu76/online
    -bash: echo: write error: Cannot allocate memory
    
    The per-cpu vsyscall pvclock data pointer assigns either an element of the
    static array hv_clock_boot (#vCPU <= 64) or dynamically allocated memory
    hvclock_mem (vCPU > 64), the dynamically memory will not be allocated if
    kvmclock vsyscall is disabled, this can result in cpu hotpluged fails in
    kvmclock_setup_percpu() which returns -ENOMEM. It's broken for no-vsyscall
    and sometimes you end up with vsyscall disabled if the host does something
    strange. This patch fixes it by allocating this dynamically memory
    unconditionally even if vsyscall is disabled.
    
    Fixes: 6a1cac56 ("x86/kvm: Use __bss_decrypted attribute in shared variables")
    Reported-by: default avatarZelin Deng <zelin.deng@linux.alibaba.com>
    Cc: Brijesh Singh <brijesh.singh@amd.com>
    Cc: stable@vger.kernel.org#v4.19-rc5+
    Signed-off-by: default avatarWanpeng Li <wanpengli@tencent.com>
    Message-Id: <1614130683-24137-1-git-send-email-wanpengli@tencent.com>
    Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
    d7eb79c6
kvmclock.c 9.44 KB