Re: [GIT PULL] fscache rewrite -- please drop for now

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 8/10/20 12:06 PM, Jeff Layton wrote:
On Mon, 2020-08-10 at 12:35 -0400, David Wysochanski wrote:
On Mon, Aug 10, 2020 at 11:48 AM David Howells <dhowells@xxxxxxxxxx> wrote:
Steve French <smfrench@xxxxxxxxx> wrote:

cifs.ko also can set rsize quite small (even 1K for example, although
that will be more than 10x slower than the default 4MB so hopefully no
one is crazy enough to do that).
You can set rsize < PAGE_SIZE?

I can't imagine an SMB3 server negotiating an rsize or wsize smaller than
64K in today's world (and typical is 1MB to 8MB) but the user can specify a
much smaller rsize on mount.  If 64K is an adequate minimum, we could change
the cifs mount option parsing to require a certain minimum rsize if fscache
is selected.
I've borrowed the 256K granule size used by various AFS implementations for
the moment.  A 512-byte xattr can thus hold a bitmap covering 1G of file
space.


Is it possible to make the granule size configurable, then reject a
registration if the size is too small or not a power of 2?  Then a
netfs using the API could try to set equal to rsize, and then error
out with a message if the registration was rejected.

...or maybe we should just make fscache incompatible with an
rsize that isn't an even multiple of 256k? You need to set mount options
for both, typically, so it would be fairly trivial to check this at
mount time, I'd think.


Yes - if fscache is specified on mount it would be easy to round rsize up (or down), at least for cifs.ko (perhaps simply in the mount.cifs helper so a warning could be returned to the user) to whatever boundary you prefer in fscache.   The default of 4MB (or 1MB for mounts to some older servers) should be fine.  Similarly if the user requested the default but the server negotiated an unusual size, not a multiple of 256K, we could round try to round it down if possible (or fail the mount if not possible to round it down to 256K).




[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux