Commit 6290e23f authored by Dmitry Bogdanov's avatar Dmitry Bogdanov Committed by Martin K. Petersen
Browse files

scsi: target: core: UA on all LUNs after reset 

Allocate UNIT ATTENTION "BUS DEVICE RESET OCCURRED" on all LUNs on all
target ports of the device upon reception of TMF LUN RESET.

This change passes libiscsi test SCSI.MultipathIO.Reset.

Link: https://lore.kernel.org/r/20220913163602.20597-1-d.bogdanov@yadro.com


Signed-off-by: default avatarDmitry Bogdanov <d.bogdanov@yadro.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 3e2deba7

drivers/target/target_core_device.c

+19 −0
Original line number Diff line number Diff line
@@ -284,6 +284,25 @@ void target_pr_kref_release(struct kref *kref) 
	complete(&deve->pr_comp);

}



/*

 * Establish UA condition on SCSI device - all LUNs

 */

void target_dev_ua_allocate(struct se_device *dev, u8 asc, u8 ascq)

{

	struct se_dev_entry *se_deve;

	struct se_lun *lun;



	spin_lock(&dev->se_port_lock);

	list_for_each_entry(lun, &dev->dev_sep_list, lun_dev_link) {



		spin_lock(&lun->lun_deve_lock);

		list_for_each_entry(se_deve, &lun->lun_deve_list, lun_link)

			core_scsi3_ua_allocate(se_deve, asc, ascq);

		spin_unlock(&lun->lun_deve_lock);

	}

	spin_unlock(&dev->se_port_lock);

}



static void

target_luns_data_has_changed(struct se_node_acl *nacl, struct se_dev_entry *new,

			     bool skip_new)

drivers/target/target_core_internal.h

+1 −0
Original line number Diff line number Diff line
@@ -89,6 +89,7 @@ int target_configure_device(struct se_device *dev); 
void	target_free_device(struct se_device *);

int	target_for_each_device(int (*fn)(struct se_device *dev, void *data),

			       void *data);

void	target_dev_ua_allocate(struct se_device *dev, u8 asc, u8 ascq);



/* target_core_configfs.c */

extern struct configfs_item_operations target_core_dev_item_ops;

drivers/target/target_core_transport.c

+1 −2
Original line number Diff line number Diff line
@@ -3531,8 +3531,7 @@ static void target_tmr_work(struct work_struct *work) 
		tmr->response = (!ret) ? TMR_FUNCTION_COMPLETE :

					 TMR_FUNCTION_REJECTED;

		if (tmr->response == TMR_FUNCTION_COMPLETE) {

			target_ua_allocate_lun(cmd->se_sess->se_node_acl,

					       cmd->orig_fe_lun, 0x29,

			target_dev_ua_allocate(dev, 0x29,

					       ASCQ_29H_BUS_DEVICE_RESET_FUNCTION_OCCURRED);

		}

		break;