Messages in this thread | | | Date | Thu, 29 Apr 2021 17:43:54 +0900 | From | Dominique Martinet <> | Subject | Re: [PATCH v7 07/31] netfs: Make a netfs helper module |
| |
David Howells wrote on Thu, Apr 29, 2021 at 09:41:37AM +0100: > Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > I see later patches make AFS and FSCACHE select NETFS_SUPPORT. If this > > is just a library of functions, to be selected by its users, then please > > make the symbol invisible. > > Ideally, yes, it would be an invisible symbol enabled by select from the > network filesystems that use it - but doing that means that you can't choose > whether to build it in or build it as a module.
Afaik such dependencies are then built as a module if everything it depends on are modules, and built-in if any of these are built-in.
I think most users would be fine with that -- there's little reason to have netfs built-in if AFS ceph etc all are modules?
-- Dominique
| |