All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] scsi: qla2xxx: restore initiator in dual mode
@ 2021-09-15 15:32 Dmitry Bogdanov
  2021-09-15 15:41 ` Himanshu Madhani
  2021-09-22  4:44 ` Martin K. Petersen
  0 siblings, 2 replies; 3+ messages in thread
From: Dmitry Bogdanov @ 2021-09-15 15:32 UTC (permalink / raw)
  To: Martin Petersen, target-devel
  Cc: linux-scsi, linux, Nilesh Javali, GR-QLogic-Storage-Upstream,
	Dmitry Bogdanov

In dual mode in case of disabling the target, the whole port goes offline
and Initiator is turned off too.

This patch fixes restoring Initiator mode after disabling Target in dual mode.

Fixes: 0645cb8350cd ("scsi: qla2xxx: Add mode control for each physical port")
Signed-off-by: Dmitry Bogdanov <d.bogdanov@yadro.com>
---
This patchset is intended for scsi-fix.
---
 drivers/scsi/qla2xxx/qla_init.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/scsi/qla2xxx/qla_init.c b/drivers/scsi/qla2xxx/qla_init.c
index 1e4e3e83b5c7..5fc7697f0af4 100644
--- a/drivers/scsi/qla2xxx/qla_init.c
+++ b/drivers/scsi/qla2xxx/qla_init.c
@@ -7169,7 +7169,8 @@ qla2x00_abort_isp(scsi_qla_host_t *vha)
 				return 0;
 			break;
 		case QLA2XXX_INI_MODE_DUAL:
-			if (!qla_dual_mode_enabled(vha))
+			if (!qla_dual_mode_enabled(vha) &&
+			    !qla_ini_mode_enabled(vha))
 				return 0;
 			break;
 		case QLA2XXX_INI_MODE_ENABLED:
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] scsi: qla2xxx: restore initiator in dual mode
  2021-09-15 15:32 [PATCH] scsi: qla2xxx: restore initiator in dual mode Dmitry Bogdanov
@ 2021-09-15 15:41 ` Himanshu Madhani
  2021-09-22  4:44 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Himanshu Madhani @ 2021-09-15 15:41 UTC (permalink / raw)
  To: Dmitry Bogdanov
  Cc: Martin Petersen, open list:SCSI TARGET SUBSYSTEM, linux-scsi,
	linux, Nilesh Javali, GR-QLogic-Storage-Upstream



> On Sep 15, 2021, at 10:32 AM, Dmitry Bogdanov <d.bogdanov@yadro.com> wrote:
> 
> In dual mode in case of disabling the target, the whole port goes offline
> and Initiator is turned off too.
> 
> This patch fixes restoring Initiator mode after disabling Target in dual mode.
> 
> Fixes: 0645cb8350cd ("scsi: qla2xxx: Add mode control for each physical port")
> Signed-off-by: Dmitry Bogdanov <d.bogdanov@yadro.com>
> ---
> This patchset is intended for scsi-fix.
> ---
> drivers/scsi/qla2xxx/qla_init.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/scsi/qla2xxx/qla_init.c b/drivers/scsi/qla2xxx/qla_init.c
> index 1e4e3e83b5c7..5fc7697f0af4 100644
> --- a/drivers/scsi/qla2xxx/qla_init.c
> +++ b/drivers/scsi/qla2xxx/qla_init.c
> @@ -7169,7 +7169,8 @@ qla2x00_abort_isp(scsi_qla_host_t *vha)
> 				return 0;
> 			break;
> 		case QLA2XXX_INI_MODE_DUAL:
> -			if (!qla_dual_mode_enabled(vha))
> +			if (!qla_dual_mode_enabled(vha) &&
> +			    !qla_ini_mode_enabled(vha))
> 				return 0;
> 			break;
> 		case QLA2XXX_INI_MODE_ENABLED:
> -- 
> 2.25.1
> 

Looks Good.

Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.com>

--
Himanshu Madhani	 Oracle Linux Engineering


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] scsi: qla2xxx: restore initiator in dual mode
  2021-09-15 15:32 [PATCH] scsi: qla2xxx: restore initiator in dual mode Dmitry Bogdanov
  2021-09-15 15:41 ` Himanshu Madhani
@ 2021-09-22  4:44 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2021-09-22  4:44 UTC (permalink / raw)
  To: target-devel, Dmitry Bogdanov
  Cc: Martin K . Petersen, Nilesh Javali, linux-scsi,
	GR-QLogic-Storage-Upstream, linux

On Wed, 15 Sep 2021 18:32:39 +0300, Dmitry Bogdanov wrote:

> In dual mode in case of disabling the target, the whole port goes offline
> and Initiator is turned off too.
> 
> This patch fixes restoring Initiator mode after disabling Target in dual mode.
> 
> 

Applied to 5.15/scsi-fixes, thanks!

[1/1] scsi: qla2xxx: restore initiator in dual mode
      https://git.kernel.org/mkp/scsi/c/5f8579038842

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-09-22  4:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-15 15:32 [PATCH] scsi: qla2xxx: restore initiator in dual mode Dmitry Bogdanov
2021-09-15 15:41 ` Himanshu Madhani
2021-09-22  4:44 ` Martin K. Petersen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.