linux-stable.git/drivers/nvme, branch v5.19.2

nvme: catch -ENODEV from nvme_revalidate_zones again

2022-08-17T13:15:42+00:00

[ Upstream commit e06b425bc835ead08b9fd935bf5e47eef473e7a0 ]

nvme_revalidate_zones can also return -ENODEV if e.g. zone sizes aren't
constant or not a power of two.  In that case we should jump to marking
the gendisk hidden and only support pass through.

Fixes: 602e57c9799c ("nvme: also mark passthrough-only namespaces ready in nvme_update_ns_info")
Reported-by: Joel Granados 
Signed-off-by: Christoph Hellwig 
Reviewed-by: Joel Granados 
Reviewed-by: Sagi Grimberg 
Signed-off-by: Jens Axboe 
Signed-off-by: Sasha Levin

nvme: define compat_ioctl again to unbreak 32-bit userspace.

2022-08-17T13:15:42+00:00

[ Upstream commit a25d4261582cf00dad884c194d21084836663d3d ]

Commit 89b3d6e60550 ("nvme: simplify the compat ioctl handling") removed
the initialization of compat_ioctl from the nvme block_device_operations
structures.

Presumably the expectation was that 32-bit ioctls would be directed
through the regular handler but this is not the case: failing to assign
.compat_ioctl actually means that the compat case is disabled entirely,
and any attempt to submit nvme ioctls from 32-bit userspace fails
outright with -ENOTTY.

For example:

  % smartctl -x /dev/nvme0n1
  [...]
  Read NVMe Identify Controller failed: NVME_IOCTL_ADMIN_CMD: Inappropriate ioctl for device

The blkdev_compat_ptr_ioctl helper can be used to direct compat calls
through the main ioctl handler and makes things work again.

Fixes: 89b3d6e60550 ("nvme: simplify the compat ioctl handling")
Signed-off-by: Nick Bowler 
Reviewed-by: Guixin Liu 
Signed-off-by: Christoph Hellwig 
Signed-off-by: Jens Axboe 
Signed-off-by: Sasha Levin

nvme: use command_id instead of req->tag in trace_nvme_complete_rq()

2022-08-17T13:15:42+00:00

[ Upstream commit 679c54f2de672b7d79d02f8c4ad483ff6dd8ce2e ]

Use command_id instead of req->tag in trace_nvme_complete_rq(),
because of commit e7006de6c238 ("nvme: code command_id with a genctr
for use authentication after release"), cmd->common.command_id is set to
((genctl & 0xf)< 12 | req->tag), no longer req->tag, which makes cid in
trace_nvme_complete_rq and trace_nvme_setup_cmd are not the same.

Fixes: e7006de6c238 ("nvme: code command_id with a genctr for use authentication after release")
Signed-off-by: Bean Huo 
Signed-off-by: Christoph Hellwig 
Signed-off-by: Jens Axboe 
Signed-off-by: Sasha Levin

nvme-pci: Crucial P2 has bogus namespace ids

2022-07-25T05:34:07+00:00

This adds a quirk for the Crucial P2.

Signed-off-by: Tobias Gruetzmacher 
Signed-off-by: Christoph Hellwig

nvme: fix block device naming collision

2022-07-14T14:35:25+00:00

The issue exists when multipath is enabled and the namespace is
shared, but all the other controller checks at nvme_is_unique_nsid()
are false. The reason for this issue is that nvme_is_unique_nsid()
returns false when is called from nvme_mpath_alloc_disk() due to an
uninitialized value of head->shared. The patch fixes it by setting
head->shared before nvme_mpath_alloc_disk() is called.

Fixes: 5974ea7ce0f9 ("nvme: allow duplicate NSIDs for private namespaces")
Signed-off-by: Israel Rukshin 
Reviewed-by: Keith Busch 
Reviewed-by: Max Gurtovoy 
Signed-off-by: Christoph Hellwig

nvme-pci: fix freeze accounting for error handling

2022-07-14T14:35:25+00:00

A reset on a live device experiencing a link error still needs to have
the queue freeze state started for the subsequent reinitialization. Skip
only the register read if the device is not present instead of bypassing
the freeze checks.

Fixes: b98235d3a471e ("nvme-pci: harden drive presence detect in nvme_dev_disable()")
Reported-by: Niklas Schnelle 
Signed-off-by: Keith Busch 
Tested-by: Niklas Schnelle 
Signed-off-by: Christoph Hellwig

nvme: use struct group for generic command dwords

2022-07-06T17:12:56+00:00

This will allow the trace event to know the full size of the data
intended to be copied and silence read overflow checks.

Reported-by: John Garry 
Suggested-by: Christoph Hellwig 
Signed-off-by: Keith Busch 
Signed-off-by: Christoph Hellwig

nvme-pci: phison e16 has bogus namespace ids

2022-07-06T16:18:41+00:00

Add the quirk.

Link: https://bugzilla.kernel.org/show_bug.cgi?id=216049
Reported-by: Chris Egolf 
Signed-off-by: Keith Busch 
Reviewed-by: Chaitanya Kulkarni 
Signed-off-by: Christoph Hellwig

nvme-pci: add NVME_QUIRK_BOGUS_NID for ADATA IM2P33F8ABR1

2022-06-30T06:24:33+00:00

ADATA IM2P33F8ABR1 reports bogus eui64 values that appear to be the same
across all drives. Quirk them out so they are not marked as "non globally
unique" duplicates.

Co-developed-by: Felipe de Jesus Araujo da Conceição 
Signed-off-by: Felipe de Jesus Araujo da Conceição 
Signed-off-by: Lamarque V. Souza 

Cc: stable@vger.kernel.org
Signed-off-by: Christoph Hellwig

nvmet: add a clear_ids attribute for passthru targets

2022-06-30T06:23:24+00:00

If the clear_ids attribute is set to true, the EUI/GUID/UUID is cleared
for the passthru target.  By default, loop targets will set clear_ids to
true.

This resolves an issue where a connect to a passthru target fails when
using a trtype of 'loop' because EUI/GUID/UUID is not unique.

Fixes: 2079f41ec6ff ("nvme: check that EUI/GUID/UUID are globally unique")
Signed-off-by: Alan Adamson 
Reviewed-by: Keith Busch 
Reviewed-by: Chaitanya Kulkarni 
Signed-off-by: Christoph Hellwig