diff options
| author | Christoph Hellwig <hch@lst.de> | 2016-05-02 15:45:21 +0200 |
|---|---|---|
| committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2016-05-10 01:19:18 -0700 |
| commit | 22d11759a4e7018f8cd7914e4e706ca2c96d6c01 (patch) | |
| tree | 788d1e4e1828d66f60f9d444c298081013e4d7fa /drivers/target/tcm_fc/tcm_fc.h | |
| parent | fba81f8831b20272a97a990e5d47c332e9b1f65d (diff) | |
| download | op-kernel-dev-22d11759a4e7018f8cd7914e4e706ca2c96d6c01.zip op-kernel-dev-22d11759a4e7018f8cd7914e4e706ca2c96d6c01.tar.gz | |
target: make ->shutdown_session optional
Turns out the template and thus many drivers got the return value wrong:
0 means the fabrics driver needs to put a session reference, which no
driver except for the iSCSI target drivers did. Fortunately none of these
drivers supports explicit Node ACLs, so the bug was harmless.
Even without that only qla2xxx and iscsi every did real work in
shutdown_session, so get rid of the boilerplate code in all other
drivers.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target/tcm_fc/tcm_fc.h')
| -rw-r--r-- | drivers/target/tcm_fc/tcm_fc.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/target/tcm_fc/tcm_fc.h b/drivers/target/tcm_fc/tcm_fc.h index c30003b..e28209b 100644 --- a/drivers/target/tcm_fc/tcm_fc.h +++ b/drivers/target/tcm_fc/tcm_fc.h @@ -139,7 +139,6 @@ extern unsigned int ft_debug_logging; * Session ops. */ void ft_sess_put(struct ft_sess *); -int ft_sess_shutdown(struct se_session *); void ft_sess_close(struct se_session *); u32 ft_sess_get_index(struct se_session *); u32 ft_sess_get_port_name(struct se_session *, unsigned char *, u32); |
