linux-kernel - Re: [PATCH plinth/topic-sas-4.18] {topost} scsi: hisi_sas: send HARD RESET to clear the previous affiliation of STP target port when init device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Thu, 23 Aug 2018 17:18:27 +0800
From:   "chenxiang (M)" <chenxiang66@...ilicon.com>
To:     <jejb@...ux.vnet.ibm.com>, <martin.petersen@...cle.com>
CC:     <linuxarm@...wei.com>, <linux-scsi@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <john.garry@...wei.com>
Subject: Re: [PATCH plinth/topic-sas-4.18] {topost} scsi: hisi_sas: send HARD
 RESET to clear the previous affiliation of STP target port when init device

Sorry, sent the wrong mail list and  please ignore the patch.

在 2018/8/23 15:58, c00284940 写道:
> If exchange SAS expander from one SAS controller to other SAS controller
> without power down, the STP target port will maintain previous affiliation and
> reject all subsequent connection requests from other STP initiator ports
> with OPEN_REJECT (STP RESOURCES BUSY).
>
> To solve this issue, send HARD RESET to clear the previous affiliation of
> STP target port according to SPL(chapter 6.19.4).
>
> Signed-off-by: Xiang Chen <chenxiang66@...ilicon.com>
> ---
>   drivers/scsi/hisi_sas/hisi_sas_main.c | 9 +++++++++
>   1 file changed, 9 insertions(+)
>
> diff --git a/drivers/scsi/hisi_sas/hisi_sas_main.c b/drivers/scsi/hisi_sas/hisi_sas_main.c
> index bb0fa24..9225138 100644
> --- a/drivers/scsi/hisi_sas/hisi_sas_main.c
> +++ b/drivers/scsi/hisi_sas/hisi_sas_main.c
> @@ -612,6 +612,7 @@ static int hisi_sas_init_device(struct domain_device *device)
>   	struct hisi_sas_tmf_task tmf_task;
>   	int retry = HISI_SAS_SRST_ATA_DISK_CNT;
>   	struct hisi_hba *hisi_hba = dev_to_hisi_hba(device);
> +	struct sas_phy *local_phy;
>   
>   	switch (device->dev_type) {
>   	case SAS_END_DEVICE:
> @@ -627,6 +628,14 @@ static int hisi_sas_init_device(struct domain_device *device)
>   	case SAS_SATA_PM:
>   	case SAS_SATA_PM_PORT:
>   	case SAS_SATA_PENDING:
> +		/*
> +		 * send HARD RESET to clear previous affiliation of
> +		 * STP target port
> +		 */
> +		local_phy = sas_get_local_phy(device);
> +		if (!scsi_is_sas_phy_local(local_phy))
> +			sas_phy_reset(local_phy, 1);
> +		sas_put_local_phy(local_phy);
>   		while (retry-- > 0) {
>   			rc = hisi_sas_softreset_ata_disk(device);
>   			if (!rc)