Re: [PATCH 5/7] arch: m68k: remove ReiserFS from defconfig

On Mon, Sep 19, 2023 at 03:19, Geert Uytterhoeven <[email protected]> wrote:
> Hi Peter,
> 
> On Mon, Sep 18, 2023 at 7:57 PM Peter Lafreniere [email protected] wrote:
> 
> > ReiserFS has been deprecated for a year and a half, yet is still built
> > as part of a defconfig kernel.
> > 
> > According to commit eb103a51640e ("reiserfs: Deprecate reiserfs"), the
> > filesystem is slated to be removed in 2025. Remove it from the defconfig
> > profiles now, as part of its deprecation process.
> > 
> > Signed-off-by: Peter Lafreniere [email protected]
> 
> 
> Thanks for your patch!
> 
> Given these defconfigs enable ReiserFS as a module (not built-in),
> and that ReiserFS is still available upstream, and enabled by distros
> like Debian, I would like to keep it that way.
> The m68k defconfigs will be updated after the removal of ReiserFS,
> as part of the regular defconfig resync.

I still think that since ReiserFS is being removed soon enough and that there
are a negligible number of users, it should be left out of defconfig builds.
allmodconfig and custom configs (like Debian's) exist for a reason.

But yours is a reasonable approach anyhow. Especially for an arch that is in
the same position as some of the more exotic filesystems that you build as
modules. Thanks for taking a look.

Cheers,

Peter Lafreniere
<[email protected]>