Re: [PATCH v6 02/16] Documentation: filesystems: update NTFS driver documentation

From: Namjae Jeon

Date: Tue Feb 03 2026 - 08:05:09 EST


On Tue, Feb 3, 2026 at 2:44 PM Christoph Hellwig <hch@xxxxxx> wrote:
>
> On Tue, Feb 03, 2026 at 07:01:48AM +0900, Namjae Jeon wrote:
> > Update the NTFS driver documentation (Documentation/filesystems/ntfs.rst)
> > to reflect the current implementation state after switching to iomap and
> > folio instead buffer-head.
> >
> > Changes include:
>
> "Changes include" doesn't really add much value, and feels like AI
> slop.
>
> I'd rewrite the message as:
>
> Update the NTFS driver documentation to reflect the update implementation.
> Remove outdated sections (web site, old features list, known bugs,
> volume/stripe sets with MD/DM driver, limitations of old driver), add a
> concise overview of current driver features and long-term maintenance
> focus, add a utilities support section pointing to ntfsprogs-plus project
> and update mount options list with current supported options.
Okay, I will use this in the next version.
>
> I'd probably also move this last in the series.
Okay.
>
> > +nls=name Deprecated option. Still supported but please use
> > + iocharset=name in the future.
>
> A lot of these mount options sections starts the first line with tab
> indentation and then continue with spaces only. Please stick to one
> of them.
Okay, I will update it in the next version.
>
> Otherwise looks good:
>
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
Thanks for your review!