• Alison Schofield's avatar
    cxl/pmem: Remove CXL SET_PARTITION_INFO from exclusive_cmds list · 6aa657f4
    Alison Schofield authored
    With SET_PARTITION_INFO on the exclusive_cmds list for the CXL_PMEM
    driver, userspace cannot execute a set-partition command without
    first unbinding the pmem driver from the device.
    
    When userspace requests a partition change to take effect on the
    next reboot this unbind requirement is unnecessarily restrictive.
    The driver does not need to enforce an unbind because partitions
    will not change until the next reboot. Of course, userspace still
    needs to be aware that changing the size of persistent capacity
    on the next reboot will result in the loss of data stored. That
    can happen regardless of whether it is presently bound at the time
    of issuing the set-partition command.
    
    When userspace requests a partition change to take effect immediately,
    restrictions are needed. The CXL_MEM driver currently blocks the usage
    of immediate mode, making the presence of SET_PARTITION_INFO, in this
    exclusive commands list, redundant.
    
    In the future, when the CXL_MEM driver adds support for immediate
    changes to device partitions it will ensure that the partition change
    will not affect any active decode. That means the work will not fall
    right back here, onto the CXL_PMEM driver.
    Signed-off-by: default avatarAlison Schofield <alison.schofield@intel.com>
    Link: https://lore.kernel.org/r/accc6abc878f0662093b81490a1a052f2ff6f06e.1648687552.git.alison.schofield@intel.comSigned-off-by: default avatarDan Williams <dan.j.williams@intel.com>
    6aa657f4
pmem.c 8.96 KB