Re: [PATCH] hostfs: Add const qualifier to host_root in hostfs_fill_super()

From: Richard Weinberger
Date: Tue Jun 11 2024 - 16:19:34 EST


----- Ursprüngliche Mail -----
> Von: "Nathan Chancellor" <nathan@xxxxxxxxxx>
> An: "Christian Brauner" <brauner@xxxxxxxxxx>
> CC: "Hongbo Li" <lihongbo22@xxxxxxxxxx>, "richard" <richard@xxxxxx>, "anton ivanov" <anton.ivanov@xxxxxxxxxxxxxxxxxx>,
> "Johannes Berg" <johannes@xxxxxxxxxxxxxxxx>, "linux-um" <linux-um@xxxxxxxxxxxxxxxxxxx>, "linux-kernel"
> <linux-kernel@xxxxxxxxxxxxxxx>, "Nathan Chancellor" <nathan@xxxxxxxxxx>
> Gesendet: Dienstag, 11. Juni 2024 21:58:41
> Betreff: [PATCH] hostfs: Add const qualifier to host_root in hostfs_fill_super()

> After the recent conversion to the new mount API, there is a warning
> when building hostfs (which may be upgraded to an error via
> CONFIG_WERROR=y):
>
> fs/hostfs/hostfs_kern.c: In function 'hostfs_fill_super':
> fs/hostfs/hostfs_kern.c:942:27: warning: initialization discards 'const'
> qualifier from pointer target type [-Wdiscarded-qualifiers]
> 942 | char *host_root = fc->source;
> | ^~
>
> Add the 'const' qualifier, as host_root will not be modified after its
> assignment. Move the assignment to keep the existing reverse Christmas
> tree order intact.
>
> Fixes: cd140ce9f611 ("hostfs: convert hostfs to use the new mount API")
> Signed-off-by: Nathan Chancellor <nathan@xxxxxxxxxx>

Acked-by: Richard Weinberger <richard@xxxxxx>

Thanks,
//richard