mbox series

[SRU,jammy:linux,0/1] Undefined Behavior Sanitizer (UBSAN) causes failure to match symbols

Message ID 20230518155546.168561-1-john.cabaj@canonical.com
Headers show
Series Undefined Behavior Sanitizer (UBSAN) causes failure to match symbols | expand

Message

John Cabaj May 18, 2023, 3:55 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/2003374

[Impact]
* When UBSAN is enabled in an s390x kernel configuration, kpatch-build can fail to find matching symbols in the vmlinux symbol table. This was discovered in both Jammy 5.15 and Kinetic 5.19 kernels.
* UBSAN was disabled in the Jammy 5.15 s390x kernel configuration to resolve the issue
* Upstream kpatch recently resolved the issue via https://github.com/dynup/kpatch/pull/1332/commits/85344cf524cccc7b8e10bf04ed38a0f586bffd10, so this patch re-enables UBSAN for Jammy 5.15 s390x kernel configuration.

[Fix]
* Simple config change

[Test Case]
* Used kpatch-build v0.9.8 to build and load a fs/proc/meminfo.c Livepatch on s390x kernel successfully

[Where problems could occur]
* Should not introduce any regressions as functionality was enabled since original Jammy 5.15 kernel until v5.15.0-70.77

John Cabaj (1):
  UBUNTU: [Config] s390x: Re-adding UBSAN to configuration

 debian.master/config/annotations | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

Comments

Dimitri John Ledkov May 19, 2023, 12:02 a.m. UTC | #1
On Thu, 18 May 2023 at 16:56, John Cabaj <john.cabaj@canonical.com> wrote:
>
> BugLink: https://bugs.launchpad.net/bugs/2003374
>
> [Impact]
> * When UBSAN is enabled in an s390x kernel configuration, kpatch-build can fail to find matching symbols in the vmlinux symbol table. This was discovered in both Jammy 5.15 and Kinetic 5.19 kernels.
> * UBSAN was disabled in the Jammy 5.15 s390x kernel configuration to resolve the issue
> * Upstream kpatch recently resolved the issue via https://github.com/dynup/kpatch/pull/1332/commits/85344cf524cccc7b8e10bf04ed38a0f586bffd10, so this patch re-enables UBSAN for Jammy 5.15 s390x kernel configuration.

Is that fix already uploaded to all relevant kpatch packages we have
in various places?

>
> [Fix]
> * Simple config change
>
> [Test Case]
> * Used kpatch-build v0.9.8 to build and load a fs/proc/meminfo.c Livepatch on s390x kernel successfully
>
> [Where problems could occur]
> * Should not introduce any regressions as functionality was enabled since original Jammy 5.15 kernel until v5.15.0-70.77
>
> John Cabaj (1):
>   UBUNTU: [Config] s390x: Re-adding UBSAN to configuration
>
>  debian.master/config/annotations | 26 +++++++++++++-------------
>  1 file changed, 13 insertions(+), 13 deletions(-)
>
> --
> 2.34.1
>
>
> --
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
John Cabaj May 19, 2023, 1:36 a.m. UTC | #2
On 5/18/23 7:02 PM, Dimitri John Ledkov wrote:
> On Thu, 18 May 2023 at 16:56, John Cabaj <john.cabaj@canonical.com> wrote:
>>
>> BugLink: https://bugs.launchpad.net/bugs/2003374
>>
>> [Impact]
>> * When UBSAN is enabled in an s390x kernel configuration, kpatch-build can fail to find matching symbols in the vmlinux symbol table. This was discovered in both Jammy 5.15 and Kinetic 5.19 kernels.
>> * UBSAN was disabled in the Jammy 5.15 s390x kernel configuration to resolve the issue
>> * Upstream kpatch recently resolved the issue via https://github.com/dynup/kpatch/pull/1332/commits/85344cf524cccc7b8e10bf04ed38a0f586bffd10, so this patch re-enables UBSAN for Jammy 5.15 s390x kernel configuration.
> 
> Is that fix already uploaded to all relevant kpatch packages we have
> in various places?

I think that the new version is ready to go, though I'm not certain it's uploaded everywhere yet. Adding Ben for comment.


John 

> 
>>
>> [Fix]
>> * Simple config change
>>
>> [Test Case]
>> * Used kpatch-build v0.9.8 to build and load a fs/proc/meminfo.c Livepatch on s390x kernel successfully
>>
>> [Where problems could occur]
>> * Should not introduce any regressions as functionality was enabled since original Jammy 5.15 kernel until v5.15.0-70.77
>>
>> John Cabaj (1):
>>   UBUNTU: [Config] s390x: Re-adding UBSAN to configuration
>>
>>  debian.master/config/annotations | 26 +++++++++++++-------------
>>  1 file changed, 13 insertions(+), 13 deletions(-)
>>
>> --
>> 2.34.1
>>
>>
>> --
>> kernel-team mailing list
>> kernel-team@lists.ubuntu.com
>> https://lists.ubuntu.com/mailman/listinfo/kernel-team
> 
> 
>
Benjamin M Romer May 19, 2023, 3:32 p.m. UTC | #3
On Thu, 2023-05-18 at 20:36 -0500, John Cabaj wrote:
> On 5/18/23 7:02 PM, Dimitri John Ledkov wrote:

> I think that the new version is ready to go, though I'm not certain
> it's uploaded everywhere yet. Adding Ben for comment.

The latest version is in Livepatch Tools Testing (0.9.8+22.04.1-
production2) and has been tested using the current production livepatch
for 5.15.0-71-generic, built on s390, and it works fine. 

I copied the updated kpatch-build to the appropriate builder (11.3.0-
1ubuntu1+22.04.1), repackaged and successfully built the livepatch for
s390, and manually tested it against the 5.15.0-71-generic kernel.

-- Ben
Tim Gardner May 23, 2023, 1:17 p.m. UTC | #4
On 5/18/23 9:55 AM, John Cabaj wrote:
> BugLink: https://bugs.launchpad.net/bugs/2003374
> 
> [Impact]
> * When UBSAN is enabled in an s390x kernel configuration, kpatch-build can fail to find matching symbols in the vmlinux symbol table. This was discovered in both Jammy 5.15 and Kinetic 5.19 kernels.
> * UBSAN was disabled in the Jammy 5.15 s390x kernel configuration to resolve the issue
> * Upstream kpatch recently resolved the issue via https://github.com/dynup/kpatch/pull/1332/commits/85344cf524cccc7b8e10bf04ed38a0f586bffd10, so this patch re-enables UBSAN for Jammy 5.15 s390x kernel configuration.
> 
> [Fix]
> * Simple config change
> 
> [Test Case]
> * Used kpatch-build v0.9.8 to build and load a fs/proc/meminfo.c Livepatch on s390x kernel successfully
> 
> [Where problems could occur]
> * Should not introduce any regressions as functionality was enabled since original Jammy 5.15 kernel until v5.15.0-70.77
> 
> John Cabaj (1):
>    UBUNTU: [Config] s390x: Re-adding UBSAN to configuration
> 
>   debian.master/config/annotations | 26 +++++++++++++-------------
>   1 file changed, 13 insertions(+), 13 deletions(-)
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>
Kamal Mostafa June 16, 2023, 6:04 p.m. UTC | #5
LGTM.

Acked-by: Kamal Mostafa <kamal@canonical.com>

 -Kamal

On Thu, May 18, 2023 at 10:55:45AM -0500, John Cabaj wrote:
> BugLink: https://bugs.launchpad.net/bugs/2003374
> 
> [Impact]
> * When UBSAN is enabled in an s390x kernel configuration, kpatch-build can fail to find matching symbols in the vmlinux symbol table. This was discovered in both Jammy 5.15 and Kinetic 5.19 kernels.
> * UBSAN was disabled in the Jammy 5.15 s390x kernel configuration to resolve the issue
> * Upstream kpatch recently resolved the issue via https://github.com/dynup/kpatch/pull/1332/commits/85344cf524cccc7b8e10bf04ed38a0f586bffd10, so this patch re-enables UBSAN for Jammy 5.15 s390x kernel configuration.
> 
> [Fix]
> * Simple config change
> 
> [Test Case]
> * Used kpatch-build v0.9.8 to build and load a fs/proc/meminfo.c Livepatch on s390x kernel successfully
> 
> [Where problems could occur]
> * Should not introduce any regressions as functionality was enabled since original Jammy 5.15 kernel until v5.15.0-70.77
> 
> John Cabaj (1):
>   UBUNTU: [Config] s390x: Re-adding UBSAN to configuration
> 
>  debian.master/config/annotations | 26 +++++++++++++-------------
>  1 file changed, 13 insertions(+), 13 deletions(-)
> 
> -- 
> 2.34.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Roxana Nicolescu July 7, 2023, 12:29 p.m. UTC | #6
On 18/05/2023 17:55, John Cabaj wrote:
> BugLink: https://bugs.launchpad.net/bugs/2003374
>
> [Impact]
> * When UBSAN is enabled in an s390x kernel configuration, kpatch-build can fail to find matching symbols in the vmlinux symbol table. This was discovered in both Jammy 5.15 and Kinetic 5.19 kernels.
> * UBSAN was disabled in the Jammy 5.15 s390x kernel configuration to resolve the issue
> * Upstream kpatch recently resolved the issue via https://github.com/dynup/kpatch/pull/1332/commits/85344cf524cccc7b8e10bf04ed38a0f586bffd10, so this patch re-enables UBSAN for Jammy 5.15 s390x kernel configuration.
>
> [Fix]
> * Simple config change
>
> [Test Case]
> * Used kpatch-build v0.9.8 to build and load a fs/proc/meminfo.c Livepatch on s390x kernel successfully
>
> [Where problems could occur]
> * Should not introduce any regressions as functionality was enabled since original Jammy 5.15 kernel until v5.15.0-70.77
>
> John Cabaj (1):
>    UBUNTU: [Config] s390x: Re-adding UBSAN to configuration
>
>   debian.master/config/annotations | 26 +++++++++++++-------------
>   1 file changed, 13 insertions(+), 13 deletions(-)
>
>
Applied to jammy:master-next. Thanks!

Roxana