]> granicus.if.org Git - llvm/commitdiff
[Support] On AIX, Check ENOTSUP on posix_fallocate instead of EOPNOTSUPP
authorHubert Tong <hubert.reinterpretcast@gmail.com>
Thu, 4 Apr 2019 00:40:34 +0000 (00:40 +0000)
committerHubert Tong <hubert.reinterpretcast@gmail.com>
Thu, 4 Apr 2019 00:40:34 +0000 (00:40 +0000)
Summary:
`posix_fallocate` can fail if the underlying filesystem does not support
it; and, on AIX, such a failure is reported by a return value of
`ENOTSUP`. The existing code checks only for `EOPNOTSUPP`, which may
share the same value as `ENOTSUP`, but is not required to.

Reviewers: xingxue, sfertile, jasonliu

Reviewed By: xingxue

Subscribers: kristina, jsji, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D60175

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@357662 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Support/Unix/Path.inc

index a02585c3bdc264a9d04d736267d37d646f2030a7..05ccc6cc5e444fd5b0a255d5a69334b76eba3976 100644 (file)
@@ -492,7 +492,12 @@ std::error_code resize_file(int FD, uint64_t Size) {
   // If we have posix_fallocate use it. Unlike ftruncate it always allocates
   // space, so we get an error if the disk is full.
   if (int Err = ::posix_fallocate(FD, 0, Size)) {
-    if (Err != EINVAL && Err != EOPNOTSUPP)
+#ifdef _AIX
+    constexpr int NotSupportedError = ENOTSUP;
+#else
+    constexpr int NotSupportedError = EOPNOTSUPP;
+#endif
+    if (Err != EINVAL && Err != NotSupportedError)
       return std::error_code(Err, std::generic_category());
   }
 #endif