linux-kernel - Re: [PATCH 6.1 000/440] 6.1.103-rc1 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] [thread-next>] [day] [month] [year] [list]

Message-ID: <2024073137-scouting-wooing-ec33@gregkh>
Date: Wed, 31 Jul 2024 08:07:15 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: Frank Scheiner <frank.scheiner@....de>
Cc: akpm@...ux-foundation.org, allen.lkml@...il.com, broonie@...nel.org,
	conor@...nel.org, f.fainelli@...il.com, jonathanh@...dia.com,
	linux-kernel@...r.kernel.org, linux@...ck-us.net,
	lkft-triage@...ts.linaro.org, patches@...nelci.org,
	patches@...ts.linux.dev, pavel@...x.de, rwarsow@....de,
	shuah@...nel.org, srw@...dewatkins.net, stable@...r.kernel.org,
	sudipm.mukherjee@...il.com, torvalds@...ux-foundation.org
Subject: Re: [PATCH 6.1 000/440] 6.1.103-rc1 review

On Wed, Jul 31, 2024 at 01:24:25AM +0200, Frank Scheiner wrote:
> Dear Greg,
> 
> 6259151c04d4e0085e00d2dcb471ebdd1778e72e from mainline is missing in
> 6.1.103-rc1 and completes 39823b47bbd40502632ffba90ebb34fff7c8b5e8.
> 
> Please note that the second hunk from 6259151 needs to be modified to
> cleanly apply to 6.1.103-rc1. Example on [1].
> 
> [1]:
> https://github.com/linux-ia64/linux-stable-rc/blob/__mirror/patches/linux-6.1.y/6259151c04d4e0085e00d2dcb471ebdd1778e72e.patch
> 
> ****
> 
> ## Long version ##
> 
> This patch series breaks operation of the hp-sim kernel in ski. I think
> it happens when trying to access the ext4 root FS in the simulation, see
> for example [2] for more details.
> 
> [2]: https://github.com/linux-ia64/linux-stable-rc/issues/3
> 
> From the call trace:
> ```
> [...]
> [<a0000001000263f0>] die+0x1b0/0x3e0
> [<a00000010004bd40>] ia64_do_page_fault+0x680/0x9c0
> [<a00000010000c4e0>] ia64_leave_kernel+0x0/0x280
> [<a00000010063f6a0>] dd_limit_depth+0x80/0x140
> [...]
> ```
> 
> ...I tracked it down to [3] and following the linked discussion ([4]),
> this is actually patch 2 of 2. And 1 of 2 ([5]) is in upstream, but not
> in 6.1.103-rc1.
> 
> [3]:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=39823b47bbd40502632ffba90ebb34fff7c8b5e8
> 
> [4]: https://lore.kernel.org/all/20240509170149.7639-3-bvanassche@acm.org/
> 
> [5]:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=6259151c04d4e0085e00d2dcb471ebdd1778e72e
> 
> Applying that patch ([5]) (plus adapting hunk #2 of it) on top of
> 6.1.103-rc1 fixes it for me ([6]).
> 
> [6]:
> https://github.com/linux-ia64/linux-stable-rc/actions/runs/10170700172#summary-28130632329

Thanks for this, I'll just drop the original offending commit here as
that seems like the simplest way.  If these really need to be in the
6.1.y tree, I'll be glad to take a backported series that people have
tested to verify it all works properly.

greg k-h