Browse Source

fs/hugetlbfs/inode.c: remove redundant -ENIVAL return from hugetlbfs_setattr()

There is no need to have a local return code set with -EINVAL when both
the conditions following it return error codes appropriately.  Just
remove the redundant one.

Link: http://lkml.kernel.org/r/20170929145444.17611-1-khandual@linux.vnet.ibm.com
Signed-off-by: Anshuman Khandual <khandual@linux.vnet.ibm.com>
Reviewed-by: Mike Kravetz <mike.kravetz@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Anshuman Khandual 7 years ago
parent
commit
007ab7b49a
1 changed files with 0 additions and 1 deletions
  1. 0 1
      fs/hugetlbfs/inode.c

+ 0 - 1
fs/hugetlbfs/inode.c

@@ -668,7 +668,6 @@ static int hugetlbfs_setattr(struct dentry *dentry, struct iattr *attr)
 		return error;
 		return error;
 
 
 	if (ia_valid & ATTR_SIZE) {
 	if (ia_valid & ATTR_SIZE) {
-		error = -EINVAL;
 		if (attr->ia_size & ~huge_page_mask(h))
 		if (attr->ia_size & ~huge_page_mask(h))
 			return -EINVAL;
 			return -EINVAL;
 		error = hugetlb_vmtruncate(inode, attr->ia_size);
 		error = hugetlb_vmtruncate(inode, attr->ia_size);