Re: [PATCH v2 00/53] fscache: Rewrite index API and management system

From: Linus Torvalds
Date: Fri Oct 22 2021 - 16:25:17 EST


On Fri, Oct 22, 2021 at 9:58 AM Linus Torvalds
<torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:
>
> and if (c) is the thing that all the network filesystem people want,
> then what the heck is the point in keeping dead code around? At that
> point, all the rename crap is just extra work, extra noise, and only a
> distraction. There's no upside that I can see.

Again, I'm not a fan of (c) as an option, but if done, then the
simplest model would appear to be:

- remove the old fscache code, obviously disabling the Kconfig for it
for each filesystem, all in one fell swoop.

- add the new fscache code (possibly preferably in sane chunks that
explains the parts).

- then do a "convert to new world order and enable" commit
individually for each filesystem

but as mentioned, there's no sane way to bisect things, or have a sane
development history in this kind of situation.

Linus