Received-SPF: pass (google.com: domain of linux-kernel+bounces-94095-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Precedence: bulk
MIME-Version: 1.0
References: <20240305074649.580820283@linuxfoundation.org> <20240305-arson-panhandle-afa453ccb0aa@wendy>
 <1a9a5456-ea3f-8a30-d8db-f49269966e71@w6rz.net> <fc131a72-6421-ad75-44bd-5ab7761d1cc4@w6rz.net>
 <013ebb78-46df-0db7-d95e-51229273bd2c@w6rz.net> <2024030521-uncover-pushchair-e68c@gregkh>
In-Reply-To: <2024030521-uncover-pushchair-e68c@gregkh>
From: Luna Jernberg <droidbittin@gmail.com>
Date: Wed, 6 Mar 2024 15:55:06 +0100
Message-ID: <CADo9pHhF-yaw_EgenkUt5Z6t9_TW9XroWeoeXVRquLQJfuO+FA@mail.gmail.com>
Subject: Re: [PATCH 6.7 000/163] 6.7.9-rc2 review
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Ron Economos <re@w6rz.net>, Conor Dooley <conor.dooley@microchip.com>, stable@vger.kernel.org, 
	patches@lists.linux.dev, linux-kernel@vger.kernel.org, 
	torvalds@linux-foundation.org, akpm@linux-foundation.org, linux@roeck-us.net, 
	shuah@kernel.org, patches@kernelci.org, lkft-triage@lists.linaro.org, 
	pavel@denx.de, jonathanh@nvidia.com, f.fainelli@gmail.com, 
	sudipm.mukherjee@gmail.com, srw@sladewatkins.net, rwarsow@gmx.de, 
	conor@kernel.org, allen.lkml@gmail.com
Content-Type: text/plain; charset="UTF-8"

Tested-by: Luna Jernberg <droidbittin@gmail.com>

Den tis 5 mars 2024 kl 12:38 skrev Greg Kroah-Hartman
<gregkh@linuxfoundation.org>:
>
> On Tue, Mar 05, 2024 at 01:52:59AM -0800, Ron Economos wrote:
> > On 3/5/24 1:48 AM, Ron Economos wrote:
> > > On 3/5/24 1:27 AM, Ron Economos wrote:
> > > > On 3/5/24 12:31 AM, Conor Dooley wrote:
> > > > > On Tue, Mar 05, 2024 at 07:58:57AM +0000, Greg Kroah-Hartman wrote:
> > > > > > This is the start of the stable review cycle for the 6.7.9 release.
> > > > > > There are 163 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 Thu, 07 Mar 2024 07:46:26 +0000.
> > > > > > Anything received after that time might be too late.
> > > > > > Samuel Holland <samuel.holland@sifive.com>
> > > > > >      riscv: Save/restore envcfg CSR during CPU suspend
> > > > > >
> > > > > > Samuel Holland <samuel.holland@sifive.com>
> > > > > >      riscv: Add a custom ISA extension for the [ms]envcfg CSR
> > > > > I left a comment in response to the off-list email about this patch,
> > > > > I don't think it's gonna work as the number this custom extension has
> > > > > been given exceeds the max in 6.7/
> > > > >
> > > > > Cheers,
> > > > > Conor.
> > > > >
> > > > > > Samuel Holland <samuel.holland@sifive.com>
> > > > > >      riscv: Fix enabling cbo.zero when running in M-mode
> > > >
> > > > Yeah, it doesn't work. Here's the new error:
> > > >
> > > > arch/riscv/kernel/cpufeature.c:180:9: error: implicit declaration of
> > > > function '__RISCV_ISA_EXT_SUPERSET'; did you mean
> > > > 'RISCV_ISA_EXT_SVPBMT'? [-Werror=implicit-function-declaration]
> > > >   180 |         __RISCV_ISA_EXT_SUPERSET(zicbom,
> > > > RISCV_ISA_EXT_ZICBOM, riscv_xlinuxenvcfg_exts),
> > > >       |         ^~~~~~~~~~~~~~~~~~~~~~~~
> > > >       |         RISCV_ISA_EXT_SVPBMT
> > > > arch/riscv/kernel/cpufeature.c:180:34: error: 'zicbom' undeclared
> > > > here (not in a function)
> > > >   180 |         __RISCV_ISA_EXT_SUPERSET(zicbom,
> > > > RISCV_ISA_EXT_ZICBOM, riscv_xlinuxenvcfg_exts),
> > > >       |                                  ^~~~~~
> > > > arch/riscv/kernel/cpufeature.c:181:34: error: 'zicboz' undeclared
> > > > here (not in a function)
> > > >   181 |         __RISCV_ISA_EXT_SUPERSET(zicboz,
> > > > RISCV_ISA_EXT_ZICBOZ, riscv_xlinuxenvcfg_exts),
> > > >       |                                  ^~~~~~
> > > > cc1: some warnings being treated as errors
> > > > make[4]: *** [scripts/Makefile.build:243:
> > > > arch/riscv/kernel/cpufeature.o] Error 1
> > > > make[3]: *** [scripts/Makefile.build:480: arch/riscv/kernel] Error 2
> > > > make[2]: *** [scripts/Makefile.build:480: arch/riscv] Error 2
> > > >
> > > >
> > > This depends on a much earlier patch, "riscv: add ISA extension parsing
> > > for vector crypto" (upstream commit
> > > aec3353963b8de889c3f1ab7cc8ba11e99626606).
> > >
> > > I think the best solution will be to revert all three patches.
> > >
> > > riscv: Save/restore envcfg CSR during CPU suspend
> > >
> > > riscv: Add a custom ISA extension for the [ms]envcfg CSR
> > >
> > > riscv: Fix enabling cbo.zero when running in M-mode
> > >
> > >
> > >
> > Sorry, "riscv: Fix enabling cbo.zero when running in M-mode" is okay. Just
> > the first two.
>
> Ok, will go drop both of them now and will push out a -rc3.
>
> thanks,
>
> greg k-h
>