ACK/Cmnt: [SRU] [L/Unstable] [PATCH 0/6] Fix UBSAN in Intel EDAC driver

[Stefan Bader]

On 26.07.23 07:59, Kai-Heng Feng wrote:
> BugLink: https://bugs.launchpad.net/bugs/2028746
> 
> [Impact]
> UBSAN complains shift-out-of-bounds:
> [   13.875276] UBSAN: shift-out-of-bounds in drivers/edac/skx_common.c:369:16
> [   13.875279] shift exponent -66 is negative
> 
> [Fix]
> Skip the absent memory controllers so ~0 won't be used.
> 
> [Test]
> Boot system with patch applied, no more UBSAN error in dmesg.
> 
> [Where problems could occur]
> The fix itself is trivial, however the dependency patches may bring
> behavioural changes as new platform support is also added.
> 
> Qiuxu Zhuo (6):
>    EDAC/skx_common: Enable EDAC support for the "near" memory
>    EDAC/skx_common: Delete duplicated and unreachable code
>    EDAC/i10nm: Add Intel Emerald Rapids server support
>    EDAC/i10nm: Make more configurations CPU model specific
>    EDAC/i10nm: Add Intel Granite Rapids server support
>    EDAC/i10nm: Skip the absent memory controllers
> 
>   drivers/edac/i10nm_base.c | 409 ++++++++++++++++++++++++++++++++------
>   drivers/edac/skx_common.c |  76 +++----
>   drivers/edac/skx_common.h |  61 +++++-
>   3 files changed, 438 insertions(+), 108 deletions(-)
> 

In the hope that this has settled a bit with unstable.

Acked-by: Stefan Bader <stefan.bader at canonical.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0xE8675DEECBEECEA3.asc
Type: application/pgp-keys
Size: 44613 bytes
Desc: OpenPGP public key
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20230803/73964e91/attachment-0001.key>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20230803/73964e91/attachment-0001.sig>