Test requirements (this is required and block this SRU, in my opinion) - Even though the patches are targeted only against Sapphire Rapids, it is important that the Test Plan covers other CPU models too.
This is to ensure not only the fix for Sapphire Rapids is correct, but also that there are no regressions in other CPU models. Also, these changes require very domain specific knowledge to review, which is not usually available for all domains within the SRU team, so the more testing that can prove the SRU is correct and regression safe, the better. (Also note that libopenblas0 is depended on by other packages; `reverse-depends -r jammy libopenblas0`). - Can the Test Plan section please explicitly cover other CPU models too? - Can the Test Plan section please explicitly mention a *correctness* test? This may already be the case with 'dbgemm test program', but is unclear. - This should help validate the '* 2' multiplier removal in patch 1, mentioned to be related to gcc-11 (default in Jammy), but the upstream PR [1] decription says gcc 11.3 in Jammy (currently 11.4) is not affected. [1] https://github.com/OpenMathLib/OpenBLAS/pull/4048 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/2040027 Title: [SPR][ICX] OpenBLAS: Issues while testing sbgemm for Sapphire Rapids (SPR) To manage notifications about this bug go to: https://bugs.launchpad.net/intel/+bug/2040027/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
