Explore sendfile() for copyup operations

stanford-rc / fuse-migratefs

Filesystem overlay for transparent, distributed migration of active data across separate storage systems.

GNU General Public License v3.0

40 stars 10 forks source link

Explore sendfile() for copyup operations #4

Closed kcgthb closed 5 years ago

kcgthb commented 5 years ago

Maybe sendfile() could be used instead of read()/write() to make copyup operations more efficient?

giuseppe commented 5 years ago

I've a PR here: https://github.com/containers/fuse-overlayfs/pull/62

thiell commented 5 years ago

Thanks @giuseppe this was super helpful. But I think I found an issue when sendfile returns -1, you then did copied += n which could be a problem. I changed that a bit.

thiell commented 5 years ago

@giuseppe Submitted as PR containers/fuse-overlayfs/pull/70 for review