Commit d88f1833 authored by David Woodhouse's avatar David Woodhouse Committed by Al Viro
Browse files

[PATCH] Remove XFS buffered readdir hack 



Now that we've moved the readdir hack to the nfsd code, we can
remove the local version from the XFS code.

Signed-off-by: default avatarDavid Woodhouse <David.Woodhouse@intel.com>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 14f7dd63

fs/xfs/linux-2.6/xfs_file.c

+0 −128
Original line number Diff line number Diff line
@@ -204,15 +204,6 @@ xfs_file_fsync( 
	return -xfs_fsync(XFS_I(dentry->d_inode));

}



/*

 * Unfortunately we can't just use the clean and simple readdir implementation

 * below, because nfs might call back into ->lookup from the filldir callback

 * and that will deadlock the low-level btree code.

 *

 * Hopefully we'll find a better workaround that allows to use the optimal

 * version at least for local readdirs for 2.6.25.

 */

#if 0

STATIC int

xfs_file_readdir(

	struct file	*filp,
@@ -244,125 +235,6 @@ xfs_file_readdir( 
		return -error;

	return 0;

}

#else



struct hack_dirent {

	u64		ino;

	loff_t		offset;

	int		namlen;

	unsigned int	d_type;

	char		name[];

};



struct hack_callback {

	char		*dirent;

	size_t		len;

	size_t		used;

};



STATIC int

xfs_hack_filldir(

	void		*__buf,

	const char	*name,

	int		namlen,

	loff_t		offset,

	u64		ino,

	unsigned int	d_type)

{

	struct hack_callback *buf = __buf;

	struct hack_dirent *de = (struct hack_dirent *)(buf->dirent + buf->used);

	unsigned int reclen;



	reclen = ALIGN(sizeof(struct hack_dirent) + namlen, sizeof(u64));

	if (buf->used + reclen > buf->len)

		return -EINVAL;



	de->namlen = namlen;

	de->offset = offset;

	de->ino = ino;

	de->d_type = d_type;

	memcpy(de->name, name, namlen);

	buf->used += reclen;

	return 0;

}



STATIC int

xfs_file_readdir(

	struct file	*filp,

	void		*dirent,

	filldir_t	filldir)

{

	struct inode	*inode = filp->f_path.dentry->d_inode;

	xfs_inode_t	*ip = XFS_I(inode);

	struct hack_callback buf;

	struct hack_dirent *de;

	int		error;

	loff_t		size;

	int		eof = 0;

	xfs_off_t       start_offset, curr_offset, offset;



	/*

	 * Try fairly hard to get memory

	 */

	buf.len = PAGE_CACHE_SIZE;

	do {

		buf.dirent = kmalloc(buf.len, GFP_KERNEL);

		if (buf.dirent)

			break;

		buf.len >>= 1;

	} while (buf.len >= 1024);



	if (!buf.dirent)

		return -ENOMEM;



	curr_offset = filp->f_pos;

	if (curr_offset == 0x7fffffff)

		offset = 0xffffffff;

	else

		offset = filp->f_pos;



	while (!eof) {

		unsigned int reclen;



		start_offset = offset;



		buf.used = 0;

		error = -xfs_readdir(ip, &buf, buf.len, &offset,

				     xfs_hack_filldir);

		if (error || offset == start_offset) {

			size = 0;

			break;

		}



		size = buf.used;

		de = (struct hack_dirent *)buf.dirent;

		while (size > 0) {

			curr_offset = de->offset /* & 0x7fffffff */;

			if (filldir(dirent, de->name, de->namlen,

					curr_offset & 0x7fffffff,

					de->ino, de->d_type)) {

				goto done;

			}



			reclen = ALIGN(sizeof(struct hack_dirent) + de->namlen,

				       sizeof(u64));

			size -= reclen;

			de = (struct hack_dirent *)((char *)de + reclen);

		}

	}



 done:

	if (!error) {

		if (size == 0)

			filp->f_pos = offset & 0x7fffffff;

		else if (de)

			filp->f_pos = curr_offset;

	}



	kfree(buf.dirent);

	return error;

}

#endif



STATIC int

xfs_file_mmap(