Commit ad650f5b authored by Dave Chinner's avatar Dave Chinner Committed by Ben Myers

xfs: fallback to vmalloc for large buffers in xfs_attrmulti_attr_get

xfsdump uses for a large buffer for extended attributes, which has a
kmalloc'd shadow buffer in the kernel. This can fail after the
system has been running for some time as it is a high order
allocation. Add a fallback to vmalloc so that it doesn't require
contiguous memory and so won't randomly fail while xfsdump is
running.
Signed-off-by: default avatarDave Chinner <dchinner@redhat.com>
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarMark Tinguely <tinguely@sgi.com>
Signed-off-by: default avatarBen Myers <bpm@sgi.com>
parent 6eb24660
...@@ -450,9 +450,12 @@ xfs_attrmulti_attr_get( ...@@ -450,9 +450,12 @@ xfs_attrmulti_attr_get(
if (*len > XATTR_SIZE_MAX) if (*len > XATTR_SIZE_MAX)
return EINVAL; return EINVAL;
kbuf = kmalloc(*len, GFP_KERNEL); kbuf = kmem_zalloc(*len, KM_SLEEP | KM_MAYFAIL);
if (!kbuf) {
kbuf = kmem_zalloc_large(*len);
if (!kbuf) if (!kbuf)
return ENOMEM; return ENOMEM;
}
error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags); error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags);
if (error) if (error)
...@@ -462,7 +465,10 @@ xfs_attrmulti_attr_get( ...@@ -462,7 +465,10 @@ xfs_attrmulti_attr_get(
error = EFAULT; error = EFAULT;
out_kfree: out_kfree:
kfree(kbuf); if (is_vmalloc_addr(kbuf))
kmem_free_large(kbuf);
else
kmem_free(kbuf);
return error; return error;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment