[LU-15969] new project VFS interface ->fileattr_get for PCC project quota Created: 24/Jun/22  Updated: 19/Jul/23

Status: Reopened
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: Lustre 2.16.0

Type: Bug Priority: Minor
Reporter: Qian Yingjin Assignee: Qian Yingjin
Resolution: Unresolved Votes: 0
Labels: None

Issue Links:
Related
Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

The new Ubuntu 2204 kernel introduces a new project set/get VFS interface ->fileattr_get() and ->fileattr_set().
In PCC, we set the project ID for PCC copies on PCC backend device via FS_IOC_FSSETXATTR:

old_fs = force_uaccess_begin();
        rc = file->f_op->unlocked_ioctl(file, FS_IOC_FSSETXATTR,
                                        (unsigned long)&fsx);
force_uaccess_end(old_fs);

However, the new kernel removes this project IOCTL, this result in the ioctl returns -ENOTTY.
and result in the sanity-pcc test_39 failed.

We need to add compatibility check for this new project VFS interface, and fix the setting for PCC project ID properly.



 Comments   
Comment by Gerrit Updater [ 17/Apr/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50649
Subject: LU-15969 llite: add support for ->fileattr_get/set
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: afcba16dec36605613f25e5953a0c710b12fc52b

Comment by Gerrit Updater [ 19/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50649/
Subject: LU-15969 llite: add support for ->fileattr_get/set
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 1addb410788f0415a85bef72afff8533ac9da012

Comment by Peter Jones [ 19/May/23 ]

Landed for 2.16

Comment by Jian Yu [ 23/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50649/

This patch caused group lock regression failures on Ubuntu 22.04, SLES15 SP4, RHEL 9.1 and 9.2:
https://testing.whamcloud.com/test_sessions/6430eeca-c44e-483d-86cd-88466fa6fc64
https://testing.whamcloud.com/test_sessions/dd21b19c-afe9-44d1-9439-ccc7fba8bc47

Comment by Gerrit Updater [ 23/May/23 ]

"Oleg Drokin <green@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51093
Subject: Revert "LU-15969 llite: add support for ->fileattr_get/set"
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8a2bbbd0a9c2f897014f0278a6ef2850d5972f79

Comment by Gerrit Updater [ 23/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51093/
Subject: Revert "LU-15969 llite: add support for ->fileattr_get/set"
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f0e1c585d9873cbe342ea07f768b42ad1ce1be9f

Comment by Gerrit Updater [ 23/May/23 ]

"Neil Brown <neilb@suse.de>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51107
Subject: LU-15969 llite: add support for ->fileattr_get/set
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 14612b8806c95deffa10d03b6031518c9a6b7f94

Comment by Gerrit Updater [ 11/Jul/23 ]

"Shaun Tancheff <shaun.tancheff@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51634
Subject: LU-15969 llite: add support for ->fileattr_get/set (DUPE)
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a5dde9e2f9ce3b024220a04631de2b2006120934

Comment by Gerrit Updater [ 19/Jul/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51107/
Subject: LU-15969 llite: add support for ->fileattr_get/set
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d7aa8de396f585e7001a7ae49dfcba3707005135

Generated at Sat Feb 10 03:22:51 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.