Bug 723669

Summary: rdma modules should not load by default when iscsi starts
Product: Red Hat Enterprise Linux 5 Reporter: Dave Wysochanski <dwysocha>
Component: iscsi-initiator-utilsAssignee: Chris Leech <cleech>
Status: CLOSED WONTFIX QA Contact: Bruno Goncalves <bgoncalv>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.6CC: bgoncalv, coughlan, fge, kzhang, mchristi, sforsber
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-10-17 15:37:41 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Dave Wysochanski 2011-07-20 19:13:59 UTC
Description of problem:

"service iscsi start" results in rdma_* modules being loaded by default.
This is not a common configuration and they do not appear to be unloaded automatically.



# lsmod | grep rdma
# service iscsi start
iscsid is stopped
Starting iSCSI daemon:                                     [  OK  ]
                                                           [  OK  ]
Setting up iSCSI targets: Logging in to [iface: default, target: iqn.1992-08.com.netapp:rhel6-iscsi-tgt, portal: 192.168.122.200,3260]
Logging in to [iface: default, target: iqn.1992-08.com.netapp:rhel6-iscsi-tgt, portal: 192.168.122.201,3260]
Login to [iface: default, target: iqn.1992-08.com.netapp:rhel6-iscsi-tgt, portal: 192.168.122.200,3260] successful.
Login to [iface: default, target: iqn.1992-08.com.netapp:rhel6-iscsi-tgt, portal: 192.168.122.201,3260] successful.
                                                           [  OK  ]
# lsmod | grep rdma
rdma_cm                68689  1 ib_iser
ib_cm                  72809  1 rdma_cm
iw_cm                  43465  1 rdma_cm
ib_sa                  74953  2 rdma_cm,ib_cm
ib_core               104901  6 ib_iser,rdma_cm,ib_cm,iw_cm,ib_sa,ib_mad
ib_addr                41673  1 rdma_cm
# rpm -q iscsi-initiator-utils
iscsi-initiator-utils-6.2.0.872-6.el5
# uname -a
Linux rhel5.6-node1 2.6.18-238.el5 #1 SMP Sun Dec 19 14:22:44 EST 2010 x86_64 x86_64 x86_64 GNU/Linux

Comment 1 Dave Wysochanski 2011-07-20 19:15:04 UTC
Hopefully this is an easy fix - add a config parameter or something.

Comment 2 Mike Christie 2011-07-21 03:43:52 UTC
(In reply to comment #1)
> Hopefully this is an easy fix - add a config parameter or something.

Why are you only picking on iser :) The rdma modules are loaded because ib_iser is.

The iscsi init scripts are dumb and just load all possible iscsi modules when the service starts. So I was asking about picking on iser above because I think we  do not want to auto load any iscsi module.

I have been working on a patch so that when a iscsiadm/iscsid command requests to use a module it is loaded at that time. This should solve your issue. Adding a devel ack to add that to 5.8.

Comment 3 Gris Ge 2011-08-01 08:24:43 UTC
(In reply to comment #2)
> I have been working on a patch so that when a iscsiadm/iscsid command requests
> to use a module it is loaded at that time. This should solve your issue. Adding
> a devel ack to add that to 5.8.
Mike, 

so it will acked like in RHEL6, service iscsid start will not start the daemon,
iscsiadm discovery do.

Comment 4 Mike Christie 2011-08-01 16:40:55 UTC
Not exactly. The services will start like they do today. The services would just no load the iscsi kernel modules when they start. The iscsi kernel modules will load when someone runs a iscsiadm command that needs them.

So

iscsiadm -m session
iscsiadm -m node

would not load a module since we are just printing session/record info, but


iscsiadm -m discovery -t st -p ip
iscsiadm -m node ..... -l

would load the iscsi module needed by those commands.

Comment 5 Gris Ge 2011-08-02 06:31:46 UTC
Thanks for the clarification.

Comment 7 Mike Christie 2012-01-18 01:41:07 UTC
There is a nice modparam lib now, so going to work on getting that added to rhel and converting to that. Move to 5.9.

Comment 8 Mike Christie 2012-03-02 21:45:00 UTC
*** Bug 797776 has been marked as a duplicate of this bug. ***

Comment 11 Mike Christie 2012-03-29 21:46:40 UTC
Fix is upstream so it will be brought in the next rebase. Adding devel ack.