summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristoph Hellwig2018-02-22 16:24:08 +0100
committerKeith Busch2018-02-22 09:45:30 +0100
commit5a1e59533380a3fd04593e4ab2d4633ebf7745c1 (patch)
treeab461e66bee5611d0345314cbee27aeee0d7f8a4
parentmm: hide a #warning for COMPILE_TEST (diff)
downloadkernel-qcow2-linux-5a1e59533380a3fd04593e4ab2d4633ebf7745c1.tar.gz
kernel-qcow2-linux-5a1e59533380a3fd04593e4ab2d4633ebf7745c1.tar.xz
kernel-qcow2-linux-5a1e59533380a3fd04593e4ab2d4633ebf7745c1.zip
nvme-fabrics: don't check for non-NULL module in nvmf_register_transport
THIS_MODULE evaluates to NULL when used from code built into the kernel, thus breaking built-in transport modules. Remove the bogus check. Fixes: 0de5cd36 ("nvme-fabrics: protect against module unload during create_ctrl") Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Sagi Grimberg <sagi@grimberg.me> Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de> Signed-off-by: Keith Busch <keith.busch@intel.com>
-rw-r--r--drivers/nvme/host/fabrics.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/nvme/host/fabrics.c b/drivers/nvme/host/fabrics.c
index 5dd4ceefed8f..a1c58e35075e 100644
--- a/drivers/nvme/host/fabrics.c
+++ b/drivers/nvme/host/fabrics.c
@@ -493,7 +493,7 @@ EXPORT_SYMBOL_GPL(nvmf_should_reconnect);
*/
int nvmf_register_transport(struct nvmf_transport_ops *ops)
{
- if (!ops->create_ctrl || !ops->module)
+ if (!ops->create_ctrl)
return -EINVAL;
down_write(&nvmf_transports_rwsem);