Whamcloud - gitweb
LU-15125 o2iblnd: wrong list used for kib_connd_waits 16/45316/2
authorChris Horn <chris.horn@hpe.com>
Wed, 20 Oct 2021 19:58:52 +0000 (14:58 -0500)
committerOleg Drokin <green@whamcloud.com>
Wed, 3 Nov 2021 02:37:13 +0000 (02:37 +0000)
The ibc_list field of struct kib_conn is used for the kib_connd_waits
list, but kiblnd_connd() uses ibc_sched_list in the
list_first_entry_or_null macro.

Test-Parameters: trivial
Fixes: 34b57a6f8f ("LU-12678 lnet: use list_first_entry() in lnet/klnds subdirectory.")
Signed-off-by: Chris Horn <chris.horn@hpe.com>
Change-Id: I0cc8a94636a5129956c53e48ae96b27ece5f0228
Reviewed-on: https://review.whamcloud.com/45316
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: James Simmons <jsimmons@infradead.org>
Reviewed-by: Alexey Lyashkov <alexey.lyashkov@hpe.com>
Reviewed-by: Serguei Smirnov <ssmirnov@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lnet/klnds/o2iblnd/o2iblnd_cb.c

index 87eff59..d7571f4 100644 (file)
@@ -3569,8 +3569,7 @@ kiblnd_connd (void *arg)
                }
 
                conn = list_first_entry_or_null(&kiblnd_data.kib_connd_waits,
                }
 
                conn = list_first_entry_or_null(&kiblnd_data.kib_connd_waits,
-                                               struct kib_conn,
-                                               ibc_sched_list);
+                                               struct kib_conn, ibc_list);
                if (conn) {
                        list_del(&conn->ibc_list);
                        spin_unlock_irqrestore(lock, flags);
                if (conn) {
                        list_del(&conn->ibc_list);
                        spin_unlock_irqrestore(lock, flags);