ACK: [PATCH 0/1 v2][j/l linux] Don't WARN_ON_ONCE() for a broken discovery table

Andrei Gherzan andrei.gherzan at canonical.com
Wed Jan 10 20:02:02 UTC 2024


On 24/01/08 09:58AM, Tim Gardner wrote:
> BugLink: https://bugs.launchpad.net/bugs/2048404
> 
> v2 - Don't include extraneous patch.
> 
> SRU Justification
> 
> [Impact]
> 
>   perf/x86/uncore: Don't WARN_ON_ONCE() for a broken discovery table
> 
>     The kernel warning message is triggered, when SPR MCC is used.
> 
>     [ 17.945331] ------------[ cut here ]------------
>     [ 17.946305] WARNING: CPU: 65 PID: 1 at
>     arch/x86/events/intel/uncore_discovery.c:184
>     intel_uncore_has_discovery_tables+0x4c0/0x65c
>     [ 17.946305] Modules linked in:
>     [ 17.946305] CPU: 65 PID: 1 Comm: swapper/0 Not tainted
>     5.4.17-2136.313.1-X10-2c+ #4
> 
>     It's caused by the broken discovery table of UPI.
> 
>     The discovery tables are from hardware. Except for dropping the broken
>     information, there is nothing Linux can do. Using WARN_ON_ONCE() is
>     overkilled.
> 
>     Use the pr_info() to replace WARN_ON_ONCE(), and specify what uncore unit
>     is dropped and the reason.
> 
> [Fix]
> 
> Commit 5d515ee40cb57ea5331998f27df7946a69f14dc3 ('perf/x86/uncore: Don't WARN_ON_ONCE() for a broken discovery table')
> 
> [Test]
> 
> AWS tested
> 
> [Regression potential]
> 
> Low regression potential. Only the code that prints the warning has been changed.
> 
> [Other Info]
> 
> SF: #00347830

Acked-by: Andrei Gherzan <andrei.gherzan at canonical.com>

-- 
Andrei Gherzan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20240110/0317b79d/attachment.sig>


More information about the kernel-team mailing list