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
Replace linux_reflink extension module #1294
Conversation
f704f47
to
8e5716c
Compare
Closing in favor of #1296. |
Re-opening this since it has some commentary. |
0666ebb
to
f9d765f
Compare
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.
Some initial comments. Thanks a lot for doing this.
I haven't yet tested it or worked through the calls/compared with docs yet though (I assume Zac is planning on doing the same too).
I'm currently dog fooding it with an world update of ~250 packages on my main system. No glaring issues noticed yet. |
012fdf8
to
464bad7
Compare
9857682
to
3e9a817
Compare
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.
Just saw this one issue, otherwise seems correct to me.
aeac756
to
1db2b58
Compare
3a40acf
to
c22dd63
Compare
Python 3.8 added support for copy_file_range in the os module, so we can just call that directly. Also, we can use the FICLONE ioctl for fast file clones on supported filesystems (btrfs). Signed-off-by: Mike Gilbert <floppym@gentoo.org>
Actually create sparse blocks at the start and end. Check file size before/after copying. Ensure sparse output when _fastcopy succeeds. Signed-off-by: Mike Gilbert <floppym@gentoo.org>
c22dd63
to
0cc3f9e
Compare
Python 3.8 added support for copy_file_range in the os module, so we can just call that directly.
Also, we can use the FICLONE ioctl for fast file clones on supported filesystems (btrfs).
If we think this needs some performance testing, I would appreciate any help with that.