mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-12-27 12:21:22 -05:00
xfs: use the new ->i_state accessors
Change generated with coccinelle and fixed up by hand as appropriate. Signed-off-by: Mateusz Guzik <mjguzik@gmail.com> Signed-off-by: Christian Brauner <brauner@kernel.org>
This commit is contained in:
committed by
Christian Brauner
parent
a18d43041b
commit
18c61399f6
@@ -1249,7 +1249,7 @@ xchk_irele(
|
||||
* hits do not clear DONTCACHE, so we must do it here.
|
||||
*/
|
||||
spin_lock(&VFS_I(ip)->i_lock);
|
||||
VFS_I(ip)->i_state &= ~I_DONTCACHE;
|
||||
inode_state_clear(VFS_I(ip), I_DONTCACHE);
|
||||
spin_unlock(&VFS_I(ip)->i_lock);
|
||||
}
|
||||
|
||||
|
||||
@@ -1933,7 +1933,7 @@ xrep_inode_pptr(
|
||||
* Unlinked inodes that cannot be added to the directory tree will not
|
||||
* have a parent pointer.
|
||||
*/
|
||||
if (inode->i_nlink == 0 && !(inode->i_state & I_LINKABLE))
|
||||
if (inode->i_nlink == 0 && !(inode_state_read_once(inode) & I_LINKABLE))
|
||||
return 0;
|
||||
|
||||
/* Children of the superblock do not have parent pointers. */
|
||||
|
||||
@@ -915,7 +915,7 @@ xchk_pptr_looks_zapped(
|
||||
* Temporary files that cannot be linked into the directory tree do not
|
||||
* have attr forks because they cannot ever have parents.
|
||||
*/
|
||||
if (inode->i_nlink == 0 && !(inode->i_state & I_LINKABLE))
|
||||
if (inode->i_nlink == 0 && !(inode_state_read_once(inode) & I_LINKABLE))
|
||||
return false;
|
||||
|
||||
/*
|
||||
|
||||
@@ -514,7 +514,7 @@ xfs_can_free_eofblocks(
|
||||
* Caller must either hold the exclusive io lock; or be inactivating
|
||||
* the inode, which guarantees there are no other users of the inode.
|
||||
*/
|
||||
if (!(VFS_I(ip)->i_state & I_FREEING))
|
||||
if (!(inode_state_read_once(VFS_I(ip)) & I_FREEING))
|
||||
xfs_assert_ilocked(ip, XFS_IOLOCK_EXCL);
|
||||
|
||||
/* prealloc/delalloc exists only on regular files */
|
||||
|
||||
@@ -285,7 +285,7 @@ xfs_inode_mark_sick(
|
||||
* is not the case here.
|
||||
*/
|
||||
spin_lock(&VFS_I(ip)->i_lock);
|
||||
VFS_I(ip)->i_state &= ~I_DONTCACHE;
|
||||
inode_state_clear(VFS_I(ip), I_DONTCACHE);
|
||||
spin_unlock(&VFS_I(ip)->i_lock);
|
||||
}
|
||||
|
||||
@@ -309,7 +309,7 @@ xfs_inode_mark_corrupt(
|
||||
* is not the case here.
|
||||
*/
|
||||
spin_lock(&VFS_I(ip)->i_lock);
|
||||
VFS_I(ip)->i_state &= ~I_DONTCACHE;
|
||||
inode_state_clear(VFS_I(ip), I_DONTCACHE);
|
||||
spin_unlock(&VFS_I(ip)->i_lock);
|
||||
}
|
||||
|
||||
|
||||
@@ -334,7 +334,7 @@ xfs_reinit_inode(
|
||||
dev_t dev = inode->i_rdev;
|
||||
kuid_t uid = inode->i_uid;
|
||||
kgid_t gid = inode->i_gid;
|
||||
unsigned long state = inode->i_state;
|
||||
unsigned long state = inode_state_read_once(inode);
|
||||
|
||||
error = inode_init_always(mp->m_super, inode);
|
||||
|
||||
@@ -345,7 +345,7 @@ xfs_reinit_inode(
|
||||
inode->i_rdev = dev;
|
||||
inode->i_uid = uid;
|
||||
inode->i_gid = gid;
|
||||
inode->i_state = state;
|
||||
inode_state_assign_raw(inode, state);
|
||||
mapping_set_folio_min_order(inode->i_mapping,
|
||||
M_IGEO(mp)->min_folio_order);
|
||||
return error;
|
||||
@@ -411,7 +411,7 @@ xfs_iget_recycle(
|
||||
ip->i_flags |= XFS_INEW;
|
||||
xfs_perag_clear_inode_tag(pag, XFS_INO_TO_AGINO(mp, ip->i_ino),
|
||||
XFS_ICI_RECLAIM_TAG);
|
||||
inode->i_state = I_NEW;
|
||||
inode_state_assign_raw(inode, I_NEW);
|
||||
spin_unlock(&ip->i_flags_lock);
|
||||
spin_unlock(&pag->pag_ici_lock);
|
||||
|
||||
|
||||
@@ -1580,7 +1580,7 @@ xfs_iunlink_reload_next(
|
||||
next_ip->i_prev_unlinked = prev_agino;
|
||||
trace_xfs_iunlink_reload_next(next_ip);
|
||||
rele:
|
||||
ASSERT(!(VFS_I(next_ip)->i_state & I_DONTCACHE));
|
||||
ASSERT(!(inode_state_read_once(VFS_I(next_ip)) & I_DONTCACHE));
|
||||
if (xfs_is_quotacheck_running(mp) && next_ip)
|
||||
xfs_iflags_set(next_ip, XFS_IQUOTAUNCHECKED);
|
||||
xfs_irele(next_ip);
|
||||
@@ -2111,7 +2111,7 @@ xfs_rename_alloc_whiteout(
|
||||
*/
|
||||
xfs_setup_iops(tmpfile);
|
||||
xfs_finish_inode_setup(tmpfile);
|
||||
VFS_I(tmpfile)->i_state |= I_LINKABLE;
|
||||
inode_state_set_raw(VFS_I(tmpfile), I_LINKABLE);
|
||||
|
||||
*wip = tmpfile;
|
||||
return 0;
|
||||
@@ -2330,7 +2330,7 @@ xfs_rename(
|
||||
* flag from the inode so it doesn't accidentally get misused in
|
||||
* future.
|
||||
*/
|
||||
VFS_I(du_wip.ip)->i_state &= ~I_LINKABLE;
|
||||
inode_state_clear_raw(VFS_I(du_wip.ip), I_LINKABLE);
|
||||
}
|
||||
|
||||
out_commit:
|
||||
|
||||
@@ -113,9 +113,9 @@ xfs_inode_item_precommit(
|
||||
* to log the timestamps, or will clear already cleared fields in the
|
||||
* worst case.
|
||||
*/
|
||||
if (inode->i_state & I_DIRTY_TIME) {
|
||||
if (inode_state_read_once(inode) & I_DIRTY_TIME) {
|
||||
spin_lock(&inode->i_lock);
|
||||
inode->i_state &= ~I_DIRTY_TIME;
|
||||
inode_state_clear(inode, I_DIRTY_TIME);
|
||||
spin_unlock(&inode->i_lock);
|
||||
}
|
||||
|
||||
|
||||
@@ -1420,7 +1420,7 @@ xfs_setup_inode(
|
||||
bool is_meta = xfs_is_internal_inode(ip);
|
||||
|
||||
inode->i_ino = ip->i_ino;
|
||||
inode->i_state |= I_NEW;
|
||||
inode_state_set_raw(inode, I_NEW);
|
||||
|
||||
inode_sb_list_add(inode);
|
||||
/* make the inode look hashed for the writeback code */
|
||||
|
||||
@@ -17,7 +17,7 @@ xfs_can_free_cowblocks(struct xfs_inode *ip)
|
||||
{
|
||||
struct inode *inode = VFS_I(ip);
|
||||
|
||||
if ((inode->i_state & I_DIRTY_PAGES) ||
|
||||
if ((inode_state_read_once(inode) & I_DIRTY_PAGES) ||
|
||||
mapping_tagged(inode->i_mapping, PAGECACHE_TAG_DIRTY) ||
|
||||
mapping_tagged(inode->i_mapping, PAGECACHE_TAG_WRITEBACK) ||
|
||||
atomic_read(&inode->i_dio_count))
|
||||
|
||||
Reference in New Issue
Block a user