On Mon, Feb 03, 2020 at 06:58:47PM +0100, Pavel Reichl wrote: > Signed-off-by: Pavel Reichl <preichl@xxxxxxxxxx> > Suggested-by: Dave Chinner <dchinner@xxxxxxxxxx> > --- > fs/xfs/libxfs/xfs_bmap.c | 4 ++-- > fs/xfs/xfs_bmap_util.c | 4 ++-- > fs/xfs/xfs_file.c | 3 ++- > fs/xfs/xfs_inode.c | 2 +- > fs/xfs/xfs_iops.c | 2 +- > 5 files changed, 8 insertions(+), 7 deletions(-) .... > diff --git a/fs/xfs/xfs_file.c b/fs/xfs/xfs_file.c > index b8a4a3f29b36..9b3958ca73d9 100644 > --- a/fs/xfs/xfs_file.c > +++ b/fs/xfs/xfs_file.c > @@ -770,7 +770,8 @@ xfs_break_layouts( > bool retry; > int error; > > - ASSERT(xfs_isilocked(XFS_I(inode), XFS_IOLOCK_SHARED|XFS_IOLOCK_EXCL)); > + ASSERT(xfs_is_iolocked(XFS_I(inode), > + XFS_IOLOCK_SHARED|XFS_IOLOCK_EXCL)); Whitespace: XFS_IOLOCK_SHARED | XFS_IOLOCK_EXCL Otherwise looks ok. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx