diff options
-rw-r--r-- | drivers/s390/scsi/zfcp_erp.c | 7 | ||||
-rw-r--r-- | drivers/s390/scsi/zfcp_fsf.c | 3 | ||||
-rw-r--r-- | drivers/s390/scsi/zfcp_sysfs.c | 4 |
3 files changed, 2 insertions, 12 deletions
diff --git a/drivers/s390/scsi/zfcp_erp.c b/drivers/s390/scsi/zfcp_erp.c index c82fe65c4128..2c5d4567d1da 100644 --- a/drivers/s390/scsi/zfcp_erp.c +++ b/drivers/s390/scsi/zfcp_erp.c | |||
@@ -821,11 +821,6 @@ static int zfcp_erp_port_forced_strategy_close(struct zfcp_erp_action *act) | |||
821 | return ZFCP_ERP_CONTINUES; | 821 | return ZFCP_ERP_CONTINUES; |
822 | } | 822 | } |
823 | 823 | ||
824 | static void zfcp_erp_port_strategy_clearstati(struct zfcp_port *port) | ||
825 | { | ||
826 | atomic_clear_mask(ZFCP_STATUS_COMMON_ACCESS_DENIED, &port->status); | ||
827 | } | ||
828 | |||
829 | static int zfcp_erp_port_forced_strategy(struct zfcp_erp_action *erp_action) | 824 | static int zfcp_erp_port_forced_strategy(struct zfcp_erp_action *erp_action) |
830 | { | 825 | { |
831 | struct zfcp_port *port = erp_action->port; | 826 | struct zfcp_port *port = erp_action->port; |
@@ -833,7 +828,6 @@ static int zfcp_erp_port_forced_strategy(struct zfcp_erp_action *erp_action) | |||
833 | 828 | ||
834 | switch (erp_action->step) { | 829 | switch (erp_action->step) { |
835 | case ZFCP_ERP_STEP_UNINITIALIZED: | 830 | case ZFCP_ERP_STEP_UNINITIALIZED: |
836 | zfcp_erp_port_strategy_clearstati(port); | ||
837 | if ((status & ZFCP_STATUS_PORT_PHYS_OPEN) && | 831 | if ((status & ZFCP_STATUS_PORT_PHYS_OPEN) && |
838 | (status & ZFCP_STATUS_COMMON_OPEN)) | 832 | (status & ZFCP_STATUS_COMMON_OPEN)) |
839 | return zfcp_erp_port_forced_strategy_close(erp_action); | 833 | return zfcp_erp_port_forced_strategy_close(erp_action); |
@@ -933,7 +927,6 @@ static int zfcp_erp_port_strategy(struct zfcp_erp_action *erp_action) | |||
933 | 927 | ||
934 | switch (erp_action->step) { | 928 | switch (erp_action->step) { |
935 | case ZFCP_ERP_STEP_UNINITIALIZED: | 929 | case ZFCP_ERP_STEP_UNINITIALIZED: |
936 | zfcp_erp_port_strategy_clearstati(port); | ||
937 | if (p_status & ZFCP_STATUS_COMMON_OPEN) | 930 | if (p_status & ZFCP_STATUS_COMMON_OPEN) |
938 | return zfcp_erp_port_strategy_close(erp_action); | 931 | return zfcp_erp_port_strategy_close(erp_action); |
939 | break; | 932 | break; |
diff --git a/drivers/s390/scsi/zfcp_fsf.c b/drivers/s390/scsi/zfcp_fsf.c index 0fe8d5d95119..21ec5e2f584c 100644 --- a/drivers/s390/scsi/zfcp_fsf.c +++ b/drivers/s390/scsi/zfcp_fsf.c | |||
@@ -1396,8 +1396,7 @@ static void zfcp_fsf_open_port_handler(struct zfcp_fsf_req *req) | |||
1396 | port->handle = header->port_handle; | 1396 | port->handle = header->port_handle; |
1397 | atomic_set_mask(ZFCP_STATUS_COMMON_OPEN | | 1397 | atomic_set_mask(ZFCP_STATUS_COMMON_OPEN | |
1398 | ZFCP_STATUS_PORT_PHYS_OPEN, &port->status); | 1398 | ZFCP_STATUS_PORT_PHYS_OPEN, &port->status); |
1399 | atomic_clear_mask(ZFCP_STATUS_COMMON_ACCESS_DENIED | | 1399 | atomic_clear_mask(ZFCP_STATUS_COMMON_ACCESS_BOXED, |
1400 | ZFCP_STATUS_COMMON_ACCESS_BOXED, | ||
1401 | &port->status); | 1400 | &port->status); |
1402 | /* check whether D_ID has changed during open */ | 1401 | /* check whether D_ID has changed during open */ |
1403 | /* | 1402 | /* |
diff --git a/drivers/s390/scsi/zfcp_sysfs.c b/drivers/s390/scsi/zfcp_sysfs.c index 672b57219e11..6b6641298f07 100644 --- a/drivers/s390/scsi/zfcp_sysfs.c +++ b/drivers/s390/scsi/zfcp_sysfs.c | |||
@@ -73,9 +73,7 @@ ZFCP_DEFINE_ATTR(zfcp_port, port, status, "0x%08x\n", | |||
73 | ZFCP_DEFINE_ATTR(zfcp_port, port, in_recovery, "%d\n", | 73 | ZFCP_DEFINE_ATTR(zfcp_port, port, in_recovery, "%d\n", |
74 | (atomic_read(&port->status) & | 74 | (atomic_read(&port->status) & |
75 | ZFCP_STATUS_COMMON_ERP_INUSE) != 0); | 75 | ZFCP_STATUS_COMMON_ERP_INUSE) != 0); |
76 | ZFCP_DEFINE_ATTR(zfcp_port, port, access_denied, "%d\n", | 76 | ZFCP_DEFINE_ATTR_CONST(port, access_denied, "%d\n", 0); |
77 | (atomic_read(&port->status) & | ||
78 | ZFCP_STATUS_COMMON_ACCESS_DENIED) != 0); | ||
79 | 77 | ||
80 | ZFCP_DEFINE_ATTR(zfcp_unit, unit, status, "0x%08x\n", | 78 | ZFCP_DEFINE_ATTR(zfcp_unit, unit, status, "0x%08x\n", |
81 | zfcp_unit_sdev_status(unit)); | 79 | zfcp_unit_sdev_status(unit)); |