[LU-12951] LL_IOC_GETOBDCOUNT return wrong MDT counts

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.14.0
• Severity: 3
• Assignee: Wang Shilong (Inactive)
• Reporter: Wang Shilong (Inactive)

Description

We used llapi_get_obd_count() to get MDT/OST counts, which call LL_IOC_GETOBDCOUNT
internally.

However, we return lmv->lmv_mdt_descs.ltd_tgts_size as MDT count, this is not right.
@ltd_tgts_size could be larger than actual MDT count, as we preallocate ltd_tgts and resize it if necessary.

Issue Links

• is duplicated by: LU-12975 A lot of "quotactl mdtXX failed" messages when use key -v with lfs quota
• is related to: LU-13079 ost-pools test_23a: Some errors happened when getting quota info. Some devices may be not working or deactivated. The data in "[]" is inaccurate.
• is related to: LU-12378 sanity-quota test 1 fails with 'project quota isn't released after deletion'

Comments

[Peter Jones]

Landed for 2.14

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/36713/
Subject: LU-12951 lmv: fix to return correct MDT count
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 3aa8826aabc7ceb16ed5efc8606a243684274982

[Sergey Cheremencev]

Lai pointed to problems at proposed patch https://review.whamcloud.com/#/c/36768/.
So, please ignore my previous comment.

[Sergey Cheremencev]

Hello, found this only after sending a patch from LU-12975.

Suppose to use https://review.whamcloud.com/#/c/36768/ as more general solution.
It is a regression after "LU-12624 obdclass: lu_tgt_descs cleanup" and ltd_tgts_size should be changed not only in one place.
Please take a look at my patch.

Thanks

Sergey

[Gerrit Updater]

Wang Shilong ([e-mail]) uploaded a new patch: https://review.whamcloud.com/36713
Subject: LU-12951 LMV: fix to return correct MDT count
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4531f44834c8492604bcc8e40a9351eba29eda5c