Received-SPF: pass (google.com: domain of linux-kernel+bounces-92031-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Subject: Re: [PATCH 6.7 000/163] 6.7.9-rc2 review
From: Ron Economos <re@w6rz.net>
To: Conor Dooley <conor.dooley@microchip.com>,
 Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: 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
References: <20240305074649.580820283@linuxfoundation.org>
 <20240305-arson-panhandle-afa453ccb0aa@wendy>
 <1a9a5456-ea3f-8a30-d8db-f49269966e71@w6rz.net>
 <fc131a72-6421-ad75-44bd-5ab7761d1cc4@w6rz.net>
Message-ID: <013ebb78-46df-0db7-d95e-51229273bd2c@w6rz.net>
Date: Tue, 5 Mar 2024 01:52:59 -0800
User-Agent: Mozilla/5.0 (X11; Linux armv7l; rv:78.0) Gecko/20100101
 Thunderbird/78.14.0
Precedence: bulk
MIME-Version: 1.0
In-Reply-To: <fc131a72-6421-ad75-44bd-5ab7761d1cc4@w6rz.net>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-US

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.