ARM64: CPU Hotplug: can't enable more cpus than maxcpus value (kernel 4.5)

[Vadim Lomovtsev]

Hi all,

While working with system which has 48 CPUs per one socket it was found that it is not possible to enable (turn on) more CPUs than 'maxcpus' value. The kernel booted with maxcpus=4 argument:

    [root at localhost ~]# cat /proc/cmdline
    BOOT_IMAGE=/boot/Image-4.5-rc5 root=UUID=9cece803-ce48-4b84-ae29-1091591c9edc ro crashkernel=2048M at 0M console=ttyAMA0,115200n8 LANG=en_US.UTF-8 earlycon=pl011,0x87e024000000 debug maxcpus=4

Check the overall number of CPUs and online CPUs:

    [root at localhost ~]# cat /sys/devices/system/cpu/possible
    0-47
    [root at localhost ~]# cat /sys/devices/system/cpu/online
    0-3

Now trying to turn on cpu which is out of scope of initial maxcpus value:
    [root at localhost ~]# echo 1 >  /sys/devices/system/cpu/cpu4/online
    -bash: echo: write error: Invalid argument

Which happens because of cpu_present_mask is set to 0x3 and prevents running more than 'maxcpus' cpus.
    [root at localhost ~]# cat /sys/devices/system/cpu/present
        0-3

For cpu0-3 it works fine:
    [root at localhost ~]# echo 0 >  /sys/devices/system/cpu/cpu3/online
    [root at localhost ~]# echo 1 >  /sys/devices/system/cpu/cpu3/online

At the same time at the Documentation/cpu-hotplug.txt it is described that:

>    maxcpus=n    Restrict boot time cpus to n. Say if you have 4 cpus, using
>                 maxcpus=2 will only boot 2. You can choose to bring the
>                 other cpus later online, read FAQ's for more info.

and

>    cpu_present_mask: Bitmap of CPUs currently present in the system. Not all
>    of them may be online. When physical hotplug is processed by the relevant
>    subsystem (e.g ACPI) can change and new bit either be added or removed
>    from the map depending on the event is hot-add/hot-remove. There are currently
>    no locking rules as of now. Typical usage is to init topology during boot,
>    at which time hotplug is disabled.

So after system start, according to docs, it should be possible to run more CPUs than 'maxcpus' value and cpu_present_mask should contain mask for all CPUs which currently present at the system.
However for arm64 it is implemented that cpu_present_mask is explicetely set accordingly to 'maxcpus' value. Is it design intent ?

Vadim