kexec January 2021

kexec@lists.fedoraproject.org

5 participants
14 discussions

[PATCH] mkdumprd: allow spaces after 'path' config phrase when network dump
by Kazuhito Hagio 26 May '22

26 May '22

Without this patch, when there are two or more spaces after 'path' configuration phrase with ssh or nfs setting, SAVE_PATH is set to '/var/crash' in mkdumprd, and in most cases kdump service fails to start. ssh kdump(a)192.168.122.1 path /kdump ^^ This behavior would be too sensitive and different from the other configurations. With this patch, mkdumprd allows such spaces. Signed-off-by: Kazuhito Hagio <k-hagio(a)ab.jp.nec.com> --- mkdumprd | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mkdumprd b/mkdumprd index a6f7fe8..aa0abfd 100644 --- a/mkdumprd +++ b/mkdumprd @@ -13,7 +13,7 @@ export IN_KDUMP=1 conf_file="/etc/kdump.conf" SSH_KEY_LOCATION="/root/.ssh/kdump_id_rsa" -SAVE_PATH=$(grep ^path $conf_file| cut -d' ' -f2) +SAVE_PATH=$(awk '/^path/ {print $2}' $conf_file) [ -z "$SAVE_PATH" ] && SAVE_PATH=$DEFAULT_PATH # strip the duplicated "/" SAVE_PATH=$(echo $SAVE_PATH | tr -s /) -- 2.18.0

4 3

[PATCH] kdump-lib.sh: introduce functions to return recommened mem size
by Pingfan Liu 03 Feb '21

03 Feb '21

There is requirement to decide the recommended memory size for the current system. Ant the algorithm is based on /proc/iomem, so it can align with the algorithm used by reserve_crashkernel() in kernel. Signed-off-by: Pingfan Liu <piliu(a)redhat.com> --- V1 -> V2: embeded in kdump-lib.sh add check against /proc/iomem kdump-lib.sh | 68 ++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 68 insertions(+) diff --git a/kdump-lib.sh b/kdump-lib.sh index d2801da..6bb285d 100755 --- a/kdump-lib.sh +++ b/kdump-lib.sh @@ -823,3 +823,71 @@ prepare_cmdline() echo ${cmdline} } + +#get system memory size in the unit of GB +get_system_size() +{ + result=$( cat /proc/iomem | grep "System RAM" | awk -F ":" {' print $1 '} | tr [:lower:] [:upper:] | paste -sd+ ) + result="+$result" + # replace '-' with '+0x' and '+' with '-0x' + sum=$( echo $result | sed -e 's/-/K0x/g' | sed -e 's/+/-0x/g' | sed -e 's/K/+/g' ) + size=$(printf "%d\n" $(($sum))) + let size=$size/1024/1024/1024 + + echo $size +} + +get_recommend_size() +{ + local mem_size=$1 + local _ck_cmdline=$2 + + last_sz="" + last_unit="" + + IFS=',' + for i in $_ck_cmdline + do + end=$( echo $i | awk -F "-" {' print $2 '} | awk -F ":" {' print $1 '} ) + recommend=$( echo $i | awk -F "-" {' print $2 '} | awk -F ":" {' print $2 '} ) + size=${end: : -1} + unit=${end: -1} + if [ $unit == 'T' ]; then + let size=$size*1024 + fi + if [ $mem_size -lt $size ]; then + echo $recommend + unset IFS + return + fi + done + unset IFS +} + +# return recommended size based on current system RAM size +kdump_api_get_arch_recommend_size() +{ + if ! [[ -r "/proc/iomem" ]] ; then + echo "Error, can not access /proc/iomem." + return 1 + fi + arch=$( lscpu | grep Architecture | awk -F ":" {' print $2 '} | tr [:lower:] [:upper:] ) + + if [ $arch == "X86_64" ] || [ $arch == "S390" ]; then + ck_cmdline="1G-4G:160M,4G-64G:192M,64G-1T:256M,1T-:512M" + elif [ $arch == "ARM64" ]; then + ck_cmdline="2G-:448M" + elif [ $arch == "PPC64LE" ]; then + if is_fadump_capable; then + ck_cmdline="4G-16G:768M,16G-64G:1G,64G-128G:2G,128G-1T:4G,1T-2T:6G,2T-4T:12G,4T-8T:20G,8T-16T:36G,16T-32T:64G,32T-64T:128G,64T-:180G" + else + ck_cmdline="2G-4G:384M,4G-16G:512M,16G-64G:1G,64G-128G:2G,128G-:4G" + fi + fi + + ck_cmdline=$( echo $ck_cmdline | sed -e 's/-:/-102400T:/g' ) + sys_mem=$(get_system_size) + result=$( get_recommend_size $sys_mem "$ck_cmdline" ) + echo $result + return 0 +} -- 2.29.2

3 2

[PATCH] Revert "Append both nofail and x-systemd.before to kdump mount target"
by Kairui Song 01 Feb '21

01 Feb '21

That commit is trying to workaround a kernel VFS bug. Now, the VFS issue should have been fixed in all recent releases, so remove this workaround. This reverts commit 539bff40830e961d7ef85248e39bff0e40b23c91. Signed-off-by: Kairui Song <kasong(a)redhat.com> --- mkdumprd | 3 --- 1 file changed, 3 deletions(-) diff --git a/mkdumprd b/mkdumprd index 0623940..8ba8e2a 100644 --- a/mkdumprd +++ b/mkdumprd @@ -86,9 +86,6 @@ to_mount() { # drop nofail or nobootwait _options=$(echo $_options | sed 's/$^\|,$nofail$$\|,$/\1/g') _options=$(echo $_options | sed 's/$^\|,$nobootwait$$\|,$/\1/g') - # use both nofail and x-systemd.before to ensure systemd will try best to - # mount it before kdump starts, this is an attempt to improve robustness - _options="$_options,nofail,x-systemd.before=initrd-fs.target" echo "$_pdev $_new_mntpoint $_fstype $_options" } -- 2.29.2

3 2

[PATCH] Remove trace_buf_size and trace_event from the kernel boot parameters of the kdump kernel
by fj1508ic＠fujitsu.com 26 Jan '21

26 Jan '21

The kdump kernel uses resources for ftrace because trace_buf_size, which specifies the ring buffer size for ftrace, and trace_event, which specifies a valid trace event, are not removed, but the kdump kernel does not require ftrace. trace_buf_size is ignored if the specified size is 0, so specify 1. Signed-off-by: Hisashi Nagaoka <fj1508ic(a)fujitsu.com> --- kdump-lib.sh | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/kdump-lib.sh b/kdump-lib.sh index 0e38580..a54a650 100755 --- a/kdump-lib.sh +++ b/kdump-lib.sh @@ -827,5 +827,9 @@ prepare_cmdline() fi done + # Remove trace_buf_size, trace_event + cmdline=$(remove_cmdline_param "$cmdline" trace_buf_size trace_event) + cmdline="${cmdline} trace_buf_size=1" + echo ${cmdline} } -- 2.26.2

1 0

[PATCH] dracut-module-setup.sh: enable ForwardToConsole=yes in fadump mode
by Pingfan Liu 22 Jan '21

22 Jan '21

In fadump mode, it is also useful to observe kdump message through console. Hence enable it. Signed-off-by: Pingfan Liu <piliu(a)redhat.com> --- dracut-module-setup.sh | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/dracut-module-setup.sh b/dracut-module-setup.sh index 21f7105..6061dbe 100755 --- a/dracut-module-setup.sh +++ b/dracut-module-setup.sh @@ -813,13 +813,11 @@ kdump_install_systemd_conf() { # Forward logs to console directly, and don't read Kmsg, this avoids # unneccessary memory consumption and make console output more useful. # Only do so for non fadump image. - if ! is_fadump_capable; then - mkdir -p ${initdir}/etc/systemd/journald.conf.d - echo "[Journal]" > ${initdir}/etc/systemd/journald.conf.d/kdump.conf - echo "Storage=volatile" >> ${initdir}/etc/systemd/journald.conf.d/kdump.conf - echo "ReadKMsg=no" >> ${initdir}/etc/systemd/journald.conf.d/kdump.conf - echo "ForwardToConsole=yes" >> ${initdir}/etc/systemd/journald.conf.d/kdump.conf - fi + mkdir -p ${initdir}/etc/systemd/journald.conf.d + echo "[Journal]" > ${initdir}/etc/systemd/journald.conf.d/kdump.conf + echo "Storage=volatile" >> ${initdir}/etc/systemd/journald.conf.d/kdump.conf + echo "ReadKMsg=no" >> ${initdir}/etc/systemd/journald.conf.d/kdump.conf + echo "ForwardToConsole=yes" >> ${initdir}/etc/systemd/journald.conf.d/kdump.conf } install() { -- 2.29.2

2 1

[PATCH v2 0/3] Fix dracut error caused by kexec-tools
by Kairui Song 19 Jan '21

19 Jan '21

Following errors are observed when kexec-tools is installed: $ LC_ALL=C sudo rpm -ivh kernel-core-5.9.16-200.fc33.x86_64.rpm Verifying... ######################################## Preparing... ######################################## Updating / installing... kernel-core-5.9.16-200.fc33 ######################################## sort: fflush failed: 'standard output': Broken pipe sort: write error gzip: stdout: Broken pipe gzip: stdout: Broken pipe sort: write failed: 'standard output': Broken pipe sort: write error This is caused by kexec-tools' logger helper overriding dracut's helper functions. kdumpcase and earlykdump's module-setup.sh will unconditionally source kdump lib files, and polute dracut namespace. This series make the module-setup.sh file only import kdump helper when building a kdump initramfs, and don't import logger lib, as dracut module should just use dracut's logger. Update from V1: V1 forgot to update earlykdump's module-setup.sh Kairui Song (3): logger: source the logger file individually module-setup.sh: don't source $dracutfunctions module-setup.sh: don't polute the namespace unnecessarily dracut-early-kdump-module-setup.sh | 3 ++- dracut-early-kdump.sh | 1 + dracut-module-setup.sh | 14 +++++++++----- kdump-dep-generator.sh | 1 + kdump-lib-initramfs.sh | 1 + kdump-lib.sh | 6 ------ kdumpctl | 1 + mkdumprd | 1 + 8 files changed, 16 insertions(+), 12 deletions(-) -- 2.29.2

2 4

[PATCH 0/3] Fix dracut error caused by kexec-tools
by Kairui Song 19 Jan '21

19 Jan '21

Following errors are observed when kexec-tools is installed: $ LC_ALL=C sudo rpm -ivh kernel-core-5.9.16-200.fc33.x86_64.rpm Verifying... ######################################## Preparing... ######################################## Updating / installing... kernel-core-5.9.16-200.fc33 ######################################## sort: fflush failed: 'standard output': Broken pipe sort: write error gzip: stdout: Broken pipe gzip: stdout: Broken pipe sort: write failed: 'standard output': Broken pipe sort: write error This is caused by kexec-tools' logger helper overriding dracut's helper functions. kdumpcase and earlykdump's module-setup.sh will unconditionally source kdump lib files, and polute dracut namespace. This series make the module-setup.sh file only import kdump helper when building a kdump initramfs, and don't import logger lib, as dracut module should just use dracut's logger. Kairui Song (3): logger: source the logger file individually module-setup.sh: don't source $dracutfunctions module-setup.sh: don't polute the namespace unnecessarily dracut-early-kdump.sh | 1 + dracut-module-setup.sh | 14 +++++++++----- kdump-dep-generator.sh | 1 + kdump-lib-initramfs.sh | 1 + kdump-lib.sh | 6 ------ kdumpctl | 1 + mkdumprd | 1 + 7 files changed, 14 insertions(+), 11 deletions(-) -- 2.29.2

1 4

[PATCH] kdump.conf: add ipv6 example for nfs and ssh dump
by Lianbo Jiang 13 Jan '21

13 Jan '21

At present, there is no ipv6 example for nfs and ssh dump, let's add an example to the kdump.conf. Signed-off-by: Lianbo Jiang <lijiang(a)redhat.com> --- kdump.conf | 2 ++ 1 file changed, 2 insertions(+) diff --git a/kdump.conf b/kdump.conf index e4db52e2db97..f9d0d4214881 100644 --- a/kdump.conf +++ b/kdump.conf @@ -166,7 +166,9 @@ #ext4 LABEL=/boot #ext4 UUID=03138356-5e61-4ab3-b58e-27507ac41937 #nfs my.server.com:/export/tmp +#nfs [2620:52:0:800:216:3eff:fe6a:e19e]:/export/tmp #ssh user(a)my.server.com +#ssh user@2620:52:0:800:216:3eff:fe6a:e19e #sshkey /root/.ssh/kdump_id_rsa path /var/crash core_collector makedumpfile -l --message-level 7 -d 31 -- 2.17.1

2 1

[PATCH] fix kdump failure of saving vmcore with the scp + ipv6 method
by Lianbo Jiang 13 Jan '21

13 Jan '21

Currently, kdump will fail to save vmcore when using the scp and ipv6. The reason is that the scp requires IPv6 addresses to be enclosed in square brackets, but ssh doesn’t require this. Let's enclose the ipv6 address in square brackets for scp dump. Signed-off-by: Lianbo Jiang <lijiang(a)redhat.com> --- dracut-kdump.sh | 32 +++++++++++++++++++++++++++++--- kdump-lib-initramfs.sh | 1 + 2 files changed, 30 insertions(+), 3 deletions(-) diff --git a/dracut-kdump.sh b/dracut-kdump.sh index 370d217cea52..3367bc5f7c63 100755 --- a/dracut-kdump.sh +++ b/dracut-kdump.sh @@ -111,6 +111,7 @@ dump_ssh() local _dir="$KDUMP_PATH/$HOST_IP-$DATEDIR" local _host=$2 local _vmcore="vmcore" + local _ipv6_addr="" _username="" dinfo "saving to $_host:$_dir" @@ -122,8 +123,17 @@ dump_ssh() dinfo "saving vmcore" + if is_ipv6_address "$_host"; then + _username=${_host%@*} + _ipv6_addr="[${_host#*@}]" + fi + if [ "${CORE_COLLECTOR%%[[:blank:]]*}" = "scp" ]; then - scp -q $_opt /proc/vmcore "$_host:$_dir/vmcore-incomplete" + if [ -n "$_username" ] && [ -n "$_ipv6_addr" ]; then + scp -q $_opt /proc/vmcore "$_username@$_ipv6_addr:$_dir/vmcore-incomplete" + else + scp -q $_opt /proc/vmcore "$_host:$_dir/vmcore-incomplete" + fi _exitcode=$? else $CORE_COLLECTOR /proc/vmcore | ssh $_opt $_host "dd bs=512 of=$_dir/vmcore-incomplete" @@ -143,8 +153,13 @@ dump_ssh() derror "saving vmcore failed, _exitcode:$_exitcode" fi + dinfo "saving the $KDUMP_LOG_FILE to $_host:$_dir/" save_log - scp -q $_opt $KDUMP_LOG_FILE "$_host:$_dir/" + if [ -n "$_username" ] && [ -n "$_ipv6_addr" ]; then + scp -q $_opt $KDUMP_LOG_FILE "$_username@$_ipv6_addr:$_dir/" + else + scp -q $_opt $KDUMP_LOG_FILE "$_host:$_dir/" + fi _ret=$? if [ $_ret -ne 0 ]; then derror "saving log file failed, _exitcode:$_ret" @@ -161,6 +176,7 @@ save_opalcore_ssh() { local _path=$1 local _opts="$2" local _location=$3 + local _user_name="" _ipv6addr="" ddebug "_path=$_path _opts=$_opts _location=$_location" @@ -173,8 +189,18 @@ save_opalcore_ssh() { fi fi + if is_ipv6_address "$_host"; then + _user_name=${_location%@*} + _ipv6addr="[${_location#*@}]" + fi + dinfo "saving opalcore:$OPALCORE to $_location:$_path" - scp $_opts $OPALCORE $_location:$_path/opalcore-incomplete + + if [ -n "$_user_name" ] && [ -n "$_ipv6addr" ]; then + scp $_opts $OPALCORE $_user_name@$_ipv6addr:$_path/opalcore-incomplete + else + scp $_opts $OPALCORE $_location:$_path/opalcore-incomplete + fi if [ $? -ne 0 ]; then derror "saving opalcore failed" return 1 diff --git a/kdump-lib-initramfs.sh b/kdump-lib-initramfs.sh index e766f95c7d6a..4eba3db75673 100755 --- a/kdump-lib-initramfs.sh +++ b/kdump-lib-initramfs.sh @@ -168,6 +168,7 @@ dump_fs() derror "saving vmcore failed, _exitcode:$_exitcode" fi + dinfo "saving the $KDUMP_LOG_FILE to $_dump_path/" save_log mv $KDUMP_LOG_FILE $_dump_path/ if [ $_exitcode -ne 0 ]; then -- 2.17.1

2 1

[PATCH] Fix dump_fs mount point detection and fallback mount
by Kairui Song 13 Jan '21

13 Jan '21

Simplify the code and fix mount point detection. The code logic is now much simpler: if $1 is not a mount point, call "mount --target $1" again to try mount it. "mount --target" cmd itself can handle all the /etc/fstab parsing job, so drop the buggy and complex bash code. Signed-off-by: Kairui Song <kasong(a)redhat.com> --- kdump-lib-initramfs.sh | 27 ++++++++------------------- 1 file changed, 8 insertions(+), 19 deletions(-) diff --git a/kdump-lib-initramfs.sh b/kdump-lib-initramfs.sh index a60a52a..cd9d512 100755 --- a/kdump-lib-initramfs.sh +++ b/kdump-lib-initramfs.sh @@ -117,26 +117,15 @@ dump_fs() { local _exitcode local _mp=$1 - local _dev=$(get_mount_info SOURCE target $_mp -f) - local _op=$(get_mount_info OPTIONS target $_mp -f) - - ddebug "_mp=$_mp _dev=$_dev _op=$_op" - - # If dump path have a corresponding device entry but not mounted, mount it. - if [ -n "$_dev" ] && [ "$_dev" != "rootfs" ]; then - if ! is_mounted "$_mp"; then - dinfo "dump target $_dev is not mounted, trying to mount..." - mkdir -p $_mp - mount -o $_op $_dev $_mp - - if [ $? -ne 0 ]; then - derror "mounting failed (mount point: $_mp, option: $_op)" - return 1 - fi + ddebug "dump_fs _mp=$_mp" + + if ! is_mounted "$_mp"; then + dinfo "dump path \"$_mp\" is not mounted, trying to mount..." + mount --target $_mp + if [ $? -ne 0 ]; then + derror "failed to dump to \"$_mp\", it's not a mount point!" + return 1 fi - else - derror "failed to dump to \"$_mp\", it's not a mount point!" - return 1 fi # Remove -F in makedumpfile case. We don't want a flat format dump here. -- 2.29.2

2 1

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

kexec January 2021