diff options
| author | Eric Sandeen <sandeen@sandeen.net> | 2007-10-11 17:38:28 +1000 | 
|---|---|---|
| committer | Lachlan McIlroy <lachlan@redback.melbourne.sgi.com> | 2008-02-07 16:46:39 +1100 | 
| commit | 64137e56d76a5c05aa4411e2f5d7121593dd9478 (patch) | |
| tree | 6bc104e0ec96051771e646d8e86c82f63b428fde /fs/xfs/xfs_alloc.c | |
| parent | 869b906078720b68711569b68de0acca6b73b675 (diff) | |
| download | olio-linux-3.10-64137e56d76a5c05aa4411e2f5d7121593dd9478.tar.xz olio-linux-3.10-64137e56d76a5c05aa4411e2f5d7121593dd9478.zip  | |
[XFS] Unwrap pagb_lock.
Un-obfuscate pagb_lock, remove mutex_lock->spin_lock macros, call
spin_lock directly, remove extraneous cookie holdover from old xfs code,
and change lock type to spinlock_t.
SGI-PV: 970382
SGI-Modid: xfs-linux-melb:xfs-kern:29743a
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
Signed-off-by: Donald Douwsma <donaldd@sgi.com>
Signed-off-by: Tim Shimmin <tes@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_alloc.c')
| -rw-r--r-- | fs/xfs/xfs_alloc.c | 17 | 
1 files changed, 7 insertions, 10 deletions
diff --git a/fs/xfs/xfs_alloc.c b/fs/xfs/xfs_alloc.c index 012a649a19c..c23fb2aaa33 100644 --- a/fs/xfs/xfs_alloc.c +++ b/fs/xfs/xfs_alloc.c @@ -2500,10 +2500,9 @@ xfs_alloc_mark_busy(xfs_trans_t *tp,  	xfs_mount_t		*mp;  	xfs_perag_busy_t	*bsy;  	int			n; -	SPLDECL(s);  	mp = tp->t_mountp; -	s = mutex_spinlock(&mp->m_perag[agno].pagb_lock); +	spin_lock(&mp->m_perag[agno].pagb_lock);  	/* search pagb_list for an open slot */  	for (bsy = mp->m_perag[agno].pagb_list, n = 0; @@ -2533,7 +2532,7 @@ xfs_alloc_mark_busy(xfs_trans_t *tp,  		xfs_trans_set_sync(tp);  	} -	mutex_spinunlock(&mp->m_perag[agno].pagb_lock, s); +	spin_unlock(&mp->m_perag[agno].pagb_lock);  }  void @@ -2543,11 +2542,10 @@ xfs_alloc_clear_busy(xfs_trans_t *tp,  {  	xfs_mount_t		*mp;  	xfs_perag_busy_t	*list; -	SPLDECL(s);  	mp = tp->t_mountp; -	s = mutex_spinlock(&mp->m_perag[agno].pagb_lock); +	spin_lock(&mp->m_perag[agno].pagb_lock);  	list = mp->m_perag[agno].pagb_list;  	ASSERT(idx < XFS_PAGB_NUM_SLOTS); @@ -2559,7 +2557,7 @@ xfs_alloc_clear_busy(xfs_trans_t *tp,  		TRACE_UNBUSY("xfs_alloc_clear_busy", "missing", agno, idx, tp);  	} -	mutex_spinunlock(&mp->m_perag[agno].pagb_lock, s); +	spin_unlock(&mp->m_perag[agno].pagb_lock);  } @@ -2578,11 +2576,10 @@ xfs_alloc_search_busy(xfs_trans_t *tp,  	xfs_agblock_t		uend, bend;  	xfs_lsn_t		lsn;  	int			cnt; -	SPLDECL(s);  	mp = tp->t_mountp; -	s = mutex_spinlock(&mp->m_perag[agno].pagb_lock); +	spin_lock(&mp->m_perag[agno].pagb_lock);  	cnt = mp->m_perag[agno].pagb_count;  	uend = bno + len - 1; @@ -2615,12 +2612,12 @@ xfs_alloc_search_busy(xfs_trans_t *tp,  	if (cnt) {  		TRACE_BUSYSEARCH("xfs_alloc_search_busy", "found", agno, bno, len, n, tp);  		lsn = bsy->busy_tp->t_commit_lsn; -		mutex_spinunlock(&mp->m_perag[agno].pagb_lock, s); +		spin_unlock(&mp->m_perag[agno].pagb_lock);  		xfs_log_force(mp, lsn, XFS_LOG_FORCE|XFS_LOG_SYNC);  	} else {  		TRACE_BUSYSEARCH("xfs_alloc_search_busy", "not-found", agno, bno, len, n, tp);  		n = -1; -		mutex_spinunlock(&mp->m_perag[agno].pagb_lock, s); +		spin_unlock(&mp->m_perag[agno].pagb_lock);  	}  	return n;  |