[LU-5774] lu_object_find_at() should NOT test conditional that doesn't change inside a loop

• Type: Improvement
• Resolution: Fixed
• Priority: Trivial
• Fix Version/s: Lustre 2.7.0
• Affects Version/s: Lustre 2.7.0
• Assignee: Isaac Huang (Inactive)
• Reporter: Isaac Huang (Inactive)

Description

In lu_object_find_at():

       while (1) {
               if (conf != NULL && conf->loc_flags & LOC_F_NOWAIT) {
                       obj = lu_object_find_try(env, dev, f, conf, NULL);

                       return obj;
        ......
        }

If I haven't missed something, it doesn't make sense to have the 'if' conditional inside the while loop - conf is a const struct lu_object_conf *, i.e. it doesn't change, so the conditional would always evaluate the same.

When I first read the code, I was misled to believe that the conditional may change so it had to be tested at each iteration.

Issue Links

• is related to: LU-5885 LFSCK 3: ‘lctl lfsck_start -t namespace’ Not Progressing Under Remove Workload