From: Gao Xiang <gaoxian...@huawei.com>

As Christoph said [1], ".. and save one
level of indentation."

[1] https://lore.kernel.org/r/20190829102426.ge20...@infradead.org/
Reported-by: Christoph Hellwig <h...@infradead.org>
Signed-off-by: Gao Xiang <gaoxian...@huawei.com>
---
 fs/erofs/inode.c | 65 ++++++++++++++++++++++++------------------------
 1 file changed, 33 insertions(+), 32 deletions(-)

diff --git a/fs/erofs/inode.c b/fs/erofs/inode.c
index fcc16d2d10cb..ee39b32bb911 100644
--- a/fs/erofs/inode.c
+++ b/fs/erofs/inode.c
@@ -188,41 +188,42 @@ static int erofs_fill_inode(struct inode *inode, int 
isdir)
        data = page_address(page);
 
        err = erofs_read_inode(inode, data + ofs);
-       if (!err) {
-               /* setup the new inode */
-               switch (inode->i_mode & S_IFMT) {
-               case S_IFREG:
-                       inode->i_op = &erofs_generic_iops;
-                       inode->i_fop = &generic_ro_fops;
-                       break;
-               case S_IFDIR:
-                       inode->i_op = &erofs_dir_iops;
-                       inode->i_fop = &erofs_dir_fops;
-                       break;
-               case S_IFLNK:
-                       err = erofs_fill_symlink(inode, data, ofs);
-                       if (err)
-                               goto out_unlock;
-                       inode_nohighmem(inode);
-                       break;
-               case S_IFCHR:
-               case S_IFBLK:
-               case S_IFIFO:
-               case S_IFSOCK:
-                       inode->i_op = &erofs_generic_iops;
-                       init_special_inode(inode, inode->i_mode, inode->i_rdev);
-                       goto out_unlock;
-               default:
-                       err = -EFSCORRUPTED;
+       if (err)
+               goto out_unlock;
+
+       /* setup the new inode */
+       switch (inode->i_mode & S_IFMT) {
+       case S_IFREG:
+               inode->i_op = &erofs_generic_iops;
+               inode->i_fop = &generic_ro_fops;
+               break;
+       case S_IFDIR:
+               inode->i_op = &erofs_dir_iops;
+               inode->i_fop = &erofs_dir_fops;
+               break;
+       case S_IFLNK:
+               err = erofs_fill_symlink(inode, data, ofs);
+               if (err)
                        goto out_unlock;
-               }
+               inode_nohighmem(inode);
+               break;
+       case S_IFCHR:
+       case S_IFBLK:
+       case S_IFIFO:
+       case S_IFSOCK:
+               inode->i_op = &erofs_generic_iops;
+               init_special_inode(inode, inode->i_mode, inode->i_rdev);
+               goto out_unlock;
+       default:
+               err = -EFSCORRUPTED;
+               goto out_unlock;
+       }
 
-               if (erofs_inode_is_data_compressed(vi->datamode)) {
-                       err = z_erofs_fill_inode(inode);
-                       goto out_unlock;
-               }
-               inode->i_mapping->a_ops = &erofs_raw_access_aops;
+       if (erofs_inode_is_data_compressed(vi->datamode)) {
+               err = z_erofs_fill_inode(inode);
+               goto out_unlock;
        }
+       inode->i_mapping->a_ops = &erofs_raw_access_aops;
 
 out_unlock:
        unlock_page(page);
-- 
2.17.1

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to