Re: [PATCH v2 01/11] ntfsplus: in-memory, on-disk structures and headers

[Christoph Hellwig <hch@infradead.org>]

On Thu, Nov 27, 2025 at 01:59:34PM +0900, Namjae Jeon wrote:
> This adds in-memory, on-disk structures, headers and documentation.

So a lot of this looks very similar to the old prematurely removed
ntfs driver.  I think reviewing would be a lot simpler if we'd
find some way to bring that back, allowing to focus on the new
code.  I'm not sure how easy that would be as the old version
probably won't build, but a modified revert that doesn't wire it
up to Kconfig would still significantly reduce the diff.  Especially
with the rename back to ntfs as suggested.

I can see that you don't want to do the rest as small incremental
patches, but even a very small number of larger patches ontop of that
base would help a lot.

> +iocharset=name		Deprecated option.  Still supported but please use
> +			nls=name in the future.  See description for nls=name.

Is there much of a point in bringin this back?

> + * ntfs_read_mapping_folio - map a folio into accessible memory, reading it if necessary

The very long comment for something that is just a trivial wrapper
around read_mapping_folio is odd.  Also why does ntrfs need the special
EINTR handling that other file systems don't?

> +/* sizeof()= 40 (0x28) bytes */

You might want to add static_assert() calls instead of the comments
to enforce this.