Thread (12 messages) 12 messages, 5 authors, 2021-12-17

Re: [PATCH] ext4: compare inode's i_projid with EXT4_DEF_PROJID rather than check EXT4_INODE_PROJINHERIT flag

From: Roman Anufriev <hidden>
Date: 2021-12-10 19:55:17

On Tue, 7 Dec 2021, Theodore Y. Ts'o wrote:
On Tue, Dec 07, 2021 at 05:05:19PM +0300, Roman Anufriev wrote:
quoted
quoted
Commit 7ddf79a10395 ("ext4: only set project inherit bit for directory")
removes EXT4_INODE_PROJINHERIT flag from regular files. This makes
ext4_statfs() output incorrect (function does not apply quota limits
on used/available space, etc) when called on dentry of regular file
with project quota enabled.
Under what circumstance is userspace trying to call statfs on a file
descriptor?

Removing the test for EXT4_INODE_PROJINHERIT will cause
incorrect/misleading results being returned in the case where we have
a directory where a directory hierarchy is using project id's, but
which is *not* using PROJINHERIT.
I'm not sure I quite understood what will be wrong in that case, because
as Dave mentioned:
PROJINHERIT just indicates the default projid that an inode is
created with; ...
 								Roman
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help