Skip to content

Commit

Permalink
changes from review
Browse files Browse the repository at this point in the history
  • Loading branch information
jreidinger committed Dec 9, 2022
1 parent fc66f99 commit 16c49dc
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/lib/kdump/kdump_calibrator.rb
Expand Up @@ -162,8 +162,9 @@ def system
def run_kdumptool
out = Yast::SCR.Execute(Yast::Path.new(".target.bash_output"), kdumptool_cmd)
if out["exit"].zero?
# ensure that fadump is set to unsupported as fadump can be missing in output
# and it means fadump not supported ( as oppose to high mem which is always in output)
# If fadump is missing from output, it means it isn't supported (unlike
# high mem, which is always in output). Thus, let's set fadump related
# values to zero before parsing the kdumptool output
@max_fadump = @min_fadump = @default_fadump = 0
proposal = parse(out["stdout"])
# Populate @min_low, @max_low, @total_memory, etc.
Expand Down

0 comments on commit 16c49dc

Please sign in to comment.