xfs-4.18: quota refactor
Get series with b4
b4 am 152596612549.27525.9876256551560849585.stgit@magnoliaVersions newest first · linked by author and title
- v3 Latest14 patches 2 repliesIn review
- v21 patch 1 replySuperseded
- v11 patch 6 repliesSuperseded
Patches in v3 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/14 | xfs: release new dquot buffer on defer_finish errorfs/ | +28 | 0 replies | |
| 2/14 | xfs: don't spray logs when dquot flush/purge failfs/ fs/ | +3 | 0 replies | |
| 3/14 | xfs: refactor XFS_QMOPT_DQNEXT out of existencefs/ fs/ fs/ +4 more | +108 | 0 replies | |
| 4/14 | xfs: refactor dquot cache handlingfs/ | +78 | 0 replies | |
| 5/14 | xfs: delegate dqget input checks to helper functionfs/ | +31 | 0 replies | |
| 6/14 | xfs: remove unnecessary xfs_qm_dqattach parameterfs/ fs/ fs/ +5 more | +19 | 0 replies | |
| 7/14 | xfs: split out dqget for inodes from regular dqgetfs/ fs/ fs/ +5 more | +134 | 0 replies | |
| 8/14 | xfs: fetch dquots directly during quotacheckfs/ | +12 | 0 replies | |
| 9/14 | xfs: refactor incore dquot initialization functionsfs/ | +51 | 0 replies | |
| 10/14 | xfs: refactor xfs_qm_dqtobp and xfs_qm_dqallocfs/ | +122 | 0 replies | |
| 11/14 | xfs: remove direct calls to _qm_dqreadfs/ fs/ fs/ | +39 | 0 replies | |
| 12/14 | xfs: replace XFS_QMOPT_DQALLOC with a simple booleanfs/ fs/ fs/ +3 more | +19 | 0 replies | |
| 13/14 | xfs: rename on-disk dquot counter zap functionsfs/ | +8 | 1 reply | |
| 14/14 | xfs: refactor dquot iterationfs/ fs/ fs/ | +63 | 1 reply |
Reviews trailers per patch
All 14 patches have a Reviewed-by or Acked-by.
| Patch | Brian Foster | Christoph Hellwig |
|---|---|---|
| 1/14 | R-b | |
| 2/14 | R-b | |
| 3/14 | R-b | R-b |
| 4/14 | R-b | R-b |
| 5/14 | R-b | R-b |
| 6/14 | R-b | R-b |
| 7/14 | R-b | R-b |
| 8/14 | R-b | R-b |
| 9/14 | R-b | R-b |
| 10/14 | R-b | |
| 11/14 | R-b | R-b |
| 12/14 | R-b | |
| 13/14 | R-b | R-b |
| 14/14 | R-b | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v3 00/14] xfs-4.18: quota refactor
Hi all,
Here's a series to refactor and clean out the incohesive mess that xfs_qm_dqget has become. I'm reposting this per Christoph request since the v2 thread became very hairy and disorganized.
I've distinguished three contexts in which dqget can be called, and converted them into separate externally visible interfaces -- _qm_dqget_next, which returns the first initialized dquot with an id at least as high as the id passed in; _qm_dqget_inode, which returns the dquot for a given (inode, type); and so _qm_dqget now only returns the dquot for a given (id, type). I removed unused parameters from the dqattach functions, renamed _qm_dqiterate (since it now only knows how to zap dquot counters during quotacheck), and added a real function to iterate all dquots of a given type. I also re-hid _qm_dqread in favor of a new _qm_dqget_uncached which has a better description of why we bypass the dquot cache.
This means that all _dqget variants no longer take QMOPT flags, and QMOPT_NEXT is gone. XFS_QMOPT_DQALLOC is now a boolean and can only be specified for functions where it makes sense.
This whole thing will be in for-next tomorrow, with any luck.
--D