Project

General

Profile

Actions

action #87788

open

[qe-core][s390x] Additional boot parameters based on workaround or not

Added by skriesch about 3 years ago. Updated 9 days ago.

Status:
In Progress
Priority:
Normal
Assignee:
-
Category:
Bugs in existing tests
Target version:
-
Start date:
2021-01-15
Due date:
% Done:

80%

Estimated time:
Difficulty:

Description

We had a discussion in https://bugzilla.suse.com/show_bug.cgi?id=1156053#c59, whether we would require both boot parameters "hardened_usercopy=off" and "hvc_iucv=8" or not.
Developers are saying, that the bug would be fixed. Needles are looking ok.
The workaround of the following if statement is used by openQA, so that both kernel parameters are used and anything is going wrong, so that we have got a SoftFailed as a result.

The used if statement:
if ((is_sle('>=15-SP2') || is_tumbleweed()) && get_var('WORKAROUND_BUGS') =~ 'bsc1156047') {
$params .= ' hardened_usercopy=off hvc_iucv=8';
record_soft_failure('bsc#1156053 - hardened_usercopy=off to avoid "/dev/hvc0: cannot get controlling tty: Operation not permitted" (Kernel memory overwrite attempt detected to SLUB object - illegal operation)');
}

Test with the if statement (based on my PR):
https://openqa.opensuse.org/tests/1589414#

Test without the if statement:
https://openqa.opensuse.org/tests/1589601#

We would have green tests without the workaround. We need a solution, if we would keep the if statement.

Actions #2

Updated by skriesch about 3 years ago

Interesting! The test with additional kernel parameters has got a timeout failure (referenced in https://progress.opensuse.org/issues/81682) as a result: https://openqa.opensuse.org/tests/1589414#step/system_prepare/5

It seems that anything is going wrong or makes the system unstable with these kernel parameters. Or the timeout is temporary. That has happened without hardened_usercopy=off, too

Actions #3

Updated by skriesch about 3 years ago

@Sergio Which test version is working correctly in your point of view (with if statement or without the if statement)?

Actions #4

Updated by SLindoMansilla about 3 years ago

skriesch wrote:

@Sergio Which test version is working correctly in your point of view (with if statement or without the if statement)?

Since it works without the if, the if should be removed :)

Actions #5

Updated by SLindoMansilla about 3 years ago

I have also removed the setting WORKAROUND_BUGS=bsc1156047 from https://openqa.opensuse.org/admin/machines (s390x-zVM-vswitch-l2)

Actions #6

Updated by skriesch about 3 years ago

I wanted to know your opinion because it could be that the if statement has got a really important test feature. :)

Actions #7

Updated by SLindoMansilla about 3 years ago

skriesch wrote:

I wanted to know your opinion because it could be that the if statement has got a really important test feature. :)

No, it was only for the workaround.

But, the setting EXTRABOOTPARAMS: hvc_iucv=8 is still present on https://openqa.opensuse.org/admin/job_templates/34
We have to still investigate why was/is it needed.

Actions #8

Updated by skriesch about 3 years ago

Should I keep this ticket open because of this topic after the merge of my PR then?

Actions #9

Updated by SLindoMansilla about 3 years ago

skriesch wrote:

Should I keep this ticket open because of this topic after the merge of my PR then?

Yes, the subject of this ticket also affects that setting, so it would be a good idea to handle it on this ticket too.

Actions #10

Updated by skriesch about 3 years ago

Actions #11

Updated by AdaLovelace about 3 years ago

  • Category set to Bugs in existing tests
  • Status changed from New to In Progress
  • % Done changed from 0 to 80
Actions #12

Updated by okurz about 2 years ago

  • Subject changed from [s390x] Additional boot parameters based on workaround or not to [qe-core][s390x] Additional boot parameters based on workaround or not

I suggest for QE-Core to crosscheck

Actions #13

Updated by slo-gin about 1 year ago

This ticket was set to Normal priority but was not updated within the SLO period. Please consider picking up this ticket or just set the ticket to the next lower priority.

Actions #14

Updated by slo-gin 9 days ago

This ticket was set to Normal priority but was not updated within the SLO period. Please consider picking up this ticket or just set the ticket to the next lower priority.

Actions

Also available in: Atom PDF