Merge pull request #5114 from pks-t/pks/bigfile-refactoring Removal of `p_fallocate`
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246
diff --git a/src/posix.c b/src/posix.c
index 32ca704..bffe02e 100644
--- a/src/posix.c
+++ b/src/posix.c
@@ -155,44 +155,6 @@ int p_rename(const char *from, const char *to)
return -1;
}
-int p_fallocate(int fd, off_t offset, off_t len)
-{
-#if defined (__APPLE__) || (defined (__NetBSD__) && __NetBSD_Version__ < 700000000)
- fstore_t prealloc;
- struct stat st;
- size_t newsize;
- int error;
-
- if ((error = p_fstat(fd, &st)) < 0)
- return error;
-
- if (git__add_sizet_overflow(&newsize, offset, len)) {
- errno = EINVAL;
- return -1;
- }
-
- if (newsize < (unsigned long long)st.st_size)
- return 0;
-
- memset(&prealloc, 0, sizeof(prealloc));
- prealloc.fst_flags = F_ALLOCATEALL;
- prealloc.fst_posmode = F_PEOFPOSMODE;
- prealloc.fst_offset = offset;
- prealloc.fst_length = len;
-
- /*
- * fcntl will often error when the file already exists; ignore
- * this error since ftruncate will also resize the file (although
- * likely slower).
- */
- fcntl(fd, F_PREALLOCATE, &prealloc);
-
- return ftruncate(fd, (offset + len));
-#else
- return posix_fallocate(fd, offset, len);
-#endif
-}
-
#endif /* GIT_WIN32 */
ssize_t p_read(git_file fd, void *buf, size_t cnt)
diff --git a/src/posix.h b/src/posix.h
index 0119b6b..2934f24 100644
--- a/src/posix.h
+++ b/src/posix.h
@@ -115,7 +115,6 @@ extern int p_open(const char *path, int flags, ...);
extern int p_creat(const char *path, mode_t mode);
extern int p_getcwd(char *buffer_out, size_t size);
extern int p_rename(const char *from, const char *to);
-extern int p_fallocate(int fd, off_t offset, off_t len);
extern int git__page_size(size_t *page_size);
extern int git__mmap_alignment(size_t *page_size);
diff --git a/src/win32/posix_w32.c b/src/win32/posix_w32.c
index c082dba..91e1643 100644
--- a/src/win32/posix_w32.c
+++ b/src/win32/posix_w32.c
@@ -525,58 +525,6 @@ int p_creat(const char *path, mode_t mode)
return p_open(path, O_WRONLY | O_CREAT | O_TRUNC, mode);
}
-int p_fallocate(int fd, off_t offset, off_t len)
-{
- HANDLE fh = (HANDLE)_get_osfhandle(fd);
- LARGE_INTEGER zero, position, oldsize, newsize;
- size_t size;
-
- if (fh == INVALID_HANDLE_VALUE) {
- errno = EBADF;
- return -1;
- }
-
- if (offset < 0 || len <= 0) {
- errno = EINVAL;
- return -1;
- }
-
- if (git__add_sizet_overflow(&size, offset, len)) {
- errno = EINVAL;
- return -1;
- }
-
- zero.u.LowPart = 0;
- zero.u.HighPart = 0;
-
- newsize.u.LowPart = (size & 0xffffffff);
-
-#if (SIZE_MAX > UINT32_MAX)
- newsize.u.HighPart = size >> 32;
-#else
- newsize.u.HighPart = 0;
-#endif
-
- if (!GetFileSizeEx(fh, &oldsize)) {
- set_errno();
- return -1;
- }
-
- /* POSIX emulation: attempting to shrink the file is ignored */
- if (oldsize.QuadPart >= newsize.QuadPart)
- return 0;
-
- if (!SetFilePointerEx(fh, zero, &position, FILE_CURRENT) ||
- !SetFilePointerEx(fh, newsize, NULL, FILE_BEGIN) ||
- !SetEndOfFile(fh) ||
- !SetFilePointerEx(fh, position, 0, FILE_BEGIN)) {
- set_errno();
- return -1;
- }
-
- return 0;
-}
-
int p_utimes(const char *path, const struct p_timeval times[2])
{
git_win32_path wpath;
diff --git a/tests/core/posix.c b/tests/core/posix.c
index 2b60077..a459b26 100644
--- a/tests/core/posix.c
+++ b/tests/core/posix.c
@@ -35,11 +35,6 @@ void test_core_posix__initialize(void)
#endif
}
-void test_core_posix__cleanup(void)
-{
- p_unlink("fallocate_test");
-}
-
static bool supports_ipv6(void)
{
#ifdef GIT_WIN32
@@ -268,39 +263,3 @@ void test_core_posix__p_regcomp_compile_userdiff_regexps(void)
cl_assert(!error);
}
}
-
-void test_core_posix__fallocate(void)
-{
- int fd;
- struct stat st;
-
- /* fallocate a new file succeeds */
- cl_must_pass(fd = p_open("fallocate_test", O_RDWR|O_CREAT, 0666));
- cl_must_pass(p_fallocate(fd, 0, 42));
- cl_must_pass(p_fstat(fd, &st));
- cl_assert_equal_i(42, st.st_size);
- p_close(fd);
-
- /* fallocate an existing file succeeds */
- cl_must_pass(fd = p_open("fallocate_test", O_RDWR, 0666));
- cl_must_pass(p_fallocate(fd, 90, 9));
- cl_must_pass(p_fstat(fd, &st));
- cl_assert_equal_i(99, st.st_size);
- p_close(fd);
-
- /* fallocate doesn't shrink */
- cl_must_pass(fd = p_open("fallocate_test", O_RDWR, 0666));
- cl_must_pass(p_fallocate(fd, 0, 14));
- cl_must_pass(p_fstat(fd, &st));
- cl_assert_equal_i(99, st.st_size);
- p_close(fd);
-
- /* fallocate doesn't move the cursor */
- cl_must_pass(fd = p_open("fallocate_test", O_RDWR, 0666));
- cl_must_pass(p_fallocate(fd, 0, 100));
- cl_assert_equal_i(0, p_lseek(fd, 0, SEEK_CUR));
- cl_must_pass(p_lseek(fd, 42, SEEK_SET));
- cl_must_pass(p_fallocate(fd, 0, 200));
- cl_assert_equal_i(42, p_lseek(fd, 0, SEEK_CUR));
- p_close(fd);
-}
diff --git a/tests/object/tree/read.c b/tests/object/tree/read.c
index 1ba058f..a20ec38 100644
--- a/tests/object/tree/read.c
+++ b/tests/object/tree/read.c
@@ -79,46 +79,36 @@ void test_object_tree_read__two(void)
void test_object_tree_read__largefile(void)
{
- git_reference *ref;
+ const git_tree_entry *entry;
+ git_index_entry ie;
git_commit *commit;
+ git_object *object;
+ git_index *index;
git_tree *tree;
git_oid oid;
- const git_tree_entry *entry;
- git_object *object;
- git_buf file = GIT_BUF_INIT;
- int fd;
- git_index *idx;
+ char *buf;
if (!cl_is_env_set("GITTEST_INVASIVE_FS_SIZE"))
cl_skip();
- cl_git_pass(git_reference_lookup(&ref, g_repo, "refs/heads/master"));
- cl_git_pass(git_repository_index(&idx, g_repo));
+ cl_assert(buf = git__calloc(1, BIGFILE_SIZE));
- cl_git_pass(git_buf_puts(&file, git_repository_workdir(g_repo)));
- cl_git_pass(git_buf_joinpath(&file, file.ptr, BIGFILE));
+ memset(&ie, 0, sizeof(ie));
+ ie.mode = GIT_FILEMODE_BLOB;
+ ie.path = BIGFILE;
- fd = p_open(git_buf_cstr(&file), O_CREAT|O_RDWR, 0644);
- cl_assert_(fd >= 0, "invalid file descriptor");
-
- cl_must_pass(p_fallocate(fd, 0, BIGFILE_SIZE));
- cl_must_pass(p_close(fd));
-
- cl_git_pass(git_index_add_bypath(idx, BIGFILE));
- cl_repo_commit_from_index(&oid, g_repo, NULL, 0, "bigfile");
+ cl_git_pass(git_repository_index(&index, g_repo));
+ cl_git_pass(git_index_add_frombuffer(index, &ie, buf, BIGFILE_SIZE));
+ cl_repo_commit_from_index(&oid, g_repo, NULL, 0, BIGFILE);
cl_git_pass(git_commit_lookup(&commit, g_repo, &oid));
cl_git_pass(git_commit_tree(&tree, commit));
-
- entry = git_tree_entry_byname(tree, BIGFILE);
- cl_assert_(entry, "entry was NULL");
-
+ cl_assert(entry = git_tree_entry_byname(tree, BIGFILE));
cl_git_pass(git_tree_entry_to_object(&object, g_repo, entry));
- git_buf_dispose(&file);
git_object_free(object);
git_tree_free(tree);
- git_index_free(idx);
+ git_index_free(index);
git_commit_free(commit);
- git_reference_free(ref);
+ git__free(buf);
}