Added support for Max Pages

Signed-off-by: Anagh Kumar Baranwal <6824881+darthShadow@users.noreply.github.com>
This commit is contained in:
Anagh Kumar Baranwal 2020-02-10 10:27:22 +05:30
parent 37c55972e3
commit 6a86ec70ea
12 changed files with 87 additions and 3 deletions

View File

@ -73,6 +73,9 @@ func mountOptions(device string) (options []fuse.MountOption) {
if len(mountlib.ExtraFlags) > 0 { if len(mountlib.ExtraFlags) > 0 {
fs.Errorf(nil, "--fuse-flag not supported with this FUSE backend") fs.Errorf(nil, "--fuse-flag not supported with this FUSE backend")
} }
if vfsflags.Opt.MaxPages != 0 {
options = append(options, fuse.MaxPages(vfsflags.Opt.MaxPages))
}
return options return options
} }

View File

@ -98,6 +98,14 @@ func IntVarP(flags *pflag.FlagSet, p *int, name, shorthand string, value int, us
setDefaultFromEnv(flags, name) setDefaultFromEnv(flags, name)
} }
// Uint16VarP defines a flag which can be overridden by an environment variable
//
// It is a thin wrapper around pflag.Uint16VarP
func Uint16VarP(flags *pflag.FlagSet, p *uint16, name, shorthand string, value uint16, usage string) {
flags.Uint16VarP(p, name, shorthand, value, usage)
setDefaultFromEnv(flags, name)
}
// Uint32VarP defines a flag which can be overridden by an environment variable // Uint32VarP defines a flag which can be overridden by an environment variable
// //
// It is a thin wrapper around pflag.Uint32VarP // It is a thin wrapper around pflag.Uint32VarP

18
vendor/bazil.org/fuse/fuse.go generated vendored
View File

@ -156,7 +156,8 @@ func (e *MountpointDoesNotExistError) Error() string {
// progress. // progress.
func Mount(dir string, options ...MountOption) (*Conn, error) { func Mount(dir string, options ...MountOption) (*Conn, error) {
conf := mountConfig{ conf := mountConfig{
options: make(map[string]string), options: make(map[string]string),
maxPages: 32,
} }
for _, option := range options { for _, option := range options {
if err := option(&conf); err != nil { if err := option(&conf); err != nil {
@ -238,10 +239,20 @@ func initMount(c *Conn, conf *mountConfig) error {
MaxWrite: maxWrite, MaxWrite: maxWrite,
Flags: InitBigWrites | conf.initFlags, Flags: InitBigWrites | conf.initFlags,
} }
setMaxPages(r, s, conf)
r.Respond(s) r.Respond(s)
return nil return nil
} }
func setMaxPages(request *InitRequest, response *InitResponse, conf *mountConfig) {
if (request.Flags & InitMaxPages) == InitMaxPages {
response.Flags |= InitMaxPages
response.MaxPages = conf.maxPages
}
}
// A Request represents a single FUSE request received from the kernel. // A Request represents a single FUSE request received from the kernel.
// Use a type switch to determine the specific kind. // Use a type switch to determine the specific kind.
// A request of unrecognized type will have concrete type *Header. // A request of unrecognized type will have concrete type *Header.
@ -1229,6 +1240,9 @@ type InitResponse struct {
// Maximum size of a single write operation. // Maximum size of a single write operation.
// Linux enforces a minimum of 4 KiB. // Linux enforces a minimum of 4 KiB.
MaxWrite uint32 MaxWrite uint32
// Maximum number of pages in a single write operation.
// Linux enforces a minimum of 32.
MaxPages uint16
} }
func (r *InitResponse) String() string { func (r *InitResponse) String() string {
@ -1244,12 +1258,14 @@ func (r *InitRequest) Respond(resp *InitResponse) {
out.MaxReadahead = resp.MaxReadahead out.MaxReadahead = resp.MaxReadahead
out.Flags = uint32(resp.Flags) out.Flags = uint32(resp.Flags)
out.MaxWrite = resp.MaxWrite out.MaxWrite = resp.MaxWrite
out.MaxPages = resp.MaxPages
// MaxWrite larger than our receive buffer would just lead to // MaxWrite larger than our receive buffer would just lead to
// errors on large writes. // errors on large writes.
if out.MaxWrite > maxWrite { if out.MaxWrite > maxWrite {
out.MaxWrite = maxWrite out.MaxWrite = maxWrite
} }
r.respond(buf) r.respond(buf)
} }

View File

@ -274,6 +274,8 @@ const (
InitWritebackCache InitFlags = 1 << 16 InitWritebackCache InitFlags = 1 << 16
InitNoOpenSupport InitFlags = 1 << 17 InitNoOpenSupport InitFlags = 1 << 17
InitMaxPages InitFlags = 1 << 22 // Linux only
InitCaseSensitive InitFlags = 1 << 29 // OS X only InitCaseSensitive InitFlags = 1 << 29 // OS X only
InitVolRename InitFlags = 1 << 30 // OS X only InitVolRename InitFlags = 1 << 30 // OS X only
InitXtimes InitFlags = 1 << 31 // OS X only InitXtimes InitFlags = 1 << 31 // OS X only
@ -710,6 +712,8 @@ type initOut struct {
Flags uint32 Flags uint32
Unused uint32 Unused uint32
MaxWrite uint32 MaxWrite uint32
_ uint32 // Unused, refers to TimeGran
MaxPages uint16
} }
type interruptIn struct { type interruptIn struct {

View File

@ -4,4 +4,6 @@ package fuse
// //
// Linux 4.2.0 has been observed to cap this value at 128kB // Linux 4.2.0 has been observed to cap this value at 128kB
// (FUSE_MAX_PAGES_PER_REQ=32, 4kB pages). // (FUSE_MAX_PAGES_PER_REQ=32, 4kB pages).
const maxWrite = 128 * 1024 // From Linux 4.20, the cap has been increased to 1MiB
// (FUSE_MAX_PAGES_PER_REQ=256, 4kB pages).
const maxWrite = 1 * 1024 * 1024 // 1 MiB

9
vendor/bazil.org/fuse/options.go generated vendored
View File

@ -16,6 +16,7 @@ type mountConfig struct {
maxReadahead uint32 maxReadahead uint32
initFlags InitFlags initFlags InitFlags
osxfuseLocations []OSXFUSEPaths osxfuseLocations []OSXFUSEPaths
maxPages uint16
} }
func escapeComma(s string) string { func escapeComma(s string) string {
@ -317,3 +318,11 @@ func AllowNonEmptyMount() MountOption {
return nil return nil
} }
} }
// MaxPages enables the configuration of the maximum number of pages
// in the request & response from the kernel.
//
// Linux only. Others ignore this option.
func MaxPages(count uint16) MountOption {
return maxPages(count)
}

View File

@ -38,3 +38,7 @@ func noBrowse(conf *mountConfig) error {
conf.options["nobrowse"] = "" conf.options["nobrowse"] = ""
return nil return nil
} }
func maxPages(count uint16) MountOption {
return dummyOption
}

View File

@ -30,3 +30,7 @@ func exclCreate(conf *mountConfig) error {
func noBrowse(conf *mountConfig) error { func noBrowse(conf *mountConfig) error {
return nil return nil
} }
func maxPages(count uint16) MountOption {
return dummyOption
}

View File

@ -27,3 +27,13 @@ func exclCreate(conf *mountConfig) error {
func noBrowse(conf *mountConfig) error { func noBrowse(conf *mountConfig) error {
return nil return nil
} }
func maxPages(count uint16) MountOption {
return func(conf *mountConfig) error {
if count > 256 || count == 0 {
count = 256
}
conf.maxPages = count
return nil
}
}

View File

@ -55,6 +55,7 @@ var DefaultOpt = Options{
CaseInsensitive: runtime.GOOS == "windows" || runtime.GOOS == "darwin", // default to true on Windows and Mac, false otherwise CaseInsensitive: runtime.GOOS == "windows" || runtime.GOOS == "darwin", // default to true on Windows and Mac, false otherwise
WriteWait: 1000 * time.Millisecond, WriteWait: 1000 * time.Millisecond,
ReadWait: 5 * time.Millisecond, ReadWait: 5 * time.Millisecond,
MaxPages: 0, // Only for Linux
} }
// Node represents either a directory (*Dir) or a file (*File) // Node represents either a directory (*Dir) or a file (*File)
@ -206,6 +207,7 @@ type Options struct {
CaseInsensitive bool CaseInsensitive bool
WriteWait time.Duration // time to wait for in-sequence write WriteWait time.Duration // time to wait for in-sequence write
ReadWait time.Duration // time to wait for in-sequence read ReadWait time.Duration // time to wait for in-sequence read
MaxPages uint16
} }
// New creates a new VFS and root directory. If opt is nil, then // New creates a new VFS and root directory. If opt is nil, then

View File

@ -0,0 +1,22 @@
// +build linux
package vfsflags
import (
"github.com/rclone/rclone/fs/config/flags"
"github.com/spf13/pflag"
"golang.org/x/sys/unix"
)
// add any extra platform specific flags
func platformFlags(flagSet *pflag.FlagSet) {
flags.IntVarP(flagSet, &Opt.Umask, "umask", "", Opt.Umask, "Override the permission bits set by the filesystem.")
Opt.Umask = unix.Umask(0) // read the umask
unix.Umask(Opt.Umask) // set it back to what it was
Opt.UID = uint32(unix.Geteuid())
Opt.GID = uint32(unix.Getegid())
flags.Uint32VarP(flagSet, &Opt.UID, "uid", "", Opt.UID, "Override the uid field set by the filesystem.")
flags.Uint32VarP(flagSet, &Opt.GID, "gid", "", Opt.GID, "Override the gid field set by the filesystem.")
flags.Uint16VarP(flagSet, &Opt.MaxPages, "max-pages", "", Opt.MaxPages, "Set the Max Pages to be passed onto the FUSE library")
}

View File

@ -1,4 +1,4 @@
// +build linux darwin freebsd // +build darwin freebsd
package vfsflags package vfsflags