[LU-12480] add STATX_PROJID to upstream kernel

• Status: Open
• Project: Lustre
• Type: Improvement
• Priority: Minor
• Reporter: Andreas Dilger
• Assignee: WC Triage
• Resolution: Unresolved
• Labels: lad23dd, lug23dd, medium, quota
• Epic: quota

Description

Support for statx() was landed to Lustre 2.14 in LU-10934, and I realize that the upstream kernel does not support requesting the project ID as part of that interface.

It would be very useful to add a new STATX_PROJID = 0x2000 flag to the upstream kernel, and integrate this into the statx() handling for ext4, xfs, and f2fs if that is easy. It seems possible to use the sb->dq_op->get_projid() interface to fetch this at the VFS level, if available. The STATX_ATTR_PROJINHERIT = 0x20000000 flag should be added to stx_attributes_mask.

While it may take some time to get this into a kernel that we are using, better to start it now, and once it is landed upstream we can at least request to get it added to RHEL8 or similar.

You should add David Howells <[e-mail]> to the CC list for the patch review.

Issue Links

• is related to: LU-12511 Prepare lustre for adoption into the ...
• is related to: LU-10934 integrate statx() API with Lustre
• is related to: LU-12056 tar doesn't support project id
• is related to: LU-15519 fallocate does not increase projid qu...
• is related to: LU-16085 Ubuntu 22.04 sanityn test_106c: suppo...
• is related to: LU-11872 Request for option not to follow syml...
• is related to: LU-15837 "lfs find -printf" improvements
• is related to: LU-15372 add projid to changelog
• is related to: LU-15723 Add fget_projid() to llapi
• is related to: LU-9555 "df /path/to/project" should return p...

Comments

[Andreas Dilger]

Shilong had submitted patches for this, but it looks like they were not accepted for minor reasons:
https://lore.kernel.org/linux-fsdevel/[e-mail]/

and a test case for xfstests:
https://lore.kernel.org/linux-fsdevel/[e-mail]/

The fchownat() interface changes were less warmly welcomed, so it makes sense to resubmit the statx() enhancement by itself with the review comments addressed.