]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
ocfs2: Make ocfs2_get_quota_block() consistent with ocfs2_read_quota_block()
authorJan Kara <jack@suse.cz>
Tue, 25 Nov 2008 14:31:29 +0000 (15:31 +0100)
committerMark Fasheh <mfasheh@suse.com>
Mon, 5 Jan 2009 16:40:25 +0000 (08:40 -0800)
Make function return error status and not buffer pointer so that it's
consistent with ocfs2_read_quota_block().

Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: Mark Fasheh <mfasheh@suse.com>
fs/ocfs2/quota_global.c

index 8fceb0c49b3e81987d6e75fdb50167d158f83a72..e527ec6e013379b923ca2efcd94438596c440d97 100644 (file)
@@ -104,26 +104,25 @@ int ocfs2_read_quota_block(struct inode *inode, u64 v_block,
        return rc;
 }
 
-static struct buffer_head *ocfs2_get_quota_block(struct inode *inode,
-                                                int block, int *err)
+static int ocfs2_get_quota_block(struct inode *inode, int block,
+                                struct buffer_head **bh)
 {
        u64 pblock, pcount;
-       struct buffer_head *bh;
+       int err;
 
        down_read(&OCFS2_I(inode)->ip_alloc_sem);
-       *err = ocfs2_extent_map_get_blocks(inode, block, &pblock, &pcount,
-                                          NULL);
+       err = ocfs2_extent_map_get_blocks(inode, block, &pblock, &pcount, NULL);
        up_read(&OCFS2_I(inode)->ip_alloc_sem);
-       if (*err) {
-               mlog_errno(*err);
-               return NULL;
+       if (err) {
+               mlog_errno(err);
+               return err;
        }
-       bh = sb_getblk(inode->i_sb, pblock);
-       if (!bh) {
-               *err = -EIO;
-               mlog_errno(*err);
+       *bh = sb_getblk(inode->i_sb, pblock);
+       if (!*bh) {
+               err = -EIO;
+               mlog_errno(err);
        }
-       return bh;
+       return err;;
 }
 
 /* Read data from global quotafile - avoid pagecache and such because we cannot
@@ -209,7 +208,7 @@ ssize_t ocfs2_quota_write(struct super_block *sb, int type,
                err = ocfs2_read_quota_block(gqinode, blk, &bh);
                ja_type = OCFS2_JOURNAL_ACCESS_WRITE;
        } else {
-               bh = ocfs2_get_quota_block(gqinode, blk, &err);
+               err = ocfs2_get_quota_block(gqinode, blk, &bh);
                ja_type = OCFS2_JOURNAL_ACCESS_CREATE;
        }
        if (err) {