From cbdc2b8263df368bc5a1978e99a39228f30c81e6 Mon Sep 17 00:00:00 2001 From: Matt Joiner Date: Mon, 22 Feb 2021 10:10:43 +1100 Subject: [PATCH] Comment and log change on torrentfs behaviour change --- fs/file_handle.go | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/fs/file_handle.go b/fs/file_handle.go index 0ba6d2e8..64aeebef 100644 --- a/fs/file_handle.go +++ b/fs/file_handle.go @@ -3,7 +3,6 @@ package torrentfs import ( "context" "io" - "log" "bazil.org/fuse" "bazil.org/fuse/fs" @@ -47,8 +46,12 @@ func (me fileHandle) Read(ctx context.Context, req *fuse.ReadRequest, resp *fuse me.fn.FS.mu.Unlock() var n int r := missinggo.ContextedReader{r, ctx} - log.Printf("reading %v bytes at %v", len(resp.Data), req.Offset) + //log.Printf("reading %v bytes at %v", len(resp.Data), req.Offset) if true { + // A user reported on that on freebsd 12.2, the system requires that reads are + // completely filled. Their system only asks for 64KiB at a time. I've seen systems that + // can demand up to 16MiB at a time, so this gets tricky. For now, I'll restore the old + // behaviour from before 2a7352a, which nobody reported problems with. n, readErr = io.ReadFull(r, resp.Data) } else { n, readErr = r.Read(resp.Data)