linux-stable.git/fs/ocfs2/suballoc.c, branch linux-4.3.y

ocfs2: clean up redundant NULL checks before kfree

2015-09-04T23:54:41+00:00

NULL check before kfree is redundant and so clean them up.

Signed-off-by: Joseph Qi 
Reviewed-by: Mark Fasheh 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

ocfs2: neaten do_error, ocfs2_error and ocfs2_abort

2015-09-04T23:54:41+00:00

These uses sometimes do and sometimes don't have '\n' terminations.  Make
the uses consistently use '\n' terminations and remove the newline from
the functions.

Miscellanea:

o Coalesce formats
o Realign arguments

Signed-off-by: Joe Perches 
Reviewed-by: Mark Fasheh 
Cc: Joel Becker 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

ocfs2: acknowledge return value of ocfs2_error()

2015-09-04T23:54:41+00:00

Caveat: This may return -EROFS for a read case, which seems wrong.  This
is happening even without this patch series though.  Should we convert
EROFS to EIO?

Signed-off-by: Goldwyn Rodrigues 
Cc: Mark Fasheh 
Cc: Joel Becker 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

ocfs2: rollback the cleared bits if error occurs after ocfs2_block_group_clear_bits

2015-04-14T23:48:58+00:00

ocfs2_block_group_clear_bits will clear bits in block group bitmap.
Once it succeeds but fails in the following step, it will cause block
group bitmap mismatch the corresponding count recorded in dinode.
So rollback the cleared bits if error occurs.

Signed-off-by: Joseph Qi 
Cc: Mark Fasheh 
Cc: Joel Becker 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

ocfs2: iput inode alloc when failed locally

2014-04-03T23:20:57+00:00

In ocfs2_info_handle_freeinode() and ocfs2_test_inode_bit() func, after
calls ocfs2_get_system_file_inode() to get inode ref, if calls
ocfs2_info_scan_inode_alloc() or ocfs2_inode_lock() failed, we should
iput inode alloc to avoid leaking the inode.

Signed-off-by: jiangyiwen 
Reviewed-by: Joseph Qi 
Cc: Mark Fasheh 
Cc: Joel Becker 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

ocfs2: rollback alloc_dinode counts when ocfs2_block_group_set_bits() failed

2014-04-03T23:20:56+00:00

After updating alloc_dinode counts in ocfs2_alloc_dinode_update_counts(),
if ocfs2_alloc_dinode_update_bitmap() failed, there is a rare case that
some space may be lost.

So, roll back alloc_dinode counts when ocfs2_block_group_set_bits()
failed.

[akpm@linux-foundation.org: coding-style fixes]
Signed-off-by: Younger Liu 
Reviewed-by: Mark Fasheh 
Cc: Joel Becker 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

ocfs2: call ocfs2_update_inode_fsync_trans when updating any inode

2014-04-03T23:20:56+00:00

Ensure that ocfs2_update_inode_fsync_trans() is called any time we touch
an inode in a given transaction.  This is a follow-on to the previous
patch to reduce lock contention and deadlocking during an fsync
operation.

Signed-off-by: Darrick J. Wong 
Cc: Mark Fasheh 
Cc: Joel Becker 
Cc: Wengang 
Cc: Greg Marsden 
Cc: Srinivas Eeda 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

ocfs2: remove redundant ocfs2_alloc_dinode_update_counts() and ocfs2_block_group_set_bits()

2014-01-22T00:19:42+00:00

ocfs2_alloc_dinode_update_counts() and ocfs2_block_group_set_bits() are
already provided in suballoc.c.  So, the same functions in
move_extents.c are not needed any more.

Declare the functions in suballoc.h and remove redundant functions in
move_extents.c.

Signed-off-by: Younger Liu 
Cc: Younger Liu 
Cc: Mark Fasheh 
Cc: Joel Becker 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

ocfs2: return ENOMEM when sb_getblk() fails

2013-11-13T03:09:00+00:00

The only reason for sb_getblk() failing is if it can't allocate the
buffer_head.  So return ENOMEM instead when it fails.

[joseph.qi@huawei.com: ocfs2_symlink_get_block() and ocfs2_read_blocks_sync() and ocfs2_read_blocks() need the same change]
Signed-off-by: Rui Xiang 
Reviewed-by: Jie Liu 
Reviewed-by: Mark Fasheh 
Cc: Joel Becker 
Cc: Joseph Qi 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

ocfs2: remove duplicated mlog_errno() in ocfs2_relink_block_group

2013-07-03T23:07:24+00:00

Cc: Jie Liu 
Cc: Joel Becker 
Cc: Mark Fasheh 
Cc: Sunil Mushran 
Cc: Younger Liu 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds