linux.git/drivers/scsi, branch v5.18-rc2

scsi: megaraid_sas: Target with invalid LUN ID is deleted during scan

2022-04-07T02:58:17+00:00

The megaraid_sas driver supports single LUN for RAID devices. That is LUN
0. All other LUNs are unsupported. When a device scan on a logical target
with invalid LUN number is invoked through sysfs, that target ends up
getting removed.

Add LUN ID validation in the slave destroy function to avoid the target
deletion.

Link: https://lore.kernel.org/r/20220324094711.48833-1-chandrakanth.patil@broadcom.com
Signed-off-by: Chandrakanth patil 
Signed-off-by: Martin K. Petersen

scsi: ufs: ufshpb: Fix a NULL check on list iterator

2022-04-07T02:40:41+00:00

The list iterator is always non-NULL so the check 'if (!rgn)' is always
false and the dev_err() is never called. Move the check outside the loop
and determine if 'victim_rgn' is NULL, to fix this bug.

Link: https://lore.kernel.org/r/20220320150733.21824-1-xiam0nd.tong@gmail.com
Fixes: 4b5f49079c52 ("scsi: ufs: ufshpb: L2P map management for HPB read")
Reviewed-by: Daejun Park 
Signed-off-by: Xiaomeng Tong 
Signed-off-by: Martin K. Petersen

scsi: sd: Clean up gendisk if device_add_disk() failed

2022-04-07T02:37:49+00:00

We forgot to call blk_cleanup_disk() when device_add_disk() failed.  This
would cause a memory leak of gendisk and sched_tags allocated in
elevator_init_mq()

Reference:https://syzkaller.appspot.com/x/log.txt?x=13b41dcb700000
Reported-and-tested-by: syzbot+f08c77040fa163a75a46@syzkaller.appspotmail.com
Link: https://lore.kernel.org/r/20220401011018.1026553-1-haowenchao@huawei.com
Signed-off-by: Wenchao Hao 
Signed-off-by: Martin K. Petersen

scsi: mvsas: Add PCI ID of RocketRaid 2640

2022-04-07T02:27:08+00:00

The HighPoint RocketRaid 2640 is a low-cost SAS controller based on Marvell
chip. The chip in question was already supported by the kernel, just the
PCI ID of this particular board was missing.

Link: https://lore.kernel.org/r/20220309212535.402987-1-agalakhov@gmail.com
Signed-off-by: Alexey Galakhov 
Signed-off-by: Martin K. Petersen

scsi: sd: sd_read_cpr() requires VPD pages

2022-04-07T02:21:50+00:00

As such it should be called inside the scsi_device_supports_vpd()
conditional.

Link: https://lore.kernel.org/r/20220302053559.32147-13-martin.petersen@oracle.com
Fixes: e815d36548f0 ("scsi: sd: add concurrent positioning ranges support")
Cc: Damien Le Moal 
Reviewed-by: Christoph Hellwig 
Reviewed-by: Damien Le Moal 
Reviewed-by: Bart Van Assche 
Signed-off-by: Martin K. Petersen

scsi: mpt3sas: Fail reset operation if config request timed out

2022-04-07T02:14:09+00:00

As part of controller reset operation the driver issues a config request
command. If this command gets times out, then fail the controller reset
operation instead of retrying it.

Link: https://lore.kernel.org/r/20220405120637.20528-1-sreekanth.reddy@broadcom.com
Signed-off-by: Sreekanth Reddy 
Signed-off-by: Martin K. Petersen

scsi: sym53c500_cs: Stop using struct scsi_pointer

2022-04-07T02:11:52+00:00

This driver doesn't use SCp.ptr to save a SCSI command data pointer which
means "scsi pointer" is a complete misnomer here. Only a few members of
struct scsi_pointer are needed so move those to private command data.

Link: https://lore.kernel.org/r/accf71e293ba3aed6d18c8baeb405de8dfe7c935.1649235939.git.fthain@linux-m68k.org
Cc: Bart Van Assche 
Cc: Christoph Hellwig 
Reviewed-by: Bart Van Assche 
Reviewed-by: Christoph Hellwig 
Signed-off-by: Finn Thain 
Signed-off-by: Martin K. Petersen

scsi: ufs: ufs-pci: Add support for Intel MTL

2022-04-07T02:07:54+00:00

Add PCI ID and callbacks to support Intel Meteor Lake (MTL).

Link: https://lore.kernel.org/r/20220404055038.2208051-1-adrian.hunter@intel.com
Cc: stable@vger.kernel.org # v5.15+
Reviewed-by: Avri Altman 
Reviewed-by: Bart Van Assche 
Signed-off-by: Adrian Hunter 
Signed-off-by: Martin K. Petersen

scsi: mpt3sas: Fix mpt3sas_check_same_4gb_region() kdoc comment

2022-04-07T02:07:30+00:00

The start_addres argument of mpt3sas_check_same_4gb_region() was misnamed
in the function kdoc comment, resulting in the following warning when
compiling with W=1.

drivers/scsi/mpt3sas/mpt3sas_base.c:5728: warning: Function parameter or
member 'start_address' not described in 'mpt3sas_check_same_4gb_region'
drivers/scsi/mpt3sas/mpt3sas_base.c:5728: warning: Excess function
parameter 'reply_pool_start_address' description in
'mpt3sas_check_same_4gb_region'

Fix the argument name in the function kdoc comment to avoid it. While at
it, remove a useless blank line between the kdoc and function code.

Link: https://lore.kernel.org/r/20220404050041.594774-1-damien.lemoal@opensource.wdc.com
Acked-by: Sreekanth Reddy 
Signed-off-by: Damien Le Moal 
Signed-off-by: Martin K. Petersen

scsi: scsi_debug: Fix sdebug_blk_mq_poll() in_use_bm bitmap use

2022-04-07T02:04:16+00:00

The in_use_bm bitmap of struct sdebug_queue should be accessed under
protection of the qc_lock spinlock. Make sure that this lock is taken
before calling find_first_bit() at the beginning of the function
sdebug_blk_mq_poll().

Link: https://lore.kernel.org/r/20220404045547.579887-1-damien.lemoal@opensource.wdc.com
Fixes: 3fd07aecb750 ("scsi: scsi_debug: Fix qc_lock use in sdebug_blk_mq_poll()")
Acked-by: Douglas Gilbert 
Signed-off-by: Damien Le Moal 
Signed-off-by: Martin K. Petersen