From: Mr. NeilBrown <neilb@suse.de>
Date: Mon, 23 Jan 2023 21:58:40 +0000 (-0500)
Subject: LU-6142 lov: use list_for_each_entry in lov_obd.c
X-Git-Tag: 2.15.55~160
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F40%2F49740%2F3;p=fs%2Flustre-release.git

LU-6142 lov: use list_for_each_entry in lov_obd.c

Using the *_entry macro simplifies the code slightly.

Change-Id: Ia50cd2cbaf9bac6c9873ef4af7e5cbfa2c8e660e
Signed-off-by: Mr. NeilBrown <neilb@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/49740
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Shaun Tancheff <shaun.tancheff@hpe.com>
Reviewed-by: Arshad Hussain <arshad.hussain@aeoncomputing.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
---

diff --git a/lustre/lov/lov_obd.c b/lustre/lov/lov_obd.c
index c566774..04cc3ad 100644
--- a/lustre/lov/lov_obd.c
+++ b/lustre/lov/lov_obd.c
@@ -779,8 +779,7 @@ out:
 static int lov_cleanup(struct obd_device *obd)
 {
         struct lov_obd *lov = &obd->u.lov;
-	struct list_head *pos, *tmp;
-        struct pool_desc *pool;
+	struct pool_desc *pool, *tmp;
         ENTRY;
 
 	if (lov->lov_tgts_kobj) {
@@ -788,8 +787,7 @@ static int lov_cleanup(struct obd_device *obd)
 		lov->lov_tgts_kobj = NULL;
 	}
 
-	list_for_each_safe(pos, tmp, &lov->lov_pool_list) {
-		pool = list_entry(pos, struct pool_desc, pool_list);
+	list_for_each_entry_safe(pool, tmp, &lov->lov_pool_list, pool_list) {
                 /* free pool structs */
                 CDEBUG(D_INFO, "delete pool %p\n", pool);
 		/* In the function below, .hs_keycmp resolves to