Re: [PATCH 02/11] ntfsplus: add super block operations

[Mobin Aydinfar]

Hi Namjae, I built your new driver (as DKMS) and I'm using it and it 
went smooth so far. Thanks for this good driver (and also really 
practical userspace tools) but something in dmesg caught my eye:

On 10/20/25 05:37, Namjae Jeon wrote:
> This adds the implementation of superblock operations for ntfsplus.
>
> Signed-off-by: Namjae Jeon <linkinjeon@xxxxxxxxxx>
> ---
>   fs/ntfsplus/super.c | 2716 +++++++++++++++++++++++++++++++++++++++++++
>   1 file changed, 2716 insertions(+)
>   create mode 100644 fs/ntfsplus/super.c
>
> diff --git a/fs/ntfsplus/super.c b/fs/ntfsplus/super.c
> new file mode 100644
> index 000000000000..1803eeec5618
> --- /dev/null
> +++ b/fs/ntfsplus/super.c
> @@ -0,0 +1,2716 @@
> ...
> +	pr_info("volume version %i.%i, dev %s, cluster size %d\n",
> +		vol->major_ver, vol->minor_ver, sb->s_id, vol->cluster_size);
> +
> ...

Shouldn't pr_info() messages have "ntfsplus: " prefix? I mean most 
drivers do so and it is weird to me to have something like this:

[    5.431662] volume version 3.1, dev sda3, cluster size 4096
[    5.444801] volume version 3.1, dev sdb1, cluster size 4096

instead of this:

[    5.431662] ntfsplus: volume version 3.1, dev sda3, cluster size 4096
[    5.444801] ntfsplus: volume version 3.1, dev sdb1, cluster size 4096

in my dmesg. What do you think? It wouldn't be better to include 
"ntfsplus: " prefix for pr_info messages?

Best Regards