[Bug 2002429] Re: Patch needs to be removed ("debian/patches/gcc-ibmz-plt-revert.diff: Revert PLT changes from the gcc-11 branch")

[Dimitri John Ledkov]

On Wed, 17 May 2023, 07:45 Chris Halse Rogers, <2002429 at bugs.launchpad.net>
wrote:

> It looks like this is ready to go for Jammy and is already fixed in
> Lunar, but what's the Kinetic status? From what I can tell this applies
> to kinetic but there are no uploads to kinetic-proposed. Are you
> intending to deliberately leave this unfixed in kinetic?
>

`gcc` has always been fixed in kinetic.

The default GCC is what matters here. It is gcc-11 in jammy based off
whatever devel packaging has.

In kinetic this is already fixed in the default GCC, which is gcc-12 as
used to build the kernel.

We do not support building kernels with arbitrary toolchains, thus there is
no support requirement to keep gcc-11 (non default) updated in kinetic,
even as we update gcc-11 (default) in jammy & devel.

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to gcc-11-cross in Ubuntu.
https://bugs.launchpad.net/bugs/2002429

Title:
  Patch needs to be removed ("debian/patches/gcc-ibmz-plt-revert.diff:
  Revert PLT changes from the gcc-11 branch")

Status in Ubuntu on IBM z Systems:
  New
Status in gcc-11 package in Ubuntu:
  Fix Released
Status in gcc-11-cross package in Ubuntu:
  New
Status in gcc-11-cross-ports package in Ubuntu:
  New
Status in gcc-11 source package in Jammy:
  Fix Committed
Status in gcc-11-cross source package in Jammy:
  Fix Committed
Status in gcc-11-cross-ports source package in Jammy:
  Fix Committed
Status in gcc-11 source package in Kinetic:
  New
Status in gcc-11 source package in Lunar:
  Fix Released
Status in gcc-11 package in Debian:
  Fix Released

Bug description:
  [ Impact ]

   * Bug #1954676 (https://bugs.launchpad.net/bugs/1954676) was opened
  because of kernel soft lockup issues with Jammy 5.15. A temporary
  patch to gcc-11 was identified and released under debian/patches/gcc-
  ibmz-plt-revert.diff, which reverted gcc commit 0990d93 ("IBM Z: Use
  @PLT symbols for local functions in 64-bit mode"). An eventual fix to
  the kernel lockup was released under the upstream Linux kernel, commit
  f3b7e73b2c66.

   * To this end, the debian/patches/gcc-ibmz-plt-revert.diff patch
  should be removed from gcc-11 as it presents issues with object file
  symbols. An attempt at this is in my PPA -
  https://launchpad.net/~john-cabaj/+archive/ubuntu/gcc-11). This
  allowed me to get past my issues with the object file symbols.

  [ Test Plan ]

   * use kpatch-build testcases to build and load a livepatch on s390x
  kernel built with gcc-11

   * it should be successful

  [ Where problems could occur ]

   * Whilst the kernel issues was worked-around by reverting compiler
  changes, the actual issue was in the kernel code; which has since been
  fixed. In ubuntu the kernel has been fixed since Ubuntu-5.15.0-20.20

  [ Other Info ]
   
   * A bit sad that this was not detected earlier, and compiler fixed up earlier.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-z-systems/+bug/2002429/+subscriptions