[LU-6929] typo in cfs_hash_for_each_relax()

• Type: Bug
• Resolution: Fixed
• Priority: Critical
• Fix Version/s: Lustre 2.8.0
• Severity: 3
• Assignee: Niu Yawei (Inactive)
• Reporter: Niu Yawei (Inactive)

Description

This is introduced in: fe60e0135ee2334440247cde167b707b223cf11d

+       if (start > 0 && rc != 0) {
+               end = start;
+               start = 0;
+               goto again;
+       }

I think it should be "if (start > 0 && rc == 0)", rc != 0 means caller want break the iteration.

Issue Links

• is related to: LU-6529 Server side lock limits to avoid unnecessary memory exhaustion

Comments

[Peter Jones]

Landed for 2.8

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/15813/
Subject: LU-6929 libcfs: typo in cfs_hash_for_each_relax()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: dcf045eb782a0660abf092796f3c316dc72721b8

[Andreas Dilger]

Introduced in http://review.whamcloud.com/14931

[Gerrit Updater]

Niu Yawei ([e-mail]) uploaded a new patch: http://review.whamcloud.com/15813
Subject: LU-6929 libcfs: typo in cfs_hash_for_each_relax()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d1eb4612919b3e110603cf026c5c9b4a1b59ae37