ReiserFS has been considered deprecated for 19 months since commit
eb103a51640e ("reiserfs: Deprecate reiserfs"). However, there are
several architectures that still build it into their defconfig kernels.
As ReiserFS will be removed in 2025, delete all ReiserFS-related options
from defconfig files before the filesystem's removal.
The series is intended to be approved/rejected on an arch-by-arch basis.
No patch is dependant upon another in the series.
See discussion originating in,
Link: https://lore.kernel.org/linux-um/[email protected]/
Peter Lafreniere (7):
arch: um: remove ReiserFS from defconfig
arch: powerpc: remove ReiserFS from defconfig
arch: sh: remove ReiserFS from defconfig
arch: mips: remove ReiserFS from defconfig
arch: m68k: remove ReiserFS from defconfig
arch: arm: remove ReiserFS from defconfig
arch: alpha: remove ReiserFS from defconfig
arch/alpha/configs/defconfig | 1 -
arch/arm/configs/pxa_defconfig | 4 ----
arch/m68k/configs/amiga_defconfig | 1 -
arch/m68k/configs/apollo_defconfig | 1 -
arch/m68k/configs/atari_defconfig | 1 -
arch/m68k/configs/bvme6000_defconfig | 1 -
arch/m68k/configs/hp300_defconfig | 1 -
arch/m68k/configs/mac_defconfig | 1 -
arch/m68k/configs/multi_defconfig | 1 -
arch/m68k/configs/mvme147_defconfig | 1 -
arch/m68k/configs/mvme16x_defconfig | 1 -
arch/m68k/configs/q40_defconfig | 1 -
arch/m68k/configs/sun3_defconfig | 1 -
arch/m68k/configs/sun3x_defconfig | 1 -
arch/mips/configs/fuloong2e_defconfig | 1 -
arch/mips/configs/jazz_defconfig | 4 ----
arch/mips/configs/lemote2f_defconfig | 3 ---
arch/mips/configs/malta_defconfig | 5 -----
arch/mips/configs/malta_kvm_defconfig | 5 -----
arch/mips/configs/maltaup_xpa_defconfig | 5 -----
arch/mips/configs/rm200_defconfig | 4 ----
arch/powerpc/configs/44x/sam440ep_defconfig | 1 -
arch/powerpc/configs/g5_defconfig | 4 ----
arch/powerpc/configs/ppc64e_defconfig | 4 ----
arch/powerpc/configs/ppc6xx_defconfig | 5 -----
arch/sh/configs/landisk_defconfig | 1 -
arch/sh/configs/titan_defconfig | 1 -
arch/um/configs/i386_defconfig | 1 -
arch/um/configs/x86_64_defconfig | 1 -
29 files changed, 62 deletions(-)
--
2.42.0
Hi Peter,
On Tue, Sep 19, 2023 at 5:58 PM Peter Lafreniere <[email protected]> wrote:
> 2) Stops building an obsolete and largely-unused filesystem unnecessarily.
> Some hobbyist targets like m68k and alpha may prefer to keep all filesystems
> available until total removal, but others like arm and UML have no need for
> ReiserFS to be built unless specifically configured.
As UML is used a lot for testing, isn't it actually counter-productive
to remove ReiserFS from the UML defconfig? The less testing it
receives, the higher the chance of introducing regressions.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
Hi Geert,
On Tue, Sep 19, 2023 at 12:02, Geert Uytterhoeven <[email protected]> wrote:
>
> Hi Peter,
>
> On Tue, Sep 19, 2023 at 5:58 PM Peter Lafreniere [email protected] wrote:
>
> > 2) Stops building an obsolete and largely-unused filesystem unnecessarily.
> > Some hobbyist targets like m68k and alpha may prefer to keep all filesystems
> > available until total removal, but others like arm and UML have no need for
> > ReiserFS to be built unless specifically configured.
>
>
> As UML is used a lot for testing, isn't it actually counter-productive
> to remove ReiserFS from the UML defconfig? The less testing it
> receives, the higher the chance of introducing regressions.
UML is used for testing, but in my view that makes the inclusion of
ReiserFS in its defconfig even worse. Users of UML are trying to test a
particular function, and so tend to use ext[2-4], as those are included in
the defconfig and are well tested and stable. So there is no extra testing
being done on ReiserFS due to its inclusion in the defconfig.
Keeping UML's defconfig as slim as possible improves build times, which is
particularly important for kernel testing and development.
>
> Gr{oetje,eeting}s,
>
> Geert
>
Cheers,
Peter Lafreniere
Hi Peter,
On Tue, Sep 19, 2023 at 6:16 PM Peter Lafreniere <[email protected]> wrote:
> On Tue, Sep 19, 2023 at 12:02, Geert Uytterhoeven <[email protected]> wrote:
> > On Tue, Sep 19, 2023 at 5:58 PM Peter Lafreniere [email protected] wrote:
> > > 2) Stops building an obsolete and largely-unused filesystem unnecessarily.
> > > Some hobbyist targets like m68k and alpha may prefer to keep all filesystems
> > > available until total removal, but others like arm and UML have no need for
> > > ReiserFS to be built unless specifically configured.
> >
> >
> > As UML is used a lot for testing, isn't it actually counter-productive
> > to remove ReiserFS from the UML defconfig? The less testing it
> > receives, the higher the chance of introducing regressions.
>
> UML is used for testing, but in my view that makes the inclusion of
> ReiserFS in its defconfig even worse. Users of UML are trying to test a
Why?
Because you want to avoid doing any testing at all on deprecated features?
> particular function, and so tend to use ext[2-4], as those are included in
> the defconfig and are well tested and stable. So there is no extra testing
> being done on ReiserFS due to its inclusion in the defconfig.
I'd expect global file system testers to use something along the line of:
for i in $(grep -v nodev /proc/filesystems ); do
echo --- Testing $i ---
dd if=/dev/zero of=testimage bs=1M count=1 seek=10000
mkfs.$i testimage
mount testimage /mnt -t $i
[run xfstests on testimage]
rm -f testimage
done
> Keeping UML's defconfig as slim as possible improves build times, which is
> particularly important for kernel testing and development.
Good luck testing all functionality using a "slim" kernel ;-)
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
On Tue 19-09-23 18:02:39, Geert Uytterhoeven wrote:
> Hi Peter,
>
> On Tue, Sep 19, 2023 at 5:58 PM Peter Lafreniere <[email protected]> wrote:
> > 2) Stops building an obsolete and largely-unused filesystem unnecessarily.
> > Some hobbyist targets like m68k and alpha may prefer to keep all filesystems
> > available until total removal, but others like arm and UML have no need for
> > ReiserFS to be built unless specifically configured.
>
> As UML is used a lot for testing, isn't it actually counter-productive
> to remove ReiserFS from the UML defconfig? The less testing it
> receives, the higher the chance of introducing regressions.
The only testing I know about for reiserfs (besides build testing) is
syzbot. And regarding the people / bots doing filesystem testing I know
none of them uses UML. Rather it is x86 VMs these days where reiserfs is
disabled in the defconfig for a *long* time (many years). Also when you do
filesystem testing, you usually just test the few filesystems you care
about and for which you have all the tools installed. So frankly I don't
see a good reason to leave reiserfs enabled in defconfigs. But sure if
m68k or other arch wants to keep reiserfs in it's defconfig for some
consistency reasons, I'm fine with it. I just suspect that for most archs
this is just a historical reason.
Honza
--
Jan Kara <[email protected]>
SUSE Labs, CR
On Mon, 18 Sep 2023 17:56:09 +0000, Peter Lafreniere wrote:
> ReiserFS has been considered deprecated for 19 months since commit
> eb103a51640e ("reiserfs: Deprecate reiserfs"). However, there are
> several architectures that still build it into their defconfig kernels.
>
> As ReiserFS will be removed in 2025, delete all ReiserFS-related options
> from defconfig files before the filesystem's removal.
>
> [...]
Applied to powerpc/next.
[2/7] arch: powerpc: remove ReiserFS from defconfig
https://git.kernel.org/powerpc/c/c945e6f453a361b0e9daddd2be9c099d1b80d6f8
cheers