fs/adfs: newdir: merge adfs_dir_read() into adfs_f_read()
adfs_dir_read() is only called from adfs_f_read(), so merge it into that function. As new directories are always 2048 bytes in size, (which we rely on elsewhere) we can consolidate some of the code. Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
parent
7a0e4048bf
commit
9318731bec
|
@ -138,20 +138,16 @@ static int adfs_f_validate(struct adfs_dir *dir)
|
|||
}
|
||||
|
||||
/* Read and check that a directory is valid */
|
||||
static int adfs_dir_read(struct super_block *sb, u32 indaddr,
|
||||
unsigned int size, struct adfs_dir *dir)
|
||||
static int adfs_f_read(struct super_block *sb, u32 indaddr, unsigned int size,
|
||||
struct adfs_dir *dir)
|
||||
{
|
||||
const unsigned int blocksize_bits = sb->s_blocksize_bits;
|
||||
int ret;
|
||||
|
||||
/*
|
||||
* Directories which are not a multiple of 2048 bytes
|
||||
* are considered bad v2 [3.6]
|
||||
*/
|
||||
if (size & 2047)
|
||||
goto bad_dir;
|
||||
if (size && size != ADFS_NEWDIR_SIZE)
|
||||
return -EIO;
|
||||
|
||||
ret = adfs_dir_read_buffers(sb, indaddr, size, dir);
|
||||
ret = adfs_dir_read_buffers(sb, indaddr, ADFS_NEWDIR_SIZE, dir);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
|
@ -161,6 +157,8 @@ static int adfs_dir_read(struct super_block *sb, u32 indaddr,
|
|||
if (adfs_f_validate(dir))
|
||||
goto bad_dir;
|
||||
|
||||
dir->parent_id = adfs_readval(dir->newtail->dirparent, 3);
|
||||
|
||||
return 0;
|
||||
|
||||
bad_dir:
|
||||
|
@ -271,23 +269,6 @@ static int adfs_dir_find_entry(struct adfs_dir *dir, u32 indaddr)
|
|||
return ret;
|
||||
}
|
||||
|
||||
static int adfs_f_read(struct super_block *sb, u32 indaddr, unsigned int size,
|
||||
struct adfs_dir *dir)
|
||||
{
|
||||
int ret;
|
||||
|
||||
if (size != ADFS_NEWDIR_SIZE)
|
||||
return -EIO;
|
||||
|
||||
ret = adfs_dir_read(sb, indaddr, size, dir);
|
||||
if (ret)
|
||||
adfs_error(sb, "unable to read directory");
|
||||
else
|
||||
dir->parent_id = adfs_readval(dir->newtail->dirparent, 3);
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
static int
|
||||
adfs_f_setpos(struct adfs_dir *dir, unsigned int fpos)
|
||||
{
|
||||
|
|
Loading…
Reference in New Issue