| Message ID | 20250415-orchester-robben-2be52e119ee4@brauner (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | hfs{plus}: add deprecation warning | expand |
On Tue 15-04-25 09:51:37, Christian Brauner wrote: > Both the hfs and hfsplus filesystem have been orphaned since at least > 2014, i.e., over 10 years. It's time to remove them from the kernel as > they're exhibiting more and more issues and no one is stepping up to > fixing them. > > Signed-off-by: Christian Brauner <brauner@kernel.org> Looks good. And I agree hopefully it sparks interest in the maintainership because this is not completely useless filesystem: Acked-by: Jan Kara <jack@suse.cz> Honza > --- > fs/hfs/super.c | 2 ++ > fs/hfsplus/super.c | 2 ++ > 2 files changed, 4 insertions(+) > > diff --git a/fs/hfs/super.c b/fs/hfs/super.c > index fe09c2093a93..4413cd8feb9e 100644 > --- a/fs/hfs/super.c > +++ b/fs/hfs/super.c > @@ -404,6 +404,8 @@ static int hfs_init_fs_context(struct fs_context *fc) > { > struct hfs_sb_info *hsb; > > + pr_warn("The hfs filesystem is deprecated and scheduled to be removed from the kernel in 2025\n"); > + > hsb = kzalloc(sizeof(struct hfs_sb_info), GFP_KERNEL); > if (!hsb) > return -ENOMEM; > diff --git a/fs/hfsplus/super.c b/fs/hfsplus/super.c > index 948b8aaee33e..58cff4b2a3b4 100644 > --- a/fs/hfsplus/super.c > +++ b/fs/hfsplus/super.c > @@ -656,6 +656,8 @@ static int hfsplus_init_fs_context(struct fs_context *fc) > { > struct hfsplus_sb_info *sbi; > > + pr_warn("The hfsplus filesystem is deprecated and scheduled to be removed from the kernel in 2025\n"); > + > sbi = kzalloc(sizeof(struct hfsplus_sb_info), GFP_KERNEL); > if (!sbi) > return -ENOMEM; > -- > 2.47.2 >
On Tue, Apr 15, 2025 at 09:51:37AM +0200, Christian Brauner wrote: > Both the hfs and hfsplus filesystem have been orphaned since at least > 2014, i.e., over 10 years. It's time to remove them from the kernel as > they're exhibiting more and more issues and no one is stepping up to > fixing them. > > Signed-off-by: Christian Brauner <brauner@kernel.org> > --- > fs/hfs/super.c | 2 ++ > fs/hfsplus/super.c | 2 ++ > 2 files changed, 4 insertions(+) > > diff --git a/fs/hfs/super.c b/fs/hfs/super.c > index fe09c2093a93..4413cd8feb9e 100644 > --- a/fs/hfs/super.c > +++ b/fs/hfs/super.c > @@ -404,6 +404,8 @@ static int hfs_init_fs_context(struct fs_context *fc) > { > struct hfs_sb_info *hsb; > > + pr_warn("The hfs filesystem is deprecated and scheduled to be removed from the kernel in 2025\n"); Does this mean before or after the 2025 LTS kernel is released? I would say that we ought to let this circulate more widely among users, but OTOH I guess no maintainer for a decade is really bad. --D > + > hsb = kzalloc(sizeof(struct hfs_sb_info), GFP_KERNEL); > if (!hsb) > return -ENOMEM; > diff --git a/fs/hfsplus/super.c b/fs/hfsplus/super.c > index 948b8aaee33e..58cff4b2a3b4 100644 > --- a/fs/hfsplus/super.c > +++ b/fs/hfsplus/super.c > @@ -656,6 +656,8 @@ static int hfsplus_init_fs_context(struct fs_context *fc) > { > struct hfsplus_sb_info *sbi; > > + pr_warn("The hfsplus filesystem is deprecated and scheduled to be removed from the kernel in 2025\n"); > + > sbi = kzalloc(sizeof(struct hfsplus_sb_info), GFP_KERNEL); > if (!sbi) > return -ENOMEM; > -- > 2.47.2 > >
On Tue, Apr 15, 2025 at 07:49:07AM -0700, Darrick J. Wong wrote: > On Tue, Apr 15, 2025 at 09:51:37AM +0200, Christian Brauner wrote: > > Both the hfs and hfsplus filesystem have been orphaned since at least > > 2014, i.e., over 10 years. It's time to remove them from the kernel as > > they're exhibiting more and more issues and no one is stepping up to > > fixing them. > > > > Signed-off-by: Christian Brauner <brauner@kernel.org> > > --- > > fs/hfs/super.c | 2 ++ > > fs/hfsplus/super.c | 2 ++ > > 2 files changed, 4 insertions(+) > > > > diff --git a/fs/hfs/super.c b/fs/hfs/super.c > > index fe09c2093a93..4413cd8feb9e 100644 > > --- a/fs/hfs/super.c > > +++ b/fs/hfs/super.c > > @@ -404,6 +404,8 @@ static int hfs_init_fs_context(struct fs_context *fc) > > { > > struct hfs_sb_info *hsb; > > > > + pr_warn("The hfs filesystem is deprecated and scheduled to be removed from the kernel in 2025\n"); > > Does this mean before or after the 2025 LTS kernel is released? I would I would've tried before the LTS release... > say that we ought to let this circulate more widely among users, but which is a valid point. The removal of reiserfs and sysv has been pretty surgically clean. So at least from my POV it should be simple enough to revert the removal. But I'm not dealing with stable kernels so I have no intuition about the pain involved. > OTOH I guess no maintainer for a decade is really bad. > > --D > > > + > > hsb = kzalloc(sizeof(struct hfs_sb_info), GFP_KERNEL); > > if (!hsb) > > return -ENOMEM; > > diff --git a/fs/hfsplus/super.c b/fs/hfsplus/super.c > > index 948b8aaee33e..58cff4b2a3b4 100644 > > --- a/fs/hfsplus/super.c > > +++ b/fs/hfsplus/super.c > > @@ -656,6 +656,8 @@ static int hfsplus_init_fs_context(struct fs_context *fc) > > { > > struct hfsplus_sb_info *sbi; > > > > + pr_warn("The hfsplus filesystem is deprecated and scheduled to be removed from the kernel in 2025\n"); > > + > > sbi = kzalloc(sizeof(struct hfsplus_sb_info), GFP_KERNEL); > > if (!sbi) > > return -ENOMEM; > > -- > > 2.47.2 > > > >
diff --git a/fs/hfs/super.c b/fs/hfs/super.c index fe09c2093a93..4413cd8feb9e 100644 --- a/fs/hfs/super.c +++ b/fs/hfs/super.c @@ -404,6 +404,8 @@ static int hfs_init_fs_context(struct fs_context *fc) { struct hfs_sb_info *hsb; + pr_warn("The hfs filesystem is deprecated and scheduled to be removed from the kernel in 2025\n"); + hsb = kzalloc(sizeof(struct hfs_sb_info), GFP_KERNEL); if (!hsb) return -ENOMEM; diff --git a/fs/hfsplus/super.c b/fs/hfsplus/super.c index 948b8aaee33e..58cff4b2a3b4 100644 --- a/fs/hfsplus/super.c +++ b/fs/hfsplus/super.c @@ -656,6 +656,8 @@ static int hfsplus_init_fs_context(struct fs_context *fc) { struct hfsplus_sb_info *sbi; + pr_warn("The hfsplus filesystem is deprecated and scheduled to be removed from the kernel in 2025\n"); + sbi = kzalloc(sizeof(struct hfsplus_sb_info), GFP_KERNEL); if (!sbi) return -ENOMEM;
Both the hfs and hfsplus filesystem have been orphaned since at least 2014, i.e., over 10 years. It's time to remove them from the kernel as they're exhibiting more and more issues and no one is stepping up to fixing them. Signed-off-by: Christian Brauner <brauner@kernel.org> --- fs/hfs/super.c | 2 ++ fs/hfsplus/super.c | 2 ++ 2 files changed, 4 insertions(+)