fs: Use a new torrent file reader per handled request

[btrtrc.git] / fs / file_handle.go

diff --git a/fs/file_handle.go b/fs/file_handle.go
index 0ba6d2e8decb81e12e6252f2ea92ebf9a583a389..2a8fe2ccad5f6a96cefa24b5db687bec26a520e1 100644 (file)
--- a/fs/file_handle.go
+++ b/fs/file_handle.go
@@ -3,18 +3,17 @@ package torrentfs
 import (
        "context"
        "io"
-       "log"
 
-       "bazil.org/fuse"
-       "bazil.org/fuse/fs"
-       "github.com/anacrolix/missinggo"
+       "github.com/anacrolix/fuse"
+       "github.com/anacrolix/fuse/fs"
+       "github.com/anacrolix/missinggo/v2"
 
        "github.com/anacrolix/torrent"
 )
 
 type fileHandle struct {
        fn fileNode
-       r  torrent.Reader
+       tf *torrent.File
 }
 
 var _ interface {
@@ -27,7 +26,8 @@ func (me fileHandle) Read(ctx context.Context, req *fuse.ReadRequest, resp *fuse
        if req.Dir {
                panic("read on directory")
        }
-       r := me.r
+       r := me.tf.NewReader()
+       defer r.Close()
        pos, err := r.Seek(req.Offset, io.SeekStart)
        if err != nil {
                panic(err)
@@ -47,8 +47,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)
@@ -78,5 +82,5 @@ func (me fileHandle) Read(ctx context.Context, req *fuse.ReadRequest, resp *fuse
 }
 
 func (me fileHandle) Release(context.Context, *fuse.ReleaseRequest) error {
-       return me.r.Close()
+       return nil
 }