linux-kernel - Re: [PATCH v2 11/11] ntfsplus: add Kconfig and Makefile

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAKYAXd8K76CeQNtR-QOMSJ_JjuoiibuQkd4NhkPPM_CQNdNajw@mail.gmail.com>
Date: Fri, 28 Nov 2025 12:02:25 +0900
From: Namjae Jeon <linkinjeon@...nel.org>
To: Amir Goldstein <amir73il@...il.com>
Cc: viro@...iv.linux.org.uk, brauner@...nel.org, hch@...radead.org, hch@....de, 
	tytso@....edu, willy@...radead.org, jack@...e.cz, djwong@...nel.org, 
	josef@...icpanda.com, sandeen@...deen.net, rgoldwyn@...e.com, 
	xiang@...nel.org, dsterba@...e.com, pali@...nel.org, ebiggers@...nel.org, 
	neil@...wn.name, linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org, 
	iamjoonsoo.kim@....com, cheol.lee@....com, jay.sim@....com, gunho.lee@....com
Subject: Re: [PATCH v2 11/11] ntfsplus: add Kconfig and Makefile

On Thu, Nov 27, 2025 at 10:12 PM Amir Goldstein <amir73il@...il.com> wrote:
>
> On Thu, Nov 27, 2025 at 1:40 PM Namjae Jeon <linkinjeon@...nel.org> wrote:
> >
> > On Thu, Nov 27, 2025 at 8:22 PM Amir Goldstein <amir73il@...il.com> wrote:
> > >
> > > On Thu, Nov 27, 2025 at 6:01 AM Namjae Jeon <linkinjeon@...nel.org> wrote:
> > > >
> > > > This adds the Kconfig and Makefile for ntfsplus.
> > > >
> > > > Signed-off-by: Namjae Jeon <linkinjeon@...nel.org>
> > > > ---
> > > >  fs/Kconfig           |  1 +
> > > >  fs/Makefile          |  1 +
> > > >  fs/ntfsplus/Kconfig  | 45 ++++++++++++++++++++++++++++++++++++++++++++
> > > >  fs/ntfsplus/Makefile | 18 ++++++++++++++++++
> > > >  4 files changed, 65 insertions(+)
> > > >  create mode 100644 fs/ntfsplus/Kconfig
> > > >  create mode 100644 fs/ntfsplus/Makefile
> > > >
> > > > diff --git a/fs/Kconfig b/fs/Kconfig
> > > > index 0bfdaecaa877..70d596b99c8b 100644
> > > > --- a/fs/Kconfig
> > > > +++ b/fs/Kconfig
> > > > @@ -153,6 +153,7 @@ menu "DOS/FAT/EXFAT/NT Filesystems"
> > > >  source "fs/fat/Kconfig"
> > > >  source "fs/exfat/Kconfig"
> > > >  source "fs/ntfs3/Kconfig"
> > > > +source "fs/ntfsplus/Kconfig"
> > > >
> > > >  endmenu
> > > >  endif # BLOCK
> > > > diff --git a/fs/Makefile b/fs/Makefile
> > > > index e3523ab2e587..2e2473451508 100644
> > > > --- a/fs/Makefile
> > > > +++ b/fs/Makefile
> > > > @@ -91,6 +91,7 @@ obj-y                         += unicode/
> > > >  obj-$(CONFIG_SMBFS)            += smb/
> > > >  obj-$(CONFIG_HPFS_FS)          += hpfs/
> > > >  obj-$(CONFIG_NTFS3_FS)         += ntfs3/
> > > > +obj-$(CONFIG_NTFSPLUS_FS)      += ntfsplus/
> > >
> > > I suggested in another reply to keep the original ntfs name
> > >
> > > More important is to keep your driver linked before the unmaintained
> > > ntfs3, so that it hopefully gets picked up before ntfs3 for auto mount type
> > > if both drivers are built-in.
> > Okay, I will check it:)
> > >
> > > I am not sure if keeping the order here would guarantee the link/registration
> > > order. If not, it may make sense to mutually exclude them as built-in drivers.
> > Okay, I am leaning towards the latter.
>
> Well it's not this OR that.
> please add you driver as the original was before ntfs3
>
> obj-$(CONFIG_NTFS_FS)      += ntfs/
> obj-$(CONFIG_NTFS3_FS)         += ntfs3/
Okay.
>
> > If you have no objection, I will add the patch to mutually exclude the two ntfs implementation.
>
> You should definitely allow them both if at least one is built as a module
> I think it would be valuable for testing.
>
> Just that
> CONFIG_NTFS_FS=y
> CONFIG_NTFS3_FS=y
>
> I don't see the usefulness in allowing that.
> (other people may disagree)
>
> I think that the way to implement it is using an auxiliary choice config var
> in fs/Kconfig (i.e. CONFIG_DEFAULT_NTFS) and select/depends statements
> to only allow the default ntfs driver to be configured as 'y',
> but couldn't find a good example to point you at.
Okay. Could you please check whether the attached patch matches what
you described ?

Thanks!
>
> Thanks,
> Amir.

View attachment "0001-ntfs-restrict-built-in-NTFS-seclection-to-one-driver.patch" of type "text/x-patch" (1928 bytes)