[LU-550] The quota hash function isn't correct

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.1.0, Lustre 1.8.7
• Affects Version/s: Lustre 2.0.0, Lustre 2.1.0, Lustre 1.8.7, Lustre 1.8.6
• Severity: 3
• Assignee: Niu Yawei (Inactive)
• Reporter: Niu Yawei (Inactive)

Description

Quota code uses 'uid' or '(1 << 32) + gid' as the hash key, however, the quota hash function lqs_hash() always try to interpret such a unsigned long long key as a 'quota_adjust_qunit' struct, then calculate the hash value by quota_adjust_qunit->qaq_id, since the misused key->qaq_id most likely be a certain value (0 or something else), the calculated hash value for all uid/gid will be same.

I think this problem might be introduced by some landing or hash converting long time ago, since we can still store/fetch the qunit for each uig/gid (though they are always in same bucket), the bug was not detected by any test.

Issue Links

• Trackbacks: Changelog 1.8 {}version 1.8.7wc1{} {}Support for networks: socklnd \any kernel supported by Lustre, qswlnd Qsnet kernel modules 5.20 and later, openiblnd IbGold 1.8.2, o2iblnd OFED 1.3, 1.4.1, 1.4.2, 1.5.1, 1.5.2, 1.5.3.1 and 1.5.3.2 gmlnd GM 2.1....

Comments

[Niu Yawei (Inactive)]

landed for 1.8.7 and 2.1

[Build Master (Inactive)]

Integrated in lustre-master » i686,server,el5,inkernel #255
LU-550 incorrect quota hash function

Oleg Drokin : d24dce9f96591cf63d7c1ab22c36b9aed16a4ef2
Files :

• lustre/quota/quota_context.c

[Build Master (Inactive)]

Integrated in lustre-master » i686,client,el5,ofa #255
LU-550 incorrect quota hash function

Oleg Drokin : d24dce9f96591cf63d7c1ab22c36b9aed16a4ef2
Files :

• lustre/quota/quota_context.c

[Build Master (Inactive)]

Integrated in lustre-master » i686,client,el6,inkernel #255
LU-550 incorrect quota hash function

Oleg Drokin : d24dce9f96591cf63d7c1ab22c36b9aed16a4ef2
Files :

• lustre/quota/quota_context.c

[Build Master (Inactive)]

Integrated in lustre-master » i686,server,el5,ofa #255
LU-550 incorrect quota hash function

Oleg Drokin : d24dce9f96591cf63d7c1ab22c36b9aed16a4ef2
Files :

• lustre/quota/quota_context.c

[Build Master (Inactive)]

Integrated in lustre-master » i686,client,el5,inkernel #255
LU-550 incorrect quota hash function

Oleg Drokin : d24dce9f96591cf63d7c1ab22c36b9aed16a4ef2
Files :

• lustre/quota/quota_context.c

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,client,ubuntu1004,inkernel #255
LU-550 incorrect quota hash function

Oleg Drokin : d24dce9f96591cf63d7c1ab22c36b9aed16a4ef2
Files :

• lustre/quota/quota_context.c

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,client,el6,inkernel #255
LU-550 incorrect quota hash function

Oleg Drokin : d24dce9f96591cf63d7c1ab22c36b9aed16a4ef2
Files :

• lustre/quota/quota_context.c

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,server,el5,inkernel #255
LU-550 incorrect quota hash function

Oleg Drokin : d24dce9f96591cf63d7c1ab22c36b9aed16a4ef2
Files :

• lustre/quota/quota_context.c

[Build Master (Inactive)]

Integrated in lustre-master » x86_64,client,sles11,inkernel #255
LU-550 incorrect quota hash function

Oleg Drokin : d24dce9f96591cf63d7c1ab22c36b9aed16a4ef2
Files :

• lustre/quota/quota_context.c