diff options
author | Christoph Hellwig <hch@infradead.org> | 2010-03-03 09:05:03 -0500 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2010-03-04 18:20:29 -0500 |
commit | b43fa8284d7790d9cca32c9c55e24f29be2fa33b (patch) | |
tree | acb7f205fe2a1ae324e0e11c50f43baaa6b8dfab /fs/jfs | |
parent | 759bfee658beab14af7b357156461d0eb852be2c (diff) |
dquot: cleanup dquot transfer routine
Get rid of the transfer dquot operation - it is now always called from
the filesystem and if a filesystem really needs it's own (which none
currently does) it can just call into it's own routine directly.
Rename the now static low-level dquot_transfer helper to __dquot_transfer
and vfs_dq_transfer to dquot_transfer to have a consistent namespace,
and make the new dquot_transfer return a normal negative errno value
which all callers expect.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/jfs')
-rw-r--r-- | fs/jfs/file.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/jfs/file.c b/fs/jfs/file.c index a4229e49330e..2c201783836f 100644 --- a/fs/jfs/file.c +++ b/fs/jfs/file.c | |||
@@ -100,8 +100,9 @@ int jfs_setattr(struct dentry *dentry, struct iattr *iattr) | |||
100 | 100 | ||
101 | if ((iattr->ia_valid & ATTR_UID && iattr->ia_uid != inode->i_uid) || | 101 | if ((iattr->ia_valid & ATTR_UID && iattr->ia_uid != inode->i_uid) || |
102 | (iattr->ia_valid & ATTR_GID && iattr->ia_gid != inode->i_gid)) { | 102 | (iattr->ia_valid & ATTR_GID && iattr->ia_gid != inode->i_gid)) { |
103 | if (vfs_dq_transfer(inode, iattr)) | 103 | rc = dquot_transfer(inode, iattr); |
104 | return -EDQUOT; | 104 | if (rc) |
105 | return rc; | ||
105 | } | 106 | } |
106 | 107 | ||
107 | rc = inode_setattr(inode, iattr); | 108 | rc = inode_setattr(inode, iattr); |