-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Skip ftruncate_bug76422 if disk_free_space() <= 4GB #5283
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Followup to php#5043 In the middle of running this test with TESTS=-j9, run-tests.php stopped the test suite because the partition started with 3.5GB of free disk space and new test files couldn't be created. Reclaiming the disk space of that file also took a while.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
The build failure in php-src for Mac is unrelated, it's related to compiling the recent changes in the opcache jit
|
were you running this test on FAT32? i can see this being a problem on filesystems that does not support sparse files, but i thought all modern filesystems (even the Apple/MacOS ones) supported sparse files? |
oh, apparently Apple's FS supports sparse files, but only since 2017, so if you have a 2016 MacOS/iOS system, you don't have sparse file support.. guess that's what's happened here? (by comparison, Linux has supported sparse files since at least 1993 with the introduction of ext2) |
@divinity76 the tests were run on an ecryptfs filesystem, which is a filesystem which apparently doesn't efficiently support ftruncate - https://askubuntu.com/questions/100752/how-does-ecryptfs-impact-harddisk-performance |
This test failed when the free disk space is close to 2GB. I see the file size in the output as 0. PHP has to save the full file contents to disk (the path is in `$_FILES`) Related to phpGH-5283
This test failed when the free disk space is close to 2.15GB. I see the file size in the .out file as 0. PHP has to save the full file contents to disk (the path is in `$_FILES`) Related to phpGH-5283
Followup to #5043
In the middle of running this test with TESTS=-j9,
run-tests.php stopped the test suite because the partition started
with 3.5GB of free disk space and new test files couldn't be created.
Reclaiming the disk space of that file also took a while.