aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Peschke <mpeschke@linux.vnet.ibm.com>2014-11-13 08:59:46 -0500
committerChristoph Hellwig <hch@lst.de>2014-11-20 03:11:28 -0500
commit1b33ef23946adee4b7d9d6b16b7285ce61063451 (patch)
treec8caf09964fe79f5d6ec3b546a9537f3fed99cc5
parentccbedf117f015d4f415130069b47d63c359bc110 (diff)
zfcp: remove access control tables interface (port leftovers)
This patch removes some leftovers for commit 663e0890e31cb85f0cca5ac1faaee0d2d52880b5 "[SCSI] zfcp: remove access control tables interface". The "access denied" case for ports is gone, as well. The corresponding flag was cleared, but never set. So clean it up. Sysfs flag is kept, though, for backward-compatibility. Now it returns always 0. Signed-off-by: Martin Peschke <mpeschke@linux.vnet.ibm.com> Signed-off-by: Steffen Maier <maier@linux.vnet.ibm.com> Reviewed-by: Hannes Reinecke <hare@suse.de> Signed-off-by: Christoph Hellwig <hch@lst.de>
-rw-r--r--drivers/s390/scsi/zfcp_erp.c7
-rw-r--r--drivers/s390/scsi/zfcp_fsf.c3
-rw-r--r--drivers/s390/scsi/zfcp_sysfs.c4
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
824static void zfcp_erp_port_strategy_clearstati(struct zfcp_port *port)
825{
826 atomic_clear_mask(ZFCP_STATUS_COMMON_ACCESS_DENIED, &port->status);
827}
828
829static int zfcp_erp_port_forced_strategy(struct zfcp_erp_action *erp_action) 824static 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",
73ZFCP_DEFINE_ATTR(zfcp_port, port, in_recovery, "%d\n", 73ZFCP_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);
76ZFCP_DEFINE_ATTR(zfcp_port, port, access_denied, "%d\n", 76ZFCP_DEFINE_ATTR_CONST(port, access_denied, "%d\n", 0);
77 (atomic_read(&port->status) &
78 ZFCP_STATUS_COMMON_ACCESS_DENIED) != 0);
79 77
80ZFCP_DEFINE_ATTR(zfcp_unit, unit, status, "0x%08x\n", 78ZFCP_DEFINE_ATTR(zfcp_unit, unit, status, "0x%08x\n",
81 zfcp_unit_sdev_status(unit)); 79 zfcp_unit_sdev_status(unit));