diff options
| author | Li Zefan <lizf@cn.fujitsu.com> | 2008-05-28 08:58:56 -0500 |
|---|---|---|
| committer | Dave Kleikamp <shaggy@linux.vnet.ibm.com> | 2008-05-28 08:58:56 -0500 |
| commit | 3c65e8743bf8b5cf0f90e8d767bf1d8b50c14c76 (patch) | |
| tree | b475d031cd1abaf1a2ffef99968c00950ae2c431 /fs/jfs | |
| parent | 6536d2891ba2c4e837ba8478dc13bb173ed24a23 (diff) | |
| download | lwn-3c65e8743bf8b5cf0f90e8d767bf1d8b50c14c76.tar.gz lwn-3c65e8743bf8b5cf0f90e8d767bf1d8b50c14c76.zip | |
JFS: diAlloc() should return -EIO rather than EIO
The comment above the function says one of its return value is -EIO,
and also the caller of diAlloc() checks for -EIO:
struct inode *ialloc(struct inode *parent, umode_t mode)
{
...
rc = diAlloc(parent, S_ISDIR(mode), inode);
if (rc) {
jfs_warn("ialloc: diAlloc returned %d!", rc);
if (rc == -EIO)
make_bad_inode(inode);
...
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Dave Kleikamp <shaggy@linux.vnet.ibm.com>
Diffstat (limited to 'fs/jfs')
| -rw-r--r-- | fs/jfs/jfs_imap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/jfs/jfs_imap.c b/fs/jfs/jfs_imap.c index 734ec916beaf..d6363d8309d0 100644 --- a/fs/jfs/jfs_imap.c +++ b/fs/jfs/jfs_imap.c @@ -1520,7 +1520,7 @@ int diAlloc(struct inode *pip, bool dir, struct inode *ip) jfs_error(ip->i_sb, "diAlloc: can't find free bit " "in wmap"); - return EIO; + return -EIO; } /* determine the inode number within the |