* Re: [PATCH RESEND] um: Remove CONFIG_REISERFS_FS from defconfig
2023-09-18 13:26 ` Jan Kara
@ 2023-09-18 13:58 ` Richard Weinberger
2023-09-18 14:14 ` Peter Lafreniere
` (2 subsequent siblings)
3 siblings, 0 replies; 11+ messages in thread
From: Richard Weinberger @ 2023-09-18 13:58 UTC (permalink / raw)
To: Jan Kara
Cc: Peter Lafreniere, linux-um, anton ivanov, Johannes Berg,
reiserfs-devel
----- Ursprüngliche Mail -----
> On Mon 18-09-23 12:58:26, Peter Lafreniere wrote:
>> ReiserFS has been deprecated for a year and a half, yet is still built
>> as part of a defconfig UML kernel.
>>
>> According to commit eb103a51640e ("reiserfs: Deprecate reiserfs"), the
>> filesystem is slated to be removed in 2025. Remove it from the defconfig
>> profile now, as part of its deprecation process.
>>
>> Signed-off-by: Peter Lafreniere <peter@n8pjl.ca>
>
> Makes sense but I've noticed there are many more architectures that define
> CONFIG_REISERFS_FS in their defconfig (sh, powerpc, mips, m68k, ia64, arm,
> alpha). So perhaps we should do this for all of them when we are at it?
Yes, a combined patch makes in this case more sense.
>>
>> arch/um/configs/i386_defconfig | 1 -
>> arch/um/configs/x86_64_defconfig | 1 -
>> 2 files changed, 2 deletions(-)
For UML:
Acked-by: Richard Weinberger <richard@nod.at>
Thanks,
//richard
_______________________________________________
linux-um mailing list
linux-um@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-um
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH RESEND] um: Remove CONFIG_REISERFS_FS from defconfig
2023-09-18 13:26 ` Jan Kara
2023-09-18 13:58 ` Richard Weinberger
@ 2023-09-18 14:14 ` Peter Lafreniere
2023-09-18 14:24 ` Jan Kara
2023-09-18 14:58 ` Geert Uytterhoeven
2023-09-18 16:32 ` [PATCH RESEND] " Peter Lafreniere
3 siblings, 1 reply; 11+ messages in thread
From: Peter Lafreniere @ 2023-09-18 14:14 UTC (permalink / raw)
To: jack
Cc: Peter Lafreniere, linux-um, anton.ivanov, johannes,
reiserfs-devel, richard
On Monday, September 18th, 2023 at 09:26, Jan Kara <jack@suse.cz> wrote:
> On Mon 18-09-23 12:58:26, Peter Lafreniere wrote:
>
> > ReiserFS has been deprecated for a year and a half, yet is still built
> > as part of a defconfig UML kernel.
> >
> > According to commit eb103a51640e ("reiserfs: Deprecate reiserfs"), the
> > filesystem is slated to be removed in 2025. Remove it from the defconfig
> > profile now, as part of its deprecation process.
> >
> > Signed-off-by: Peter Lafreniere peter@n8pjl.ca
>
>
> Makes sense but I've noticed there are many more architectures that define
> CONFIG_REISERFS_FS in their defconfig (sh, powerpc, mips, m68k, ia64, arm,
> alpha). So perhaps we should do this for all of them when we are at it?
I agree. I can send a patch series for those arches except for ia64, which has
already been removed from linux-next.
>
> Honza
>
> > ---
> > Fixed invalid DKIM settings causing the previous patch to be rejected.
> >
> > arch/um/configs/i386_defconfig | 1 -
> > arch/um/configs/x86_64_defconfig | 1 -
> > 2 files changed, 2 deletions(-)
> >
> > diff --git a/arch/um/configs/i386_defconfig b/arch/um/configs/i386_defconfig
> > index e543cbac8792..9c9c77f1255a 100644
> > --- a/arch/um/configs/i386_defconfig
> > +++ b/arch/um/configs/i386_defconfig
> > @@ -61,7 +61,6 @@ CONFIG_UML_NET_DAEMON=y
> > CONFIG_UML_NET_MCAST=y
> > CONFIG_UML_NET_SLIRP=y
> > CONFIG_EXT4_FS=y
> > -CONFIG_REISERFS_FS=y
> > CONFIG_QUOTA=y
> > CONFIG_AUTOFS_FS=m
> > CONFIG_ISO9660_FS=m
> > diff --git a/arch/um/configs/x86_64_defconfig b/arch/um/configs/x86_64_defconfig
> > index 939cb12318ca..03b10d3f6816 100644
> > --- a/arch/um/configs/x86_64_defconfig
> > +++ b/arch/um/configs/x86_64_defconfig
> > @@ -59,7 +59,6 @@ CONFIG_UML_NET_DAEMON=y
> > CONFIG_UML_NET_MCAST=y
> > CONFIG_UML_NET_SLIRP=y
> > CONFIG_EXT4_FS=y
> > -CONFIG_REISERFS_FS=y
> > CONFIG_QUOTA=y
> > CONFIG_AUTOFS_FS=m
> > CONFIG_ISO9660_FS=m
> > --
> > 2.42.0
>
> --
> Jan Kara jack@suse.com
>
> SUSE Labs, CRi
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH RESEND] um: Remove CONFIG_REISERFS_FS from defconfig
2023-09-18 14:14 ` Peter Lafreniere
@ 2023-09-18 14:24 ` Jan Kara
0 siblings, 0 replies; 11+ messages in thread
From: Jan Kara @ 2023-09-18 14:24 UTC (permalink / raw)
To: Peter Lafreniere
Cc: jack, linux-um, anton.ivanov, johannes, reiserfs-devel, richard
On Mon 18-09-23 14:14:21, Peter Lafreniere wrote:
> On Monday, September 18th, 2023 at 09:26, Jan Kara <jack@suse.cz> wrote:
> > On Mon 18-09-23 12:58:26, Peter Lafreniere wrote:
> >
> > > ReiserFS has been deprecated for a year and a half, yet is still built
> > > as part of a defconfig UML kernel.
> > >
> > > According to commit eb103a51640e ("reiserfs: Deprecate reiserfs"), the
> > > filesystem is slated to be removed in 2025. Remove it from the defconfig
> > > profile now, as part of its deprecation process.
> > >
> > > Signed-off-by: Peter Lafreniere peter@n8pjl.ca
> >
> >
> > Makes sense but I've noticed there are many more architectures that define
> > CONFIG_REISERFS_FS in their defconfig (sh, powerpc, mips, m68k, ia64, arm,
> > alpha). So perhaps we should do this for all of them when we are at it?
>
> I agree. I can send a patch series for those arches except for ia64, which has
> already been removed from linux-next.
Than would be great. Thanks!
Honza
--
Jan Kara <jack@suse.com>
SUSE Labs, CR
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH RESEND] um: Remove CONFIG_REISERFS_FS from defconfig
2023-09-18 13:26 ` Jan Kara
2023-09-18 13:58 ` Richard Weinberger
2023-09-18 14:14 ` Peter Lafreniere
@ 2023-09-18 14:58 ` Geert Uytterhoeven
2023-09-18 16:32 ` Jan Kara
2023-09-18 16:35 ` Peter Lafreniere
2023-09-18 16:32 ` [PATCH RESEND] " Peter Lafreniere
3 siblings, 2 replies; 11+ messages in thread
From: Geert Uytterhoeven @ 2023-09-18 14:58 UTC (permalink / raw)
To: Jan Kara
Cc: Peter Lafreniere, linux-um, richard, anton.ivanov, johannes,
reiserfs-devel
Hi Jan,
On Mon, Sep 18, 2023 at 3:26 PM Jan Kara <jack@suse.cz> wrote:
> On Mon 18-09-23 12:58:26, Peter Lafreniere wrote:
> > ReiserFS has been deprecated for a year and a half, yet is still built
> > as part of a defconfig UML kernel.
> >
> > According to commit eb103a51640e ("reiserfs: Deprecate reiserfs"), the
> > filesystem is slated to be removed in 2025. Remove it from the defconfig
> > profile now, as part of its deprecation process.
> >
> > Signed-off-by: Peter Lafreniere <peter@n8pjl.ca>
>
> Makes sense but I've noticed there are many more architectures that define
> CONFIG_REISERFS_FS in their defconfig (sh, powerpc, mips, m68k, ia64, arm,
> alpha). So perhaps we should do this for all of them when we are at it?
The policy for the m68k defconfigs is to enable all available exotic
file systems as modules, so support for them is available when needed.
Please let me know why this should be changed, as long as ReiserFS is
still available in the upstream kernel.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
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
_______________________________________________
linux-um mailing list
linux-um@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-um
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH RESEND] um: Remove CONFIG_REISERFS_FS from defconfig
2023-09-18 14:58 ` Geert Uytterhoeven
@ 2023-09-18 16:32 ` Jan Kara
2023-09-18 16:35 ` Peter Lafreniere
1 sibling, 0 replies; 11+ messages in thread
From: Jan Kara @ 2023-09-18 16:32 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Jan Kara, Peter Lafreniere, linux-um, richard, anton.ivanov,
johannes, reiserfs-devel
On Mon 18-09-23 16:58:05, Geert Uytterhoeven wrote:
> Hi Jan,
>
> On Mon, Sep 18, 2023 at 3:26 PM Jan Kara <jack@suse.cz> wrote:
> > On Mon 18-09-23 12:58:26, Peter Lafreniere wrote:
> > > ReiserFS has been deprecated for a year and a half, yet is still built
> > > as part of a defconfig UML kernel.
> > >
> > > According to commit eb103a51640e ("reiserfs: Deprecate reiserfs"), the
> > > filesystem is slated to be removed in 2025. Remove it from the defconfig
> > > profile now, as part of its deprecation process.
> > >
> > > Signed-off-by: Peter Lafreniere <peter@n8pjl.ca>
> >
> > Makes sense but I've noticed there are many more architectures that define
> > CONFIG_REISERFS_FS in their defconfig (sh, powerpc, mips, m68k, ia64, arm,
> > alpha). So perhaps we should do this for all of them when we are at it?
>
> The policy for the m68k defconfigs is to enable all available exotic
> file systems as modules, so support for them is available when needed.
>
> Please let me know why this should be changed, as long as ReiserFS is
> still available in the upstream kernel.
There's no need to change if you don't want to. I can just imagine how for
other archs which seem to enable only "useful" filesystems in defconfig
disabling a deprecated filesystem makes sense.
Honza
--
Jan Kara <jack@suse.com>
SUSE Labs, CR
_______________________________________________
linux-um mailing list
linux-um@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-um
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH RESEND] um: Remove CONFIG_REISERFS_FS from defconfig
2023-09-18 14:58 ` Geert Uytterhoeven
2023-09-18 16:32 ` Jan Kara
@ 2023-09-18 16:35 ` Peter Lafreniere
2023-09-18 16:40 ` Peter Lafreniere
1 sibling, 1 reply; 11+ messages in thread
From: Peter Lafreniere @ 2023-09-18 16:35 UTC (permalink / raw)
To: geert
Cc: anton.ivanov, jack, johannes, linux-um, peter, reiserfs-devel,
richard
Hi Geert,
On Monday, September 18th, 2023 at 10:58, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> Hi Jan,
>
> On Mon, Sep 18, 2023 at 3:26 PM Jan Kara jack@suse.cz wrote:
>
> > On Mon 18-09-23 12:58:26, Peter Lafreniere wrote:
> >
> > > ReiserFS has been deprecated for a year and a half, yet is still built
> > > as part of a defconfig UML kernel.
> > >
> > > According to commit eb103a51640e ("reiserfs: Deprecate reiserfs"), the
> > > filesystem is slated to be removed in 2025. Remove it from the defconfig
> > > profile now, as part of its deprecation process.
> > >
> > > Signed-off-by: Peter Lafreniere peter@n8pjl.ca
> >
> > Makes sense but I've noticed there are many more architectures that define
> > CONFIG_REISERFS_FS in their defconfig (sh, powerpc, mips, m68k, ia64, arm,
> > alpha). So perhaps we should do this for all of them when we are at it?
>
>
> The policy for the m68k defconfigs is to enable all available exotic
> file systems as modules, so support for them is available when needed.
>
> Please let me know why this should be changed, as long as ReiserFS is
> still available in the upstream kernel.
I understand that policy and it makes particular sense for older/hobbyist
hardware like the m68k classic where ReiserFS is included in make defconfig.
However, ReiserFS is deprecated and should be phased out as best as possible
before its removal, and pruning config options for a filesystem that is
considered obsolete and should not be used on the newest kernels is a step
towards that.
ReiserFS is still availiable to be configured by anyone who builds their own
kernel and is still availiable in all(mod|yes)config. As such I can justify
the removal of some old config options that doesn't actually remove support for
the filesystem when it simplifies the deprecation and removal processes for an
ancient, largely unused filesystem.
Please remember that users who build and install their own kernel on hobbyist
hardware should be capable enough to select one Kconfig option if they happen to
need ReiserFS. If not, then removing ReiserFS from the defconfig on a kernel
update is an easier issue to work around than waiting to remove the filesystem
from the tree entirely.
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
>
> 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
Cheers,
Peter Lafreniere
<peter@n8pjl.ca>
And I sent the wrong email the first time. So sorry about that.
_______________________________________________
linux-um mailing list
linux-um@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-um
^ permalink raw reply [flat|nested] 11+ messages in thread* um: Remove CONFIG_REISERFS_FS from defconfig
2023-09-18 16:35 ` Peter Lafreniere
@ 2023-09-18 16:40 ` Peter Lafreniere
0 siblings, 0 replies; 11+ messages in thread
From: Peter Lafreniere @ 2023-09-18 16:40 UTC (permalink / raw)
To: peter
Cc: anton.ivanov, geert, jack, johannes, linux-um, reiserfs-devel,
richard
> > > Makes sense but I've noticed there are many more architectures that define
> > > CONFIG_REISERFS_FS in their defconfig (sh, powerpc, mips, m68k, ia64, arm,
> > > alpha). So perhaps we should do this for all of them when we are at it?
> >
> > The policy for the m68k defconfigs is to enable all available exotic
> > file systems as modules, so support for them is available when needed.
> >
> > Please let me know why this should be changed, as long as ReiserFS is
> > still available in the upstream kernel.
>
>
> I understand that policy and it makes particular sense for older/hobbyist
> hardware like the m68k classic where ReiserFS is included in make defconfig.
>
> However, ReiserFS is deprecated and should be phased out as best as possible
> before its removal, and pruning config options for a filesystem that is
> considered obsolete and should not be used on the newest kernels is a step
> towards that.
>
> ReiserFS is still availiable to be configured by anyone who builds their own
> kernel and is still availiable in all(mod|yes)config. As such I can justify
> the removal of some old config options that doesn't actually remove support for
> the filesystem when it simplifies the deprecation and removal processes for an
> ancient, largely unused filesystem.
>
> Please remember that users who build and install their own kernel on hobbyist
> hardware should be capable enough to select one Kconfig option if they happen to
> need ReiserFS. If not, then removing ReiserFS from the defconfig on a kernel
> update is an easier issue to work around than waiting to remove the filesystem
> from the tree entirely.
Though as Jan said, you're free to not accept a patch removing the config option
until the filesystem is gone for good. I just feel like it's easier to remove
ReiserFS from all defconfigs in one fell swoop.
Cheers,
Peter Lafreniere
<peter@n8pjl.ca>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH RESEND] um: Remove CONFIG_REISERFS_FS from defconfig
2023-09-18 13:26 ` Jan Kara
` (2 preceding siblings ...)
2023-09-18 14:58 ` Geert Uytterhoeven
@ 2023-09-18 16:32 ` Peter Lafreniere
3 siblings, 0 replies; 11+ messages in thread
From: Peter Lafreniere @ 2023-09-18 16:32 UTC (permalink / raw)
To: geert, jack
Cc: anton.ivanov, johannes, linux-um, peter, reiserfs-devel, richard
On Monday, September 18th, 2023 at 09:26, Jan Kara <jack@suse.cz> wrote:
> On Mon 18-09-23 12:58:26, Peter Lafreniere wrote:
>
> > ReiserFS has been deprecated for a year and a half, yet is still built
> > as part of a defconfig UML kernel.
> >
> > According to commit eb103a51640e ("reiserfs: Deprecate reiserfs"), the
> > filesystem is slated to be removed in 2025. Remove it from the defconfig
> > profile now, as part of its deprecation process.
> >
> > Signed-off-by: Peter Lafreniere peter@n8pjl.ca
>
>
> Makes sense but I've noticed there are many more architectures that define
> CONFIG_REISERFS_FS in their defconfig (sh, powerpc, mips, m68k, ia64, arm,
> alpha). So perhaps we should do this for all of them when we are at it?
I agree. I can send a patch series for those arches except for ia64, which has
already been removed from linux-next.
>
> Honza
>
> > ---
> > Fixed invalid DKIM settings causing the previous patch to be rejected.
> >
> > arch/um/configs/i386_defconfig | 1 -
> > arch/um/configs/x86_64_defconfig | 1 -
> > 2 files changed, 2 deletions(-)
> >
> > diff --git a/arch/um/configs/i386_defconfig b/arch/um/configs/i386_defconfig
> > index e543cbac8792..9c9c77f1255a 100644
> > --- a/arch/um/configs/i386_defconfig
> > +++ b/arch/um/configs/i386_defconfig
> > @@ -61,7 +61,6 @@ CONFIG_UML_NET_DAEMON=y
> > CONFIG_UML_NET_MCAST=y
> > CONFIG_UML_NET_SLIRP=y
> > CONFIG_EXT4_FS=y
> > -CONFIG_REISERFS_FS=y
> > CONFIG_QUOTA=y
> > CONFIG_AUTOFS_FS=m
> > CONFIG_ISO9660_FS=m
> > diff --git a/arch/um/configs/x86_64_defconfig b/arch/um/configs/x86_64_defconfig
> > index 939cb12318ca..03b10d3f6816 100644
> > --- a/arch/um/configs/x86_64_defconfig
> > +++ b/arch/um/configs/x86_64_defconfig
> > @@ -59,7 +59,6 @@ CONFIG_UML_NET_DAEMON=y
> > CONFIG_UML_NET_MCAST=y
> > CONFIG_UML_NET_SLIRP=y
> > CONFIG_EXT4_FS=y
> > -CONFIG_REISERFS_FS=y
> > CONFIG_QUOTA=y
> > CONFIG_AUTOFS_FS=m
> > CONFIG_ISO9660_FS=m
> > --
> > 2.42.0
>
> --
> Jan Kara jack@suse.com
>
> SUSE Labs, CRi
^ permalink raw reply [flat|nested] 11+ messages in thread