lkml.org 
[lkml]   [2022]   [Oct]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 016/210] nvme: Fix IOC_PR_CLEAR and IOC_PR_RELEASE ioctls for nvme devices
    Date
    From: Michael Kelley <mikelley@microsoft.com>

    [ Upstream commit c292a337d0e45a292c301e3cd51c35aa0ae91e95 ]

    The IOC_PR_CLEAR and IOC_PR_RELEASE ioctls are
    non-functional on NVMe devices because the nvme_pr_clear()
    and nvme_pr_release() functions set the IEKEY field incorrectly.
    The IEKEY field should be set only when the key is zero (i.e,
    not specified). The current code does it backwards.

    Furthermore, the NVMe spec describes the persistent
    reservation "clear" function as an option on the reservation
    release command. The current implementation of nvme_pr_clear()
    erroneously uses the reservation register command.

    Fix these errors. Note that NVMe version 1.3 and later specify
    that setting the IEKEY field will return an error of Invalid
    Field in Command. The fix will set IEKEY when the key is zero,
    which is appropriate as these ioctls consider a zero key to
    be "unspecified", and the intention of the spec change is
    to require a valid key.

    Tested on a version 1.4 PCI NVMe device in an Azure VM.

    Fixes: 1673f1f08c88 ("nvme: move block_device_operations and ns/ctrl freeing to common code")
    Fixes: 1d277a637a71 ("NVMe: Add persistent reservation ops")
    Signed-off-by: Michael Kelley <mikelley@microsoft.com>
    Signed-off-by: Christoph Hellwig <hch@lst.de>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/nvme/host/core.c | 6 +++---
    1 file changed, 3 insertions(+), 3 deletions(-)

    diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c
    index 487226531a7d..8e136867180a 100644
    --- a/drivers/nvme/host/core.c
    +++ b/drivers/nvme/host/core.c
    @@ -1351,14 +1351,14 @@ static int nvme_pr_preempt(struct block_device *bdev, u64 old, u64 new,

    static int nvme_pr_clear(struct block_device *bdev, u64 key)
    {
    - u32 cdw10 = 1 | (key ? 1 << 3 : 0);
    + u32 cdw10 = 1 | (key ? 0 : 1 << 3);

    - return nvme_pr_command(bdev, cdw10, key, 0, nvme_cmd_resv_register);
    + return nvme_pr_command(bdev, cdw10, key, 0, nvme_cmd_resv_release);
    }

    static int nvme_pr_release(struct block_device *bdev, u64 key, enum pr_type type)
    {
    - u32 cdw10 = nvme_pr_type(type) << 8 | (key ? 1 << 3 : 0);
    + u32 cdw10 = nvme_pr_type(type) << 8 | (key ? 0 : 1 << 3);

    return nvme_pr_command(bdev, cdw10, key, 0, nvme_cmd_resv_release);
    }
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-10-24 13:59    [W:4.873 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site