| Message ID | 163456871794.2614702.15398637170877934146.stgit@warthog.procyon.org.uk (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | fscache: Rewrite index API and management system | expand |
On Mon, 2021-10-18 at 15:51 +0100, David Howells wrote: > Temporarily disable the use of fscache by the various Linux network > filesystems, apart from afs, so that the fscache core can be rewritten. > > Signed-off-by: David Howells <dhowells@redhat.com> > cc: Dave Wysochanski <dwysocha@redhat.com> > cc: Trond Myklebust <trond.myklebust@hammerspace.com> > cc: Anna Schumaker <anna.schumaker@netapp.com> > cc: linux-nfs@vger.kernel.org > cc: Jeff Layton <jlayton@kernel.org> > cc: Ilya Dryomov <idryomov@gmail.com> > cc: ceph-devel@vger.kernel.org > cc: Steve French <sfrench@samba.org> > cc: linux-cifs@vger.kernel.org > cc: Eric Van Hensbergen <ericvh@gmail.com> > cc: Latchesar Ionkov <lucho@ionkov.net> > cc: Dominique Martinet <asmadeus@codewreck.org> > cc: v9fs-developer@lists.sourceforge.net > --- > > fs/9p/Kconfig | 2 +- > fs/ceph/Kconfig | 2 +- > fs/cifs/Kconfig | 2 +- > fs/fscache/Kconfig | 4 ++++ > fs/nfs/Kconfig | 2 +- > 5 files changed, 8 insertions(+), 4 deletions(-) > > diff --git a/fs/9p/Kconfig b/fs/9p/Kconfig > index d7bc93447c85..b11c15c30bac 100644 > --- a/fs/9p/Kconfig > +++ b/fs/9p/Kconfig > @@ -14,7 +14,7 @@ config 9P_FS > if 9P_FS > config 9P_FSCACHE > bool "Enable 9P client caching support" > - depends on 9P_FS=m && FSCACHE || 9P_FS=y && FSCACHE=y > + depends on 9P_FS=m && FSCACHE_OLD || 9P_FS=y && FSCACHE_OLD=y > help > Choose Y here to enable persistent, read-only local > caching support for 9p clients using FS-Cache > diff --git a/fs/ceph/Kconfig b/fs/ceph/Kconfig > index 94df854147d3..77ad452337ee 100644 > --- a/fs/ceph/Kconfig > +++ b/fs/ceph/Kconfig > @@ -21,7 +21,7 @@ config CEPH_FS > if CEPH_FS > config CEPH_FSCACHE > bool "Enable Ceph client caching support" > - depends on CEPH_FS=m && FSCACHE || CEPH_FS=y && FSCACHE=y > + depends on CEPH_FS=m && FSCACHE_OLD || CEPH_FS=y && FSCACHE_OLD=y > help > Choose Y here to enable persistent, read-only local > caching support for Ceph clients using FS-Cache > diff --git a/fs/cifs/Kconfig b/fs/cifs/Kconfig > index 3b7e3b9e4fd2..c5477abbcff0 100644 > --- a/fs/cifs/Kconfig > +++ b/fs/cifs/Kconfig > @@ -188,7 +188,7 @@ config CIFS_SMB_DIRECT > > config CIFS_FSCACHE > bool "Provide CIFS client caching support" > - depends on CIFS=m && FSCACHE || CIFS=y && FSCACHE=y > + depends on CIFS=m && FSCACHE_OLD || CIFS=y && FSCACHE_OLD=y > help > Makes CIFS FS-Cache capable. Say Y here if you want your CIFS data > to be cached locally on disk through the general filesystem cache > diff --git a/fs/fscache/Kconfig b/fs/fscache/Kconfig > index b313a978ae0a..7850de3bdee0 100644 > --- a/fs/fscache/Kconfig > +++ b/fs/fscache/Kconfig > @@ -38,3 +38,7 @@ config FSCACHE_DEBUG > enabled by setting bits in /sys/modules/fscache/parameter/debug. > > See Documentation/filesystems/caching/fscache.rst for more information. > + > +config FSCACHE_OLD > + bool > + default n > diff --git a/fs/nfs/Kconfig b/fs/nfs/Kconfig > index 14a72224b657..a8b73c90aa00 100644 > --- a/fs/nfs/Kconfig > +++ b/fs/nfs/Kconfig > @@ -170,7 +170,7 @@ config ROOT_NFS > > config NFS_FSCACHE > bool "Provide NFS client caching support" > - depends on NFS_FS=m && FSCACHE || NFS_FS=y && FSCACHE=y > + depends on NFS_FS=m && FSCACHE_OLD || NFS_FS=y && FSCACHE_OLD=y > help > Say Y here if you want NFS data to be cached locally on disc through > the general filesystem cache manager > > The typical way to do this would be to rebrand the existing FSCACHE Kconfig symbols into FSCACHE_OLD and then build the new fscache structure such that it exists in parallel with the old. You'd then just drop the old infrastructure once all of the fs's are converted to the new. You could even make them conflict with one another in Kconfig too, so that only one could be built in during the transition period if supporting both at runtime is too difficult. This approach of disabling everything is much more of an all-or-nothing affair. It may mean less "churn" overall, but it seems less "nice" because you have an interval of commits where fscache is non-functional. I'm not necessarily opposed to this approach, but I'd like to better understand why doing it this way was preferred.
Jeff Layton <jlayton@kernel.org> wrote: > The typical way to do this would be to rebrand the existing FSCACHE > Kconfig symbols into FSCACHE_OLD and then build the new fscache > structure such that it exists in parallel with the old. That, there, is nub of the problem. You can't have parallel cachefiles drivers: There's a single userspace interface (/dev/cachefiles) and only one driver can register it. You would need to decide at compile time whether you want the converted or the unconverted network filesystems to be cached. > You'd then just drop the old infrastructure once all of the fs's are > converted to the new. You could even make them conflict with one another in > Kconfig too, so that only one could be built in during the transition period > if supporting both at runtime is too difficult. > > This approach of disabling everything is much more of an all-or-nothing > affair. It may mean less "churn" overall, but it seems less "nice" > because you have an interval of commits where fscache is non-functional. > > I'm not necessarily opposed to this approach, but I'd like to better > understand why doing it this way was preferred. I'm trying to avoid adding two parallel drivers, but change in place so that I can test parts of it as I go along. David
diff --git a/fs/9p/Kconfig b/fs/9p/Kconfig index d7bc93447c85..b11c15c30bac 100644 --- a/fs/9p/Kconfig +++ b/fs/9p/Kconfig @@ -14,7 +14,7 @@ config 9P_FS if 9P_FS config 9P_FSCACHE bool "Enable 9P client caching support" - depends on 9P_FS=m && FSCACHE || 9P_FS=y && FSCACHE=y + depends on 9P_FS=m && FSCACHE_OLD || 9P_FS=y && FSCACHE_OLD=y help Choose Y here to enable persistent, read-only local caching support for 9p clients using FS-Cache diff --git a/fs/ceph/Kconfig b/fs/ceph/Kconfig index 94df854147d3..77ad452337ee 100644 --- a/fs/ceph/Kconfig +++ b/fs/ceph/Kconfig @@ -21,7 +21,7 @@ config CEPH_FS if CEPH_FS config CEPH_FSCACHE bool "Enable Ceph client caching support" - depends on CEPH_FS=m && FSCACHE || CEPH_FS=y && FSCACHE=y + depends on CEPH_FS=m && FSCACHE_OLD || CEPH_FS=y && FSCACHE_OLD=y help Choose Y here to enable persistent, read-only local caching support for Ceph clients using FS-Cache diff --git a/fs/cifs/Kconfig b/fs/cifs/Kconfig index 3b7e3b9e4fd2..c5477abbcff0 100644 --- a/fs/cifs/Kconfig +++ b/fs/cifs/Kconfig @@ -188,7 +188,7 @@ config CIFS_SMB_DIRECT config CIFS_FSCACHE bool "Provide CIFS client caching support" - depends on CIFS=m && FSCACHE || CIFS=y && FSCACHE=y + depends on CIFS=m && FSCACHE_OLD || CIFS=y && FSCACHE_OLD=y help Makes CIFS FS-Cache capable. Say Y here if you want your CIFS data to be cached locally on disk through the general filesystem cache diff --git a/fs/fscache/Kconfig b/fs/fscache/Kconfig index b313a978ae0a..7850de3bdee0 100644 --- a/fs/fscache/Kconfig +++ b/fs/fscache/Kconfig @@ -38,3 +38,7 @@ config FSCACHE_DEBUG enabled by setting bits in /sys/modules/fscache/parameter/debug. See Documentation/filesystems/caching/fscache.rst for more information. + +config FSCACHE_OLD + bool + default n diff --git a/fs/nfs/Kconfig b/fs/nfs/Kconfig index 14a72224b657..a8b73c90aa00 100644 --- a/fs/nfs/Kconfig +++ b/fs/nfs/Kconfig @@ -170,7 +170,7 @@ config ROOT_NFS config NFS_FSCACHE bool "Provide NFS client caching support" - depends on NFS_FS=m && FSCACHE || NFS_FS=y && FSCACHE=y + depends on NFS_FS=m && FSCACHE_OLD || NFS_FS=y && FSCACHE_OLD=y help Say Y here if you want NFS data to be cached locally on disc through the general filesystem cache manager
Temporarily disable the use of fscache by the various Linux network filesystems, apart from afs, so that the fscache core can be rewritten. Signed-off-by: David Howells <dhowells@redhat.com> cc: Dave Wysochanski <dwysocha@redhat.com> cc: Trond Myklebust <trond.myklebust@hammerspace.com> cc: Anna Schumaker <anna.schumaker@netapp.com> cc: linux-nfs@vger.kernel.org cc: Jeff Layton <jlayton@kernel.org> cc: Ilya Dryomov <idryomov@gmail.com> cc: ceph-devel@vger.kernel.org cc: Steve French <sfrench@samba.org> cc: linux-cifs@vger.kernel.org cc: Eric Van Hensbergen <ericvh@gmail.com> cc: Latchesar Ionkov <lucho@ionkov.net> cc: Dominique Martinet <asmadeus@codewreck.org> cc: v9fs-developer@lists.sourceforge.net --- fs/9p/Kconfig | 2 +- fs/ceph/Kconfig | 2 +- fs/cifs/Kconfig | 2 +- fs/fscache/Kconfig | 4 ++++ fs/nfs/Kconfig | 2 +- 5 files changed, 8 insertions(+), 4 deletions(-)