From 659f28b8552e2635b2bc4566e525f1fe098204da Mon Sep 17 00:00:00 2001 From: chrislu Date: Mon, 5 Dec 2022 23:19:38 -0800 Subject: [PATCH] simple refactoring --- weed/mount/weedfs_file_sync.go | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/weed/mount/weedfs_file_sync.go b/weed/mount/weedfs_file_sync.go index a08a37f9c..7b7c66680 100644 --- a/weed/mount/weedfs_file_sync.go +++ b/weed/mount/weedfs_file_sync.go @@ -55,9 +55,6 @@ func (wfs *WFS) Flush(cancel <-chan struct{}, in *fuse.FlushIn) fuse.Status { return fuse.ENOENT } - fh.orderedMutex.Acquire(context.Background(), 1) - defer fh.orderedMutex.Release(1) - return wfs.doFlush(fh, in.Uid, in.Gid) } @@ -87,14 +84,14 @@ func (wfs *WFS) Fsync(cancel <-chan struct{}, in *fuse.FsyncIn) (code fuse.Statu return fuse.ENOENT } - fh.orderedMutex.Acquire(context.Background(), 1) - defer fh.orderedMutex.Release(1) - return wfs.doFlush(fh, in.Uid, in.Gid) } func (wfs *WFS) doFlush(fh *FileHandle, uid, gid uint32) fuse.Status { + fh.orderedMutex.Acquire(context.Background(), 1) + defer fh.orderedMutex.Release(1) + // flush works at fh level fileFullPath := fh.FullPath() dir, name := fileFullPath.DirAndName() @@ -117,7 +114,6 @@ func (wfs *WFS) doFlush(fh *FileHandle, uid, gid uint32) fuse.Status { } err := wfs.WithFilerClient(false, func(client filer_pb.SeaweedFilerClient) error { - fh.entryLock.Lock() defer fh.entryLock.Unlock()