linux.git/fs/ext2/inode.c, branch v5.4

ext2: Always brelse bh on failure in ext2_iget()

2019-06-19T16:29:45+00:00

All but one bail out paths in ext2_iget() is releasing bh. Move the
releasing of bh into a common error handling code.

Signed-off-by: Jan Kara

ext2: add missing brelse() in ext2_iget()

2019-06-19T16:27:38+00:00

Add missing brelse() on error path of ext2_iget().

Reviewed-by: Darrick J. Wong 
Signed-off-by: Chengguang Xu 
Signed-off-by: Jan Kara

ext2: Fix a typo in ext2_getattr argument

2019-06-12T13:10:41+00:00

Fix a typo in a ext2_getattr argument

Signed-off-by: Fumiya Shigemitsu 
Signed-off-by: Jan Kara

ext2: Adjust the comment of function ext2_alloc_branch

2019-03-26T10:21:23+00:00

The name of argument is error in the header comment.
@num should be @indirect_blks.
At the same time, there was a lack of description of the two parameters
@blks and @goal.
This commit therefore fixes this header comment.

Signed-off-by: Shuning Zhang 
Signed-off-by: Jan Kara

ext2: support statx syscall

2019-02-18T14:14:43+00:00

Since statx, every filesystem should fill the attributes/attributes_mask
in routine getattr. But the generic_fillattr has not fill that, so add
ext2_getattr to do this. This can fix generic/424 while testing ext2.

Reviewed-by: zhangyi (F) 
Signed-off-by: yangerkun 
Signed-off-by: Jan Kara

ext2: Fix a typo in comment

2019-01-29T15:43:54+00:00

Fix a typo in ext2_get_blocks comment.

Signed-off-by: Liu Xiang 
Signed-off-by: Jan Kara

ext2: Annotate implicit fall through in __ext2_truncate_blocks

2019-01-28T14:50:45+00:00

There is a plan to build the kernel with -Wimplicit-fallthrough and
these places in the code produced warnings (W=1).

This commit removes the following warnings:

  fs/ext2/inode.c:1237:7: warning: this statement may fall through [-Wimplicit-fallthrough=]
  fs/ext2/inode.c:1244:7: warning: this statement may fall through [-Wimplicit-fallthrough=]

Signed-off-by: Mathieu Malaterre 
Signed-off-by: Jan Kara

ext2, dax: set ext2_dax_aops for dax files

2018-09-19T13:03:04+00:00

Sync syscall to DAX file needs to flush processor cache, but it
currently does not flush to existing DAX files.  This is because
'ext2_da_aops' is set to address_space_operations of existing DAX
files, instead of 'ext2_dax_aops', since S_DAX flag is set after
ext2_set_aops() in the open path.

Similar to ext4, change ext2_iget() to initialize i_flags before
ext2_set_aops().

Fixes: fb094c90748f ("ext2, dax: introduce ext2_dax_aops")
Signed-off-by: Toshi Kani 
Suggested-by: Jan Kara 
Cc: Jan Kara 
Cc: Dan Williams 
Cc: "Theodore Ts'o" 
Cc: Andreas Dilger 
Cc: 
Signed-off-by: Jan Kara

ext2: use ktime_get_real_seconds for timestamps

2018-06-27T11:59:18+00:00

get_seconds() is deprecated because of the y2038 overflow, so users
should migrate to 64-bit timestamps using ktime_get_real_seconds().
In ext2, the timestamps in the superblock and in the inode are all
limited to 32-bit, and this won't get fixed, so let's just stop
using the deprecated interface and keep truncating.

All users of ext2 should migrate to ext4 before 2038 to prevent this
from causing problems.

Signed-off-by: Arnd Bergmann 
Signed-off-by: Jan Kara

ext2: fix a block leak

2018-05-21T18:30:11+00:00

open file, unlink it, then use ioctl(2) to make it immutable or
append only.  Now close it and watch the blocks *not* freed...

Immutable/append-only checks belong in ->setattr().
Note: the bug is old and backport to anything prior to 737f2e93b972
("ext2: convert to use the new truncate convention") will need
these checks lifted into ext2_setattr().

Cc: stable@kernel.org
Signed-off-by: Al Viro