linux-kernel - Re: [PATCH 6.6 000/389] 6.6.76-rc2 review

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <cd10a924-ae65-4b02-aea2-e629947ca7a3@roeck-us.net>
Date: Sun, 9 Feb 2025 07:19:33 -0800
From: Guenter Roeck <linux@...ck-us.net>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>, stable@...r.kernel.org
Cc: patches@...ts.linux.dev, linux-kernel@...r.kernel.org,
 torvalds@...ux-foundation.org, akpm@...ux-foundation.org, shuah@...nel.org,
 patches@...nelci.org, lkft-triage@...ts.linaro.org, pavel@...x.de,
 jonathanh@...dia.com, f.fainelli@...il.com, sudipm.mukherjee@...il.com,
 srw@...dewatkins.net, rwarsow@....de, conor@...nel.org,
 hargar@...rosoft.com, broonie@...nel.org
Subject: Re: [PATCH 6.6 000/389] 6.6.76-rc2 review

On 2/6/25 08:06, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 6.6.76 release.
> There are 389 patches in this series, all will be posted as a response
> to this one.  If anyone has any issues with these being applied, please
> let me know.
> 
> Responses should be made by Sat, 08 Feb 2025 15:51:12 +0000.
> Anything received after that time might be too late.
> 
[ ... ]

> Hongbo Li <lihongbo22@...wei.com>
>      hostfs: fix the host directory parse when mounting.

This patch results in:

Building um:defconfig ... failed
--------------
Error log:
fs/hostfs/hostfs_kern.c:972:9: error: implicit declaration of function 'fsparam_string_empty'; did you mean 'fsparam_string'? [-Werror=implicit-function-declaration]
   972 |         fsparam_string_empty("hostfs",          Opt_hostfs),

because fsparam_string_empty() is not declared globally in v6.6.y.

The patch declaring it is 7b30851a70645 ("fs_parser: move fsparam_string_empty()
helper into header"). Applying that patch on top of 6.6.76 fixes the problem.

The problem only affects "um" builds since hostfs (CONFIG_HOSTFS) is only available
and used there. Oddly enough, the patch breaks the build of this file instead of
fixing the problem it claims to fix, and it looks like no one noticed.
On top of that, "hostfs: convert hostfs to use the new mount API" was obviously
not tested. It looks like a substantial change which would definitely warrant
testing when backported.

That makes me wonder: Should I stop build testing "um" images in older kernels ?

Thanks,
Guenter