Re: [PATCH v7 07/31] netfs: Make a netfs helper module

From: Dominique Martinet
Date: Thu Apr 29 2021 - 04:44:30 EST


David Howells wrote on Thu, Apr 29, 2021 at 09:41:37AM +0100:
> Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> 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