diff options
author | James Smart <jsmart2021@gmail.com> | 2020-04-03 16:33:20 +0200 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2020-04-04 09:09:39 +0200 |
commit | 8c5c660529209a0e324c1c1a35ce3f83d67a2aa5 (patch) | |
tree | f2d9ed8ed3cf9403e6b35b977a6329f12cf1b6c7 /include | |
parent | nvme: fix deadlock caused by ANA update wrong locking (diff) | |
download | linux-8c5c660529209a0e324c1c1a35ce3f83d67a2aa5.tar.xz linux-8c5c660529209a0e324c1c1a35ce3f83d67a2aa5.zip |
nvme-fc: Revert "add module to ops template to allow module references"
The original patch was to resolve the lldd being able to be unloaded
while being used to talk to the boot device of the system. However, the
end result of the original patch is that any driver unload while a nvme
controller is live via the lldd is now being prohibited. Given the module
reference, the module teardown routine can't be called, thus there's no
way, other than manual actions to terminate the controllers.
Fixes: 863fbae929c7 ("nvme_fc: add module to ops template to allow module references")
Cc: <stable@vger.kernel.org> # v5.4+
Signed-off-by: James Smart <jsmart2021@gmail.com>
Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/nvme-fc-driver.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/include/linux/nvme-fc-driver.h b/include/linux/nvme-fc-driver.h index 6d0d70f3219c..10f81629b9ce 100644 --- a/include/linux/nvme-fc-driver.h +++ b/include/linux/nvme-fc-driver.h @@ -270,8 +270,6 @@ struct nvme_fc_remote_port { * * Host/Initiator Transport Entrypoints/Parameters: * - * @module: The LLDD module using the interface - * * @localport_delete: The LLDD initiates deletion of a localport via * nvme_fc_deregister_localport(). However, the teardown is * asynchronous. This routine is called upon the completion of the @@ -385,8 +383,6 @@ struct nvme_fc_remote_port { * Value is Mandatory. Allowed to be zero. */ struct nvme_fc_port_template { - struct module *module; - /* initiator-based functions */ void (*localport_delete)(struct nvme_fc_local_port *); void (*remoteport_delete)(struct nvme_fc_remote_port *); |