Re: [PATCH v6 02/16] Documentation: filesystems: update NTFS driver documentation
From: Christoph Hellwig
Date: Tue Feb 03 2026 - 00:45:11 EST
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.
I'd probably also move this last in the series.
> +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.
Otherwise looks good:
Reviewed-by: Christoph Hellwig <hch@xxxxxx>