mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	xfs: reduce code duplication handling inode32/64 options
Add xfs_set_inode32() to be used to enable inode32 allocation mode. this will reduce the amount of duplicated code needed to mount/remount a filesystem with inode32 option. This patch also changes xfs_set_inode64() to return the maximum AG number that inodes can be allocated instead of set mp->m_maxagi by itself, so that the behaviour is the same as xfs_set_inode32(). This simplifies code that calls these functions and needs to know the maximum AG that inodes can be allocated in. Signed-off-by: Carlos Maiolino <cmaiolino@redhat.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Mark Tinguely <tinguely@sgi.com> Signed-off-by: Ben Myers <bpm@sgi.com>
This commit is contained in:
		
							parent
							
								
									08bf540412
								
							
						
					
					
						commit
						2d2194f61f
					
				
					 3 changed files with 72 additions and 62 deletions
				
			
		| 
						 | 
				
			
			@ -440,7 +440,7 @@ xfs_initialize_perag(
 | 
			
		|||
	xfs_agnumber_t	agcount,
 | 
			
		||||
	xfs_agnumber_t	*maxagi)
 | 
			
		||||
{
 | 
			
		||||
	xfs_agnumber_t	index, max_metadata;
 | 
			
		||||
	xfs_agnumber_t	index;
 | 
			
		||||
	xfs_agnumber_t	first_initialised = 0;
 | 
			
		||||
	xfs_perag_t	*pag;
 | 
			
		||||
	xfs_agino_t	agino;
 | 
			
		||||
| 
						 | 
				
			
			@ -500,43 +500,10 @@ xfs_initialize_perag(
 | 
			
		|||
	else
 | 
			
		||||
		mp->m_flags &= ~XFS_MOUNT_32BITINODES;
 | 
			
		||||
 | 
			
		||||
	if (mp->m_flags & XFS_MOUNT_32BITINODES) {
 | 
			
		||||
		/*
 | 
			
		||||
		 * Calculate how much should be reserved for inodes to meet
 | 
			
		||||
		 * the max inode percentage.
 | 
			
		||||
		 */
 | 
			
		||||
		if (mp->m_maxicount) {
 | 
			
		||||
			__uint64_t	icount;
 | 
			
		||||
 | 
			
		||||
			icount = sbp->sb_dblocks * sbp->sb_imax_pct;
 | 
			
		||||
			do_div(icount, 100);
 | 
			
		||||
			icount += sbp->sb_agblocks - 1;
 | 
			
		||||
			do_div(icount, sbp->sb_agblocks);
 | 
			
		||||
			max_metadata = icount;
 | 
			
		||||
		} else {
 | 
			
		||||
			max_metadata = agcount;
 | 
			
		||||
		}
 | 
			
		||||
 | 
			
		||||
		for (index = 0; index < agcount; index++) {
 | 
			
		||||
			ino = XFS_AGINO_TO_INO(mp, index, agino);
 | 
			
		||||
			if (ino > XFS_MAXINUMBER_32) {
 | 
			
		||||
				index++;
 | 
			
		||||
				break;
 | 
			
		||||
			}
 | 
			
		||||
 | 
			
		||||
			pag = xfs_perag_get(mp, index);
 | 
			
		||||
			pag->pagi_inodeok = 1;
 | 
			
		||||
			if (index < max_metadata)
 | 
			
		||||
				pag->pagf_metadata = 1;
 | 
			
		||||
			xfs_perag_put(pag);
 | 
			
		||||
		}
 | 
			
		||||
	} else {
 | 
			
		||||
		for (index = 0; index < agcount; index++) {
 | 
			
		||||
			pag = xfs_perag_get(mp, index);
 | 
			
		||||
			pag->pagi_inodeok = 1;
 | 
			
		||||
			xfs_perag_put(pag);
 | 
			
		||||
		}
 | 
			
		||||
	}
 | 
			
		||||
	if (mp->m_flags & XFS_MOUNT_32BITINODES)
 | 
			
		||||
		index = xfs_set_inode32(mp);
 | 
			
		||||
	else
 | 
			
		||||
		index = xfs_set_inode64(mp);
 | 
			
		||||
 | 
			
		||||
	if (maxagi)
 | 
			
		||||
		*maxagi = index;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -599,6 +599,71 @@ xfs_max_file_offset(
 | 
			
		|||
	return (((__uint64_t)pagefactor) << bitshift) - 1;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
xfs_agnumber_t
 | 
			
		||||
xfs_set_inode32(struct xfs_mount *mp)
 | 
			
		||||
{
 | 
			
		||||
	xfs_agnumber_t	index = 0;
 | 
			
		||||
	xfs_sb_t	*sbp = &mp->m_sb;
 | 
			
		||||
	xfs_agnumber_t	max_metadata;
 | 
			
		||||
	xfs_agino_t	agino =	XFS_OFFBNO_TO_AGINO(mp, sbp->sb_agblocks -1, 0);
 | 
			
		||||
	xfs_ino_t	ino = XFS_AGINO_TO_INO(mp, sbp->sb_agcount -1, agino);
 | 
			
		||||
	xfs_perag_t	*pag;
 | 
			
		||||
 | 
			
		||||
	/* Calculate how much should be reserved for inodes to meet
 | 
			
		||||
	 * the max inode percentage.
 | 
			
		||||
	 */
 | 
			
		||||
	if (mp->m_maxicount) {
 | 
			
		||||
		__uint64_t	icount;
 | 
			
		||||
 | 
			
		||||
		icount = sbp->sb_dblocks * sbp->sb_imax_pct;
 | 
			
		||||
		do_div(icount, 100);
 | 
			
		||||
		icount += sbp->sb_agblocks - 1;
 | 
			
		||||
		do_div(icount, sbp->sb_agblocks);
 | 
			
		||||
		max_metadata = icount;
 | 
			
		||||
	} else {
 | 
			
		||||
		max_metadata = sbp->sb_agcount;
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	for (index = 0; index < sbp->sb_agcount; index++) {
 | 
			
		||||
		ino = XFS_AGINO_TO_INO(mp, index, agino);
 | 
			
		||||
		if (ino > XFS_MAXINUMBER_32) {
 | 
			
		||||
			index++;
 | 
			
		||||
			break;
 | 
			
		||||
		}
 | 
			
		||||
 | 
			
		||||
		pag = xfs_perag_get(mp, index);
 | 
			
		||||
		pag->pagi_inodeok = 1;
 | 
			
		||||
		if (index < max_metadata)
 | 
			
		||||
			pag->pagf_metadata = 1;
 | 
			
		||||
		xfs_perag_put(pag);
 | 
			
		||||
	}
 | 
			
		||||
	return index;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
xfs_agnumber_t
 | 
			
		||||
xfs_set_inode64(struct xfs_mount *mp)
 | 
			
		||||
{
 | 
			
		||||
	xfs_agnumber_t index = 0;
 | 
			
		||||
 | 
			
		||||
	for (index = 0; index < mp->m_sb.sb_agcount; index++) {
 | 
			
		||||
		struct xfs_perag	*pag;
 | 
			
		||||
 | 
			
		||||
		pag = xfs_perag_get(mp, index);
 | 
			
		||||
		pag->pagi_inodeok = 1;
 | 
			
		||||
		pag->pagf_metadata = 0;
 | 
			
		||||
		xfs_perag_put(pag);
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	/* There is no need for lock protection on m_flags,
 | 
			
		||||
	 * the rw_semaphore of the VFS superblock is locked
 | 
			
		||||
	 * during mount/umount/remount operations, so this is
 | 
			
		||||
	 * enough to avoid concurency on the m_flags field
 | 
			
		||||
	 */
 | 
			
		||||
	mp->m_flags &= ~(XFS_MOUNT_32BITINODES |
 | 
			
		||||
			 XFS_MOUNT_SMALL_INUMS);
 | 
			
		||||
	return index;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
STATIC int
 | 
			
		||||
xfs_blkdev_get(
 | 
			
		||||
	xfs_mount_t		*mp,
 | 
			
		||||
| 
						 | 
				
			
			@ -1039,30 +1104,6 @@ xfs_restore_resvblks(struct xfs_mount *mp)
 | 
			
		|||
	xfs_reserve_blocks(mp, &resblks, NULL);
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
STATIC void
 | 
			
		||||
xfs_set_inode64(struct xfs_mount *mp)
 | 
			
		||||
{
 | 
			
		||||
	int i = 0;
 | 
			
		||||
 | 
			
		||||
	for (i = 0; i < mp->m_sb.sb_agcount; i++) {
 | 
			
		||||
		struct xfs_perag	*pag;
 | 
			
		||||
 | 
			
		||||
		pag = xfs_perag_get(mp, i);
 | 
			
		||||
		pag->pagi_inodeok = 1;
 | 
			
		||||
		pag->pagf_metadata = 0;
 | 
			
		||||
		xfs_perag_put(pag);
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	/* There is no need for lock protection on m_flags,
 | 
			
		||||
	 * the rw_semaphore of the VFS superblock is locked
 | 
			
		||||
	 * during mount/umount/remount operations, so this is
 | 
			
		||||
	 * enough to avoid concurency on the m_flags field
 | 
			
		||||
	 */
 | 
			
		||||
	mp->m_flags &= ~(XFS_MOUNT_32BITINODES |
 | 
			
		||||
			 XFS_MOUNT_SMALL_INUMS);
 | 
			
		||||
	mp->m_maxagi = i;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
STATIC int
 | 
			
		||||
xfs_fs_remount(
 | 
			
		||||
	struct super_block	*sb,
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -75,6 +75,8 @@ struct block_device;
 | 
			
		|||
extern __uint64_t xfs_max_file_offset(unsigned int);
 | 
			
		||||
 | 
			
		||||
extern void xfs_blkdev_issue_flush(struct xfs_buftarg *);
 | 
			
		||||
extern xfs_agnumber_t xfs_set_inode32(struct xfs_mount *);
 | 
			
		||||
extern xfs_agnumber_t xfs_set_inode64(struct xfs_mount *);
 | 
			
		||||
 | 
			
		||||
extern const struct export_operations xfs_export_operations;
 | 
			
		||||
extern const struct xattr_handler *xfs_xattr_handlers[];
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue