[Patch v2] module-setup: Don't handle iBFT in kdump
by Baoquan He
There are several kinds of iSCSI mode rhel support currently.
- Pure hardware iSCSI
- iBFT iSCSI
- Pure software iSCSI
Except for the 1st one that firmware takes care of everything to
make it behave like a local disk, both iBFT and pure software iSCSI
mode need pass information to kdump kernel for configuring them
correctly.
Currently kdump takes iBFT mode as a software iSCSI and collects
the related information to set up software iSCSI in 2nd kernel,
though dracut can detect and collect information to set up iBFT
iSCSI of 2nd kernel. This brings up 2 problems:
1) For iBFT mode 2 sessions will be built in 2nd kernel, one is in
iBFT mode, the other is pure software mode. This is because both
kdump and dracut collect iSCSI inforamtion separately for a certain
iBFT mode iSCSI of 1st kernel.
2) These 2 sessions of 2nd kernel for a certain session of 1st kernel
could contain two "ip=xxx" cmdline option. This will cause cmdline
handling error in dracut.
In order to avoid above 2 problems, kdump need detect iBFT mode iSCSI
and leave it to dracut. This is what is donw in this patch.
Signed-off-by: Baoquan He <bhe(a)redhat.com>
---
dracut-module-setup.sh | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/dracut-module-setup.sh b/dracut-module-setup.sh
index ecf7d3f..aa204f8 100755
--- a/dracut-module-setup.sh
+++ b/dracut-module-setup.sh
@@ -511,7 +511,11 @@ kdump_get_iscsi_initiator() {
return 1
}
-# No ibft handling yet.
+# Figure out iBFT session according to session type
+is_ibft() {
+ [ "$(kdump_iscsi_get_rec_val $1 "node.discovery_type")" = fw ]
+}
+
kdump_setup_iscsi_device() {
local path=$1
local tgt_name; local tgt_ipaddr;
@@ -534,6 +538,10 @@ kdump_setup_iscsi_device() {
return 1
fi
+ if is_ibft ${path}; then
+ return
+ fi
+
tgt_name=$(kdump_iscsi_get_rec_val ${path} "node.name")
tgt_ipaddr=$(kdump_iscsi_get_rec_val ${path} "node.conn\[0\].address")
--
2.5.5
6 years, 11 months
Fwd: Re: [PATCH 1/2] kdump.conf comments fixes
by Dave Young
----- Forwarded message from Donald Berry <dberry(a)redhat.com> -----
> Date: Wed, 22 Jun 2016 13:02:22 -0400 (EDT)
> From: Donald Berry <dberry(a)redhat.com>
> To: zhezhang(a)redhat.com
> Cc: dyoung(a)redhat.com, panand(a)redhat.com, kexec(a)lists.fedoraproject.org
> Subject: Re: [PATCH 1/2] kdump.conf comments fixes
>
> My suggestions for /etc/kdump.conf.
>
> Don
>
> ----- Original Message -----
> From: zhezhang(a)redhat.com
> To: kexec(a)lists.fedoraproject.org
> Cc: dyoung(a)redhat.com, dberry(a)redhat.com, panand(a)redhat.com, "Freeman Zhang" <zhezhang(a)redhat.com>
> Sent: Wednesday, June 22, 2016 5:22:15 AM
> Subject: [PATCH 1/2] kdump.conf comments fixes
>
> The default action comment about "halt" is wrong, default action means
> the action to perform after a vmcore saving failure.
>
> Also there are lots of typos and incorrect expressions. Fix them here as well.
>
> Signed-off-by: Freeman Zhang <zhezhang(a)redhat.com>
> Reported-by: Donald Berry <dberry(a)redhat.com>
> Reviewed-by: Dave Young <dyoung(a)redhat.com>
> ---
> kdump.conf | 89 ++++++++++++++++++++++++++++---------------------------------
> 1 file changed, 42 insertions(+), 47 deletions(-)
>
> --- kexec-tools.orig/kdump.conf
> +++ kexec-tools/kdump.conf
> @@ -3,14 +3,14 @@
> # This file contains a series of commands to perform (in order) when a
> # kernel crash has happened and the kdump kernel has been loaded. Directives in
> # this file are only applicable to the kdump initramfs, and have no effect if
> -# the root filesystem is mounted and the normal init scripts are processed
> +# the root filesystem is mounted and the normal init scripts are processed.
> #
> -# Currently only one dump target and path may be configured at once
> -# if the configured dump target fails, the default action will be preformed
> -# the default action may be configured with the default directive below. If the
> -# configured dump target succedes
> +# Currently, only one dump target and path can be specified. If the configured
> +# dump target fails, the default action will be performed. The default action
> +# can be configured with the default directive.
> +#
> +# Supported options:
> #
> -# Basics commands supported are:
> # raw <partition> - Will dd /proc/vmcore into <partition>.
> # Use persistent device names for partition devices,
> # such as /dev/vg/<devname>.
> @@ -21,11 +21,11 @@
> # ssh <user@server> - Will scp /proc/vmcore to
> # <user@server>:/var/crash/%HOST-%DATE/, supports DNS
> # NOTE: make sure user has necessary write
> -# permissions on server
> +# permissions on the server.
> #
> -# sshkey <path> - Will use the sshkey to do ssh dump
> -# Specifies the path of the ssh key you want to use
> -# when do ssh dump, the default value is
> +# sshkey <path> - Will use the sshkey to do ssh dump.
> +# Specifies the path of the ssh key to use
> +# when dumping via ssh. The default value is
> # /root/.ssh/kdump_id_rsa.
>
> sshkey <path>
> Specifies the path of the ssh key to use when dumping via ssh. The default value is /root/.ssh/kdump_id_rsa.
>
> #
> # <fs type> <partition> - Will mount -t <fs type> <partition> /mnt and copy
> @@ -50,17 +50,15 @@
> #
> # core_collector <command> <options>
> # - This allows you to specify the command to copy
> -# the vmcore. You could use the dump filtering
> -# program makedumpfile, the default one, to retrieve
> -# your core, which on some arches can drastically
> -# reduce core file size. See /sbin/makedumpfile --help
> -# for a list of options. Note that the -i and -g
> -# options are not needed here, as the initrd will
> -# automatically be populated with a config file
> -# appropriate for the running kernel.
> -# Default core_collector for raw/ssh dump is:
> +# the vmcore. The default is makedumpfile, which on
> +# some architectures can drastically reduce core file size.
> +# See /sbin/makedumpfile --help for a list of options.
> +# Note that the -i and -g options are not needed here,
> +# as the initrd will automatically be populated with a
> +# config file appropriate for the running kernel.
> +# The default core_collector for raw/ssh dump is:
> # "makedumpfile -F -l --message-level 1 -d 31".
> -# Default core_collector for other targets is:
> +# The default core_collector for other targets is:
> # "makedumpfile -l --message-level 1 -d 31".
> # For core_collector format details please refer to
> # kexec-kdump-howto.txt or kdump.conf manpage.
> @@ -70,13 +68,13 @@
> # executable just after the memory dump process
> # terminates. The exit status from the dump process
> # is fed to the kdump_post executable, which can be
> -# used to trigger different actions for success or
> -# failure.
> +# used to trigger actions for success or failure
> +# respectively.
> #
>
> core_collector <command> <options>
> This allows you to specify the command used to copy the vmcore. The default is makedumpfile, which on some architectures can drastically reduce
> core file size. See /sbin/makedumpfile --help for a list of options. Note that the -i and -g options are not needed here, as the initrd will
> automatically be populated with a config file appropriate for the running kernel.
>
> The default core_collector for raw/ssh dump is: "makedumpfile -F -l --message-level 1 -d 31". The default
> core_collector for other targets is: "makedumpfile -l --message-level 1 -d 31".
>
> If "makedumpfile -F" is used then you will get a flattened format vmcore.flat. You will need to use "makedumpfile -R" to rearrange the
> dump data from standard input to a normal dumpfile (readable with analysis tools). i.e. "makedumpfile -R vmcore < vmcore.flat"
>
> # kdump_pre <binary | script>
> -# - works just like the kdump_post directive, but instead
> +# - Works just like the kdump_post directive, but instead
> # of running after the dump process, runs immediately
> -# before. Exit status of this binary is interpreted
> +# before. Exit status of this binary is interpreted
>
> Double spaces are correct between a period and the start of the next sentence.
>
> # as follows:
> # 0 - continue with dump process as usual
> # non 0 - reboot the system
> @@ -94,23 +92,18 @@
> # initrd, typically used to set up access to
> # non-boot-path dump targets that might otherwise
> # not be accessible in the kdump environment. Multiple
> -# modules can be listed, separated by a space, and any
> +# modules can be listed, separated by spaces, and any
> # dependent modules will automatically be included.
> #
> # default <reboot | halt | poweroff | shell | dump_to_rootfs>
> -# - Action to preform in case dumping to intended target
> -# fails. If no default action is specified, "reboot"
> -# is assumed default.
> -# reboot: If the default action is reboot simply reboot
> -# the system and loose the core that you are
> -# trying to retrieve.
> -# halt: If the default action is halt, then simply
> -# halt the system after attempting to capture
> -# a vmcore, regardless of success or failure.
> -# poweroff: The system will be powered down
> -# shell: If the default action is shell, then drop to
> -# an shell session inside the initramfs from
> -# where you can try to record the core manually.
> +# - Action to perform in case dumping to intended target
> +# fails. The default is "reboot".
> +# reboot: Reboot the system and lose the vmcore.
> +# halt: Halt the system and lose the vmcore.
> +# poweroff: Power down the system.
> +# shell: Drop to a shell session inside the initramfs,
> +# then you can try to save the core
> +# manually.
> # Exiting this shell reboots the system.
> # Note: kdump uses bash as the default shell.
> # dump_to_rootfs: If non-root dump target is specified,
> @@ -119,15 +112,16 @@
> # to rootfs from initramfs context and reboot.
> #
>
> default <reboot | halt | poweroff | shell | dump_to_rootfs>
> Action to perform in case dumping to the intended target fails. The default is "reboot".
> reboot: reboot the system (this is what most people will want, as it returns the system to a nominal state).
> shell: drop to a shell session inside the initramfs, from which you can try to save the core manually. Exiting this shell reboots the system.
> Note: kdump uses bash as the default shell.
> halt: bring the system to a halt, requiring manual reset.
> poweroff: The system will be powered down.
> dump_to_rootfs: the specified root will be mounted and the dump will be saved in the "path" directory.
>
> # force_rebuild <0 | 1>
> -# - By default, kdump initrd only will be rebuilt when
> +# - By default, kdump initrd will only be rebuilt when
> # necessary. Specify 1 to force rebuilding kdump
> # initrd every time when kdump service starts.
> #
> -#override_resettable <0 | 1>
> -# - Usually a unresettable block device can't be dump target.
> -# Specifying 1 means though block target is unresettable, user
> -# understand this situation and want to try dumping. By default,
> -# it's set to 0, means not to try a destined failure.
> +# override_resettable <0 | 1>
> +# - Usually an unresettable block device can't be a dump
> +# target. Specifying 1 means that even though the block
> +# target is unresettable, the user wants to try dumping
> +# anyway. By default, it's set to 0, which will not try
> +# something destined to fail.
> #
> # dracut_args <arg(s)>
> # - Pass extra dracut options when rebuilding kdump
> @@ -135,11 +129,12 @@
> #
> # fence_kdump_args <arg(s)>
> # - Command line arguments for fence_kdump_send (it can contain
> -# all valid arguments except hosts to send notification to).
> +# all valid arguments except hosts to send notification to).
> #
> # fence_kdump_nodes <node(s)>
> -# - List of cluster node(s) separated by space to send fence_kdump
> -# notification to (this option is mandatory to enable fence_kdump).
> +# - List of cluster node(s), separated by spaces, to send
> +# fence_kdump notifications to (this option is mandatory to
> +# enable fence_kdump).
> #
>
> #raw /dev/vg/lv_kdump
>
>
>
----- End forwarded message -----
6 years, 11 months
Fwd: Re: [PATCH 2/2] kdump.conf man page fixes
by Dave Young
----- Forwarded message from Donald Berry <dberry(a)redhat.com> -----
> Date: Wed, 22 Jun 2016 13:14:24 -0400 (EDT)
> From: Donald Berry <dberry(a)redhat.com>
> To: zhezhang(a)redhat.com
> Cc: dyoung(a)redhat.com, panand(a)redhat.com, kexec(a)lists.fedoraproject.org
> Subject: Re: [PATCH 2/2] kdump.conf man page fixes
>
> My suggestions for kdump.conf man page, prefaced with 'DB:'.
>
> Don
>
> ----- Original Message -----
> From: zhezhang(a)redhat.com
> To: kexec(a)lists.fedoraproject.org
> Cc: dyoung(a)redhat.com, dberry(a)redhat.com, panand(a)redhat.com, "Freeman Zhang" <zhezhang(a)redhat.com>
> Sent: Wednesday, June 22, 2016 5:22:16 AM
> Subject: [PATCH 2/2] kdump.conf man page fixes
>
> Fix the typos and grammar problem in kdump.conf man page.
>
> Signed-off-by: Freeman Zhang <zhezhang(a)redhat.com>
> Reported-by: Donald Berry <dberry(a)redhat.com>
> Reviewed-by: Dave Young <dyoung(a)redhat.com>
> ---
> kdump.conf.5 | 79 +++++++++++++++++++++++++++--------------------------------
> 1 file changed, 37 insertions(+), 42 deletions(-)
>
> --- kexec-tools.orig/kdump.conf.5
> +++ kexec-tools/kdump.conf.5
> @@ -10,14 +10,14 @@ collection service.
>
> kdump.conf provides post-kexec instructions to the kdump kernel. It is
> stored in the initrd file managed by the kdump service. If you change
> -this file and do not want to restart before it takes effect, restart
> -the kdump service to rebuild to initrd.
> +this file and do not want to reboot in order for the changes to take
> +effect, restart the kdump service to rebuild the initrd.
>
> For most configurations, you can simply review the examples provided
> in the stock /etc/kdump.conf.
>
> .B NOTE:
> -For filesystem dump the dump target must be mounted before building
> +For filesystem dumps the dump target must be mounted before building
> kdump initramfs.
>
> kdump.conf only affects the behavior of the initramfs. Please read the
> @@ -48,8 +48,8 @@ server and that a fqdn is used as the se
>
> .B sshkey <path>
> .RS
> -Specifies the path of the ssh key you want to use when do ssh dump,
> -the default value is /root/.ssh/kdump_id_rsa.
> +Specifies the path of the ssh key to use when dumping via ssh.
> +The default value is /root/.ssh/kdump_id_rsa.
> .RE
>
> .B <fs type> <partition>
> @@ -77,26 +77,25 @@ Ignored for raw device dumps. If unset,
> .B core_collector <command> <options>
> .RS
> This allows you to specify the command to copy the vmcore.
> -You could use the dump filtering program makedumpfile, the default one,
> -to retrieve your core, which on some arches can drastically reduce
> -core file size. See /sbin/makedumpfile --help for a list of options.
> +The default is makedumpfile, which on some architectures can drastically reduce
> +core file size. See /sbin/makedumpfile --help for a list of options.
> Note that the -i and -g options are not needed here, as the initrd
> will automatically be populated with a config file appropriate
> for the running kernel.
> .PP
> Note 1: About default core collector:
> -Default core_collector for raw/ssh dump is:
> +The default core_collector for raw/ssh dump is:
> "makedumpfile -F -l --message-level 1 -d 31".
> -Default core_collector for other targets is:
> +The default core_collector for other targets is:
> "makedumpfile -l --message-level 1 -d 31".
> Even if core_collector option is commented out in kdump.conf, makedumpfile
> -is default core collector and kdump uses it internally.
> +is the default core collector and kdump uses it internally.
> If one does not want makedumpfile as default core_collector, then they
> need to specify one using core_collector option to change the behavior.
> .PP
> Note 2: If "makedumpfile -F" is used then you will get a flattened format
> vmcore.flat, you will need to use "makedumpfile -R" to rearrange the
> -dump data from stdard input to a normal dumpfile (readable with analysis
> +dump data from standard input to a normal dumpfile (readable with analysis
> tools).
> ie. "makedumpfile -R vmcore < vmcore.flat"
>
> @@ -108,11 +107,10 @@ This directive allows you to run a speci
> executable just after the memory dump process
> terminates. The exit status from the dump process
> is fed to the kdump_post executable, which can be
> -used to trigger different actions for success or
> -failure.
> +used to trigger actions for success or failure respectively.
> .PP
> Note that scripts written for use with this
> -directive must use the /bin/bash interpreter
> +directive must use the /bin/bash interpreter.
> .RE
>
> .B kdump_pre <binary | script>
> @@ -127,7 +125,7 @@ as follows:
> non 0 - reboot the system
> .PP
> Note that scripts written for this directive must use
> -the /bin/bash interpreter
> +the /bin/bash interpreter.
> .RE
>
> .B extra_bins <binaries | shell scripts>
> @@ -146,36 +144,33 @@ modules that you want to be loaded in th
> initrd, typically used to set up access to
> non-boot-path dump targets that might otherwise
> not be accessible in the kdump environment. Multiple
> -modules can be listed, separated by a space, and any
> +modules can be listed, separated by spaces, and any
> dependent modules will automatically be included.
> .RE
>
> .B default <reboot | halt | poweroff | shell | dump_to_rootfs>
> .RS
> -Action to preform in case dumping to intended target fails. If no default
> -action is specified, "reboot" is assumed default.
> -reboot: If the default action is reboot simply reboot the system (this is what
> -most people will want, as it returns the system to a nominal state). shell: If the default
> -action is shell, then drop to an shell session inside the initramfs from
> -where you can manually preform additional recovery actions. Exiting this shell
> -reboots the system. halt: bring the system to a halt, requiring manual reset
> -poweroff: The system will be powered down. dump_to_rootfs:If the default action
> -is dump_to_rootfs, specified root will be mounted and dump will be saved in "path"
> -directory.
> +Action to perform in case dumping to intended target fails. The default is "reboot".
>
> DB: Action to perform in case dumping to the intended target fails. The default is "reboot".
>
> +reboot: reboot the system (this is what most people will want, as it returns the
> +system to a normal state). shell: drop to a shell session inside the initramfs,
> +from where you can manually perform additional recovery actions. Exiting this shell
>
> DB: from which you can manually perform additional recovery actions. Exiting this shell
>
> +reboots the system. halt: bring the system to a halt, requiring manual reset.
> +poweroff: the system will be powered down. dump_to_rootfs: specified root will
> +be mounted and dump will be saved in "path" directory.
> Note: kdump uses bash as the default shell.
>
> .RE
>
> .B force_rebuild <0 | 1>
> .RS
> -By default, kdump initrd only will be rebuilt when necessary.
> +By default, kdump initrd will only be rebuilt when necessary.
> Specify 1 to force rebuilding kdump initrd every time when kdump service starts.
> .RE
>
> .B override_resettable <0 | 1>
> .RS
> -Usually a unresettable block device can't be dump target. Specifying 1 means
> -though block target is unresettable, user understand this situation and want
> -to try dumping. By default, it's set to 0, means not to try a destined failure.
> +Usually an unresettable block device can't be a dump target. Specifying 1 means
> +that even though block target is unresettable, user wants to try dumping anyway.
>
> DB: that even though the block target is unresettable, the user wants to try dumping anyway.
>
> +By default, it's set to 0, which will not try something destined to fail.
> .RE
>
>
> @@ -195,7 +190,7 @@ arguments except hosts to send notificat
>
> .B fence_kdump_nodes <node(s)>
> .RS
> -List of cluster node(s) separated by space to send fence_kdump notification
> +List of cluster node(s), separated by spaces, to send fence_kdump notification
> to (this option is mandatory to enable fence_kdump).
> .RE
>
> @@ -210,26 +205,26 @@ directly.
>
> .B options <module> <option list>
> .RS
> -Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump to add proper
> -module option as kernel command line params. Such as append loop.max_loop=1
> -to limit maximum loop devices to 1.
> +Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump to add module options as
> +kernel command line parameters. For example, specify 'loop.max_loop=1' to limit
> +maximum loop devices to 1.
> .RE
>
> .B link_delay <seconds>
> .RS
> -link_delay was used to wait a network device to initialize before using it.
> -Now dracut network module take care of this issue automaticlly.
> +link_delay was used to wait for a network device to initialize before using it.
> +Now dracut network module takes care of this issue automatically.
> .RE
>
> .B disk_timeout <seconds>
> .RS
> -Similar to link_delay, dracut ensures disks being ready before kdump uses them.
> +Similar to link_delay, dracut ensures disks are ready before kdump uses them.
> .RE
>
> .B debug_mem_level <0-3>
> .RS
> -This was used to turns on debug/verbose output of kdump scripts regarding
> -free/used memory at various points of execution. This feature has been
> +Turn on verbose debug output of kdump scripts regarding free/used memory at
> +various points of execution. This feature has been
> moved to dracut now.
> Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump and
> append dracut cmdline param rd.memdebug=[0-3] to enable the debug output.
> @@ -253,7 +248,7 @@ present in initramfs but it is not actua
> retaining blacklist option creates more confusing behavior. It has been
> deprecated.
> .PP
> -Instead use rd.driver.blacklist option on second kernel to blacklist
> +Instead, use rd.driver.blacklist option on second kernel to blacklist
> a certain module. One can edit /etc/sysconfig/kdump.conf and edit
> KDUMP_COMMANDLINE_APPEND to pass kernel command line options. Refer
> to dracut.cmdline man page for more details on module blacklist option.
> @@ -262,7 +257,7 @@ to dracut.cmdline man page for more deta
> .RE
>
> .SH EXAMPLES
> -Here is some examples for core_collector option:
> +Here are some examples for core_collector option:
> .PP
> Core collector command format depends on dump target type. Typically for
> filesystem (local/remote), core_collector should accept two arguments.
>
>
>
----- End forwarded message -----
6 years, 11 months
[PATCH V2] get_persistent_dev(): fix name contention with dracut's similar function
by Pratyush Anand
Resolves: BZ1348898
dracut-functions.sh defines a get_persistent_dev(). Earlier, we had another
local get_persistent_dev() in mkdumprd, however that was moved to
kdump-lib.sh, so that it can be reused in kdumpctl.
Since, dracut-module-setup.sh (which is dracut's
99kdumpbase/module-setup.sh) sources kdump-lib.sh. Therefore, once dracut
will execute 99kdumpbase module, it's own get_persistent_dev() function is
overwritten by kdump's version. If any other dracut module calls
get_persistent_dev() thereafter then, kdump's version is executed, which was
not expected.
Therefore rename kdump's get_persistent_dev() as kdump_get_persistent_dev()
to avoid any name contention.
Signed-off-by: Pratyush Anand <panand(a)redhat.com>
---
changes since v1:
- renamed get_kdump_persistent_dev() to kdump_get_persistent_dev()
kdump-lib.sh | 2 +-
kdumpctl | 2 +-
mkdumprd | 4 ++--
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/kdump-lib.sh b/kdump-lib.sh
index 66f2f0d77401..141a5614f1d5 100755
--- a/kdump-lib.sh
+++ b/kdump-lib.sh
@@ -86,7 +86,7 @@ to_dev_name() {
echo $dev
}
-get_persistent_dev() {
+kdump_get_persistent_dev() {
local i _tmp _dev _lookup_dirs
_dev=$(udevadm info --query=name --name="$1" 2>/dev/null)
diff --git a/kdumpctl b/kdumpctl
index fcc9ad081437..c8fc54fc7677 100755
--- a/kdumpctl
+++ b/kdumpctl
@@ -390,7 +390,7 @@ check_dump_fs_modified()
if [[ $(expr substr $_new_fstype 1 3) = "nfs" ]];then
_new_dev=$_target
else
- _new_dev=$(get_persistent_dev $_target $_new_fstype)
+ _new_dev=$(kdump_get_persistent_dev $_target $_new_fstype)
if ! findmnt $_target >/dev/null; then
echo "Dump target $_target is probably not mounted."
return 2
diff --git a/mkdumprd b/mkdumprd
index 78afb1a9425d..eb0d5e06fac9 100644
--- a/mkdumprd
+++ b/mkdumprd
@@ -109,7 +109,7 @@ to_mount() {
_mntopts="$_target $_fstype $_options"
#for non-nfs _dev converting to use udev persistent name
if [ -b "$_source" ]; then
- _pdev="$(get_persistent_dev $_source $_fstype)"
+ _pdev="$(kdump_get_persistent_dev $_source $_fstype)"
if [ $? -ne 0 ]; then
return 1
fi
@@ -522,7 +522,7 @@ do
dd if=$config_val count=1 of=/dev/null > /dev/null 2>&1 || {
perror_exit "Bad raw disk $config_val"
}
- _praw=$(get_persistent_dev $config_val "raw")
+ _praw=$(kdump_get_persistent_dev $config_val "raw")
if [ $? -ne 0 ]; then
exit 1
fi
--
2.5.5
6 years, 11 months
[PATCH] get_persistent_dev(): fix name contention with dracut's similar function
by Pratyush Anand
Resolves: BZ1348898
dracut-functions.sh defines a get_persistent_dev(). Earlier, we had another
local get_persistent_dev() in mkdumprd, however that was moved to
kdump-lib.sh, so that it can be reused in kdumpctl.
Since, dracut-module-setup.sh (which is dracut's
99kdumpbase/module-setup.sh) sources kdump-lib.sh. Therefore, once dracut
will execute 99kdumpbase module, it's own get_persistent_dev() function is
overwritten by kdump's version. If any other dracut module calls
get_persistent_dev() thereafter then, kdump's version is executed, which was
not expected.
Therefore rename kdump's get_persistent_dev() as get_kdump_persistent_dev()
to avoid any name contention.
Signed-off-by: Pratyush Anand <panand(a)redhat.com>
---
kdump-lib.sh | 2 +-
kdumpctl | 2 +-
mkdumprd | 4 ++--
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/kdump-lib.sh b/kdump-lib.sh
index 66f2f0d77401..0769e2630d6c 100755
--- a/kdump-lib.sh
+++ b/kdump-lib.sh
@@ -86,7 +86,7 @@ to_dev_name() {
echo $dev
}
-get_persistent_dev() {
+get_kdump_persistent_dev() {
local i _tmp _dev _lookup_dirs
_dev=$(udevadm info --query=name --name="$1" 2>/dev/null)
diff --git a/kdumpctl b/kdumpctl
index fcc9ad081437..21d83525680b 100755
--- a/kdumpctl
+++ b/kdumpctl
@@ -390,7 +390,7 @@ check_dump_fs_modified()
if [[ $(expr substr $_new_fstype 1 3) = "nfs" ]];then
_new_dev=$_target
else
- _new_dev=$(get_persistent_dev $_target $_new_fstype)
+ _new_dev=$(get_kdump_persistent_dev $_target $_new_fstype)
if ! findmnt $_target >/dev/null; then
echo "Dump target $_target is probably not mounted."
return 2
diff --git a/mkdumprd b/mkdumprd
index 78afb1a9425d..37b36af37174 100644
--- a/mkdumprd
+++ b/mkdumprd
@@ -109,7 +109,7 @@ to_mount() {
_mntopts="$_target $_fstype $_options"
#for non-nfs _dev converting to use udev persistent name
if [ -b "$_source" ]; then
- _pdev="$(get_persistent_dev $_source $_fstype)"
+ _pdev="$(get_kdump_persistent_dev $_source $_fstype)"
if [ $? -ne 0 ]; then
return 1
fi
@@ -522,7 +522,7 @@ do
dd if=$config_val count=1 of=/dev/null > /dev/null 2>&1 || {
perror_exit "Bad raw disk $config_val"
}
- _praw=$(get_persistent_dev $config_val "raw")
+ _praw=$(get_kdump_persistent_dev $config_val "raw")
if [ $? -ne 0 ]; then
exit 1
fi
--
2.5.5
6 years, 11 months
[PATCH 2/2] kdump.conf man page fixes
by Freeman Zhang
Fix the typos and grammar problem in kdump.conf man page.
Signed-off-by: Freeman Zhang <zhezhang(a)redhat.com>
Reported-by: Donald Berry <dberry(a)redhat.com>
Reviewed-by: Dave Young <dyoung(a)redhat.com>
---
kdump.conf.5 | 79 +++++++++++++++++++++++++++--------------------------------
1 file changed, 37 insertions(+), 42 deletions(-)
--- kexec-tools.orig/kdump.conf.5
+++ kexec-tools/kdump.conf.5
@@ -10,14 +10,14 @@ collection service.
kdump.conf provides post-kexec instructions to the kdump kernel. It is
stored in the initrd file managed by the kdump service. If you change
-this file and do not want to restart before it takes effect, restart
-the kdump service to rebuild to initrd.
+this file and do not want to reboot in order for the changes to take
+effect, restart the kdump service to rebuild the initrd.
For most configurations, you can simply review the examples provided
in the stock /etc/kdump.conf.
.B NOTE:
-For filesystem dump the dump target must be mounted before building
+For filesystem dumps the dump target must be mounted before building
kdump initramfs.
kdump.conf only affects the behavior of the initramfs. Please read the
@@ -48,8 +48,8 @@ server and that a fqdn is used as the se
.B sshkey <path>
.RS
-Specifies the path of the ssh key you want to use when do ssh dump,
-the default value is /root/.ssh/kdump_id_rsa.
+Specifies the path of the ssh key to use when dumping via ssh.
+The default value is /root/.ssh/kdump_id_rsa.
.RE
.B <fs type> <partition>
@@ -77,26 +77,25 @@ Ignored for raw device dumps. If unset,
.B core_collector <command> <options>
.RS
This allows you to specify the command to copy the vmcore.
-You could use the dump filtering program makedumpfile, the default one,
-to retrieve your core, which on some arches can drastically reduce
-core file size. See /sbin/makedumpfile --help for a list of options.
+The default is makedumpfile, which on some architectures can drastically reduce
+core file size. See /sbin/makedumpfile --help for a list of options.
Note that the -i and -g options are not needed here, as the initrd
will automatically be populated with a config file appropriate
for the running kernel.
.PP
Note 1: About default core collector:
-Default core_collector for raw/ssh dump is:
+The default core_collector for raw/ssh dump is:
"makedumpfile -F -l --message-level 1 -d 31".
-Default core_collector for other targets is:
+The default core_collector for other targets is:
"makedumpfile -l --message-level 1 -d 31".
Even if core_collector option is commented out in kdump.conf, makedumpfile
-is default core collector and kdump uses it internally.
+is the default core collector and kdump uses it internally.
If one does not want makedumpfile as default core_collector, then they
need to specify one using core_collector option to change the behavior.
.PP
Note 2: If "makedumpfile -F" is used then you will get a flattened format
vmcore.flat, you will need to use "makedumpfile -R" to rearrange the
-dump data from stdard input to a normal dumpfile (readable with analysis
+dump data from standard input to a normal dumpfile (readable with analysis
tools).
ie. "makedumpfile -R vmcore < vmcore.flat"
@@ -108,11 +107,10 @@ This directive allows you to run a speci
executable just after the memory dump process
terminates. The exit status from the dump process
is fed to the kdump_post executable, which can be
-used to trigger different actions for success or
-failure.
+used to trigger actions for success or failure respectively.
.PP
Note that scripts written for use with this
-directive must use the /bin/bash interpreter
+directive must use the /bin/bash interpreter.
.RE
.B kdump_pre <binary | script>
@@ -127,7 +125,7 @@ as follows:
non 0 - reboot the system
.PP
Note that scripts written for this directive must use
-the /bin/bash interpreter
+the /bin/bash interpreter.
.RE
.B extra_bins <binaries | shell scripts>
@@ -146,36 +144,33 @@ modules that you want to be loaded in th
initrd, typically used to set up access to
non-boot-path dump targets that might otherwise
not be accessible in the kdump environment. Multiple
-modules can be listed, separated by a space, and any
+modules can be listed, separated by spaces, and any
dependent modules will automatically be included.
.RE
.B default <reboot | halt | poweroff | shell | dump_to_rootfs>
.RS
-Action to preform in case dumping to intended target fails. If no default
-action is specified, "reboot" is assumed default.
-reboot: If the default action is reboot simply reboot the system (this is what
-most people will want, as it returns the system to a nominal state). shell: If the default
-action is shell, then drop to an shell session inside the initramfs from
-where you can manually preform additional recovery actions. Exiting this shell
-reboots the system. halt: bring the system to a halt, requiring manual reset
-poweroff: The system will be powered down. dump_to_rootfs:If the default action
-is dump_to_rootfs, specified root will be mounted and dump will be saved in "path"
-directory.
+Action to perform in case dumping to intended target fails. The default is "reboot".
+reboot: reboot the system (this is what most people will want, as it returns the
+system to a normal state). shell: drop to a shell session inside the initramfs,
+from where you can manually perform additional recovery actions. Exiting this shell
+reboots the system. halt: bring the system to a halt, requiring manual reset.
+poweroff: the system will be powered down. dump_to_rootfs: specified root will
+be mounted and dump will be saved in "path" directory.
Note: kdump uses bash as the default shell.
.RE
.B force_rebuild <0 | 1>
.RS
-By default, kdump initrd only will be rebuilt when necessary.
+By default, kdump initrd will only be rebuilt when necessary.
Specify 1 to force rebuilding kdump initrd every time when kdump service starts.
.RE
.B override_resettable <0 | 1>
.RS
-Usually a unresettable block device can't be dump target. Specifying 1 means
-though block target is unresettable, user understand this situation and want
-to try dumping. By default, it's set to 0, means not to try a destined failure.
+Usually an unresettable block device can't be a dump target. Specifying 1 means
+that even though block target is unresettable, user wants to try dumping anyway.
+By default, it's set to 0, which will not try something destined to fail.
.RE
@@ -195,7 +190,7 @@ arguments except hosts to send notificat
.B fence_kdump_nodes <node(s)>
.RS
-List of cluster node(s) separated by space to send fence_kdump notification
+List of cluster node(s), separated by spaces, to send fence_kdump notification
to (this option is mandatory to enable fence_kdump).
.RE
@@ -210,26 +205,26 @@ directly.
.B options <module> <option list>
.RS
-Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump to add proper
-module option as kernel command line params. Such as append loop.max_loop=1
-to limit maximum loop devices to 1.
+Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump to add module options as
+kernel command line parameters. For example, specify 'loop.max_loop=1' to limit
+maximum loop devices to 1.
.RE
.B link_delay <seconds>
.RS
-link_delay was used to wait a network device to initialize before using it.
-Now dracut network module take care of this issue automaticlly.
+link_delay was used to wait for a network device to initialize before using it.
+Now dracut network module takes care of this issue automatically.
.RE
.B disk_timeout <seconds>
.RS
-Similar to link_delay, dracut ensures disks being ready before kdump uses them.
+Similar to link_delay, dracut ensures disks are ready before kdump uses them.
.RE
.B debug_mem_level <0-3>
.RS
-This was used to turns on debug/verbose output of kdump scripts regarding
-free/used memory at various points of execution. This feature has been
+Turn on verbose debug output of kdump scripts regarding free/used memory at
+various points of execution. This feature has been
moved to dracut now.
Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump and
append dracut cmdline param rd.memdebug=[0-3] to enable the debug output.
@@ -253,7 +248,7 @@ present in initramfs but it is not actua
retaining blacklist option creates more confusing behavior. It has been
deprecated.
.PP
-Instead use rd.driver.blacklist option on second kernel to blacklist
+Instead, use rd.driver.blacklist option on second kernel to blacklist
a certain module. One can edit /etc/sysconfig/kdump.conf and edit
KDUMP_COMMANDLINE_APPEND to pass kernel command line options. Refer
to dracut.cmdline man page for more details on module blacklist option.
@@ -262,7 +257,7 @@ to dracut.cmdline man page for more deta
.RE
.SH EXAMPLES
-Here is some examples for core_collector option:
+Here are some examples for core_collector option:
.PP
Core collector command format depends on dump target type. Typically for
filesystem (local/remote), core_collector should accept two arguments.
6 years, 11 months
[PATCH 1/2] kdump.conf comments fixes
by Freeman Zhang
The default action comment about "halt" is wrong, default action means
the action to perform after a vmcore saving failure.
Also there are lots of typos and incorrect expressions. Fix them here as well.
Signed-off-by: Freeman Zhang <zhezhang(a)redhat.com>
Reported-by: Donald Berry <dberry(a)redhat.com>
Reviewed-by: Dave Young <dyoung(a)redhat.com>
---
kdump.conf | 89 ++++++++++++++++++++++++++++---------------------------------
1 file changed, 42 insertions(+), 47 deletions(-)
--- kexec-tools.orig/kdump.conf
+++ kexec-tools/kdump.conf
@@ -3,14 +3,14 @@
# This file contains a series of commands to perform (in order) when a
# kernel crash has happened and the kdump kernel has been loaded. Directives in
# this file are only applicable to the kdump initramfs, and have no effect if
-# the root filesystem is mounted and the normal init scripts are processed
+# the root filesystem is mounted and the normal init scripts are processed.
#
-# Currently only one dump target and path may be configured at once
-# if the configured dump target fails, the default action will be preformed
-# the default action may be configured with the default directive below. If the
-# configured dump target succedes
+# Currently, only one dump target and path can be specified. If the configured
+# dump target fails, the default action will be performed. The default action
+# can be configured with the default directive.
+#
+# Supported options:
#
-# Basics commands supported are:
# raw <partition> - Will dd /proc/vmcore into <partition>.
# Use persistent device names for partition devices,
# such as /dev/vg/<devname>.
@@ -21,11 +21,11 @@
# ssh <user@server> - Will scp /proc/vmcore to
# <user@server>:/var/crash/%HOST-%DATE/, supports DNS
# NOTE: make sure user has necessary write
-# permissions on server
+# permissions on the server.
#
-# sshkey <path> - Will use the sshkey to do ssh dump
-# Specifies the path of the ssh key you want to use
-# when do ssh dump, the default value is
+# sshkey <path> - Will use the sshkey to do ssh dump.
+# Specifies the path of the ssh key to use
+# when dumping via ssh. The default value is
# /root/.ssh/kdump_id_rsa.
#
# <fs type> <partition> - Will mount -t <fs type> <partition> /mnt and copy
@@ -50,17 +50,15 @@
#
# core_collector <command> <options>
# - This allows you to specify the command to copy
-# the vmcore. You could use the dump filtering
-# program makedumpfile, the default one, to retrieve
-# your core, which on some arches can drastically
-# reduce core file size. See /sbin/makedumpfile --help
-# for a list of options. Note that the -i and -g
-# options are not needed here, as the initrd will
-# automatically be populated with a config file
-# appropriate for the running kernel.
-# Default core_collector for raw/ssh dump is:
+# the vmcore. The default is makedumpfile, which on
+# some architectures can drastically reduce core file size.
+# See /sbin/makedumpfile --help for a list of options.
+# Note that the -i and -g options are not needed here,
+# as the initrd will automatically be populated with a
+# config file appropriate for the running kernel.
+# The default core_collector for raw/ssh dump is:
# "makedumpfile -F -l --message-level 1 -d 31".
-# Default core_collector for other targets is:
+# The default core_collector for other targets is:
# "makedumpfile -l --message-level 1 -d 31".
# For core_collector format details please refer to
# kexec-kdump-howto.txt or kdump.conf manpage.
@@ -70,13 +68,13 @@
# executable just after the memory dump process
# terminates. The exit status from the dump process
# is fed to the kdump_post executable, which can be
-# used to trigger different actions for success or
-# failure.
+# used to trigger actions for success or failure
+# respectively.
#
# kdump_pre <binary | script>
-# - works just like the kdump_post directive, but instead
+# - Works just like the kdump_post directive, but instead
# of running after the dump process, runs immediately
-# before. Exit status of this binary is interpreted
+# before. Exit status of this binary is interpreted
# as follows:
# 0 - continue with dump process as usual
# non 0 - reboot the system
@@ -94,23 +92,18 @@
# initrd, typically used to set up access to
# non-boot-path dump targets that might otherwise
# not be accessible in the kdump environment. Multiple
-# modules can be listed, separated by a space, and any
+# modules can be listed, separated by spaces, and any
# dependent modules will automatically be included.
#
# default <reboot | halt | poweroff | shell | dump_to_rootfs>
-# - Action to preform in case dumping to intended target
-# fails. If no default action is specified, "reboot"
-# is assumed default.
-# reboot: If the default action is reboot simply reboot
-# the system and loose the core that you are
-# trying to retrieve.
-# halt: If the default action is halt, then simply
-# halt the system after attempting to capture
-# a vmcore, regardless of success or failure.
-# poweroff: The system will be powered down
-# shell: If the default action is shell, then drop to
-# an shell session inside the initramfs from
-# where you can try to record the core manually.
+# - Action to perform in case dumping to intended target
+# fails. The default is "reboot".
+# reboot: Reboot the system and lose the vmcore.
+# halt: Halt the system and lose the vmcore.
+# poweroff: Power down the system.
+# shell: Drop to a shell session inside the initramfs,
+# then you can try to save the core
+# manually.
# Exiting this shell reboots the system.
# Note: kdump uses bash as the default shell.
# dump_to_rootfs: If non-root dump target is specified,
@@ -119,15 +112,16 @@
# to rootfs from initramfs context and reboot.
#
# force_rebuild <0 | 1>
-# - By default, kdump initrd only will be rebuilt when
+# - By default, kdump initrd will only be rebuilt when
# necessary. Specify 1 to force rebuilding kdump
# initrd every time when kdump service starts.
#
-#override_resettable <0 | 1>
-# - Usually a unresettable block device can't be dump target.
-# Specifying 1 means though block target is unresettable, user
-# understand this situation and want to try dumping. By default,
-# it's set to 0, means not to try a destined failure.
+# override_resettable <0 | 1>
+# - Usually an unresettable block device can't be a dump
+# target. Specifying 1 means that even though the block
+# target is unresettable, the user wants to try dumping
+# anyway. By default, it's set to 0, which will not try
+# something destined to fail.
#
# dracut_args <arg(s)>
# - Pass extra dracut options when rebuilding kdump
@@ -135,11 +129,12 @@
#
# fence_kdump_args <arg(s)>
# - Command line arguments for fence_kdump_send (it can contain
-# all valid arguments except hosts to send notification to).
+# all valid arguments except hosts to send notification to).
#
# fence_kdump_nodes <node(s)>
-# - List of cluster node(s) separated by space to send fence_kdump
-# notification to (this option is mandatory to enable fence_kdump).
+# - List of cluster node(s), separated by spaces, to send
+# fence_kdump notifications to (this option is mandatory to
+# enable fence_kdump).
#
#raw /dev/vg/lv_kdump
6 years, 11 months
[PATCH 0/2] kdump.conf and kdump.conf.5 documentation fixes
by Freeman Zhang
Hi,
This is a resend with a few fixes according to Freeman's V4.
Difference between this version and the V4:
1. split to two patches for review
2. destined to failure -> destined to fail
3. specify append loop.max_loop -> specify loop.max_loop
Thanks
Dave
6 years, 11 months
[PATCH v4] documentation fix
by Freeman Zhang
There are lots of typos and incorrect expresstions in kdump.conf,
as well as its manpage kdump.conf.5. Fix them to make it less
confusing.
Signed-off-by: Freeman Zhang <zhezhang(a)redhat.com>
Reported-by: Donald Berry <dberry(a)redhat.com>
---
v3->v4:
Combined the comments from Dave<ruyang(a)redhat.com>. When Dave's comments
were against Don's, I took Don's.
kdump.conf | 89 ++++++++++++++++++++++++++++--------------------------------
kdump.conf.5 | 77 ++++++++++++++++++++++++----------------------------
2 files changed, 78 insertions(+), 88 deletions(-)
diff --git a/kdump.conf b/kdump.conf
index 54b581d..8fe7450 100644
--- a/kdump.conf
+++ b/kdump.conf
@@ -3,14 +3,14 @@
# This file contains a series of commands to perform (in order) when a
# kernel crash has happened and the kdump kernel has been loaded. Directives in
# this file are only applicable to the kdump initramfs, and have no effect if
-# the root filesystem is mounted and the normal init scripts are processed
+# the root filesystem is mounted and the normal init scripts are processed.
#
-# Currently only one dump target and path may be configured at once
-# if the configured dump target fails, the default action will be preformed
-# the default action may be configured with the default directive below. If the
-# configured dump target succedes
+# Currently, only one dump target and path can be specified. If the configured
+# dump target fails, the default action will be performed. The default action
+# can be configured with the default directive.
+#
+# Supported options:
#
-# Basics commands supported are:
# raw <partition> - Will dd /proc/vmcore into <partition>.
# Use persistent device names for partition devices,
# such as /dev/vg/<devname>.
@@ -21,11 +21,11 @@
# ssh <user@server> - Will scp /proc/vmcore to
# <user@server>:/var/crash/%HOST-%DATE/, supports DNS
# NOTE: make sure user has necessary write
-# permissions on server
+# permissions on the server.
#
-# sshkey <path> - Will use the sshkey to do ssh dump
-# Specifies the path of the ssh key you want to use
-# when do ssh dump, the default value is
+# sshkey <path> - Will use the sshkey to do ssh dump.
+# Specifies the path of the ssh key to use
+# when dumping via ssh. The default value is
# /root/.ssh/kdump_id_rsa.
#
# <fs type> <partition> - Will mount -t <fs type> <partition> /mnt and copy
@@ -50,17 +50,15 @@
#
# core_collector <command> <options>
# - This allows you to specify the command to copy
-# the vmcore. You could use the dump filtering
-# program makedumpfile, the default one, to retrieve
-# your core, which on some arches can drastically
-# reduce core file size. See /sbin/makedumpfile --help
-# for a list of options. Note that the -i and -g
-# options are not needed here, as the initrd will
-# automatically be populated with a config file
-# appropriate for the running kernel.
-# Default core_collector for raw/ssh dump is:
+# the vmcore. The default is makedumpfile, which on
+# some architectures can drastically reduce core file size.
+# See /sbin/makedumpfile --help for a list of options.
+# Note that the -i and -g options are not needed here,
+# as the initrd will automatically be populated with a
+# config file appropriate for the running kernel.
+# The default core_collector for raw/ssh dump is:
# "makedumpfile -F -l --message-level 1 -d 31".
-# Default core_collector for other targets is:
+# The default core_collector for other targets is:
# "makedumpfile -l --message-level 1 -d 31".
# For core_collector format details please refer to
# kexec-kdump-howto.txt or kdump.conf manpage.
@@ -70,13 +68,13 @@
# executable just after the memory dump process
# terminates. The exit status from the dump process
# is fed to the kdump_post executable, which can be
-# used to trigger different actions for success or
-# failure.
+# used to trigger actions for success or failure
+# respectively.
#
# kdump_pre <binary | script>
-# - works just like the kdump_post directive, but instead
+# - Works just like the kdump_post directive, but instead
# of running after the dump process, runs immediately
-# before. Exit status of this binary is interpreted
+# before. Exit status of this binary is interpreted
# as follows:
# 0 - continue with dump process as usual
# non 0 - reboot the system
@@ -94,23 +92,18 @@
# initrd, typically used to set up access to
# non-boot-path dump targets that might otherwise
# not be accessible in the kdump environment. Multiple
-# modules can be listed, separated by a space, and any
+# modules can be listed, separated by spaces, and any
# dependent modules will automatically be included.
#
# default <reboot | halt | poweroff | shell | dump_to_rootfs>
-# - Action to preform in case dumping to intended target
-# fails. If no default action is specified, "reboot"
-# is assumed default.
-# reboot: If the default action is reboot simply reboot
-# the system and loose the core that you are
-# trying to retrieve.
-# halt: If the default action is halt, then simply
-# halt the system after attempting to capture
-# a vmcore, regardless of success or failure.
-# poweroff: The system will be powered down
-# shell: If the default action is shell, then drop to
-# an shell session inside the initramfs from
-# where you can try to record the core manually.
+# - Action to perform in case dumping to intended target
+# fails. The default is "reboot".
+# reboot: Reboot the system and lose the vmcore.
+# halt: Halt the system and lose the vmcore.
+# poweroff: Power down the system.
+# shell: Drop to a shell session inside the initramfs,
+# then you can try to save the core
+# manually.
# Exiting this shell reboots the system.
# Note: kdump uses bash as the default shell.
# dump_to_rootfs: If non-root dump target is specified,
@@ -119,15 +112,16 @@
# to rootfs from initramfs context and reboot.
#
# force_rebuild <0 | 1>
-# - By default, kdump initrd only will be rebuilt when
+# - By default, kdump initrd will only be rebuilt when
# necessary. Specify 1 to force rebuilding kdump
# initrd every time when kdump service starts.
#
-#override_resettable <0 | 1>
-# - Usually a unresettable block device can't be dump target.
-# Specifying 1 means though block target is unresettable, user
-# understand this situation and want to try dumping. By default,
-# it's set to 0, means not to try a destined failure.
+# override_resettable <0 | 1>
+# - Usually an unresettable block device can't be a dump
+# target. Specifying 1 means that even though the block
+# target is unresettable, the user wants to try dumping
+# anyway. By default, it's set to 0, which will not try
+# something destined to failure.
#
# dracut_args <arg(s)>
# - Pass extra dracut options when rebuilding kdump
@@ -135,11 +129,12 @@
#
# fence_kdump_args <arg(s)>
# - Command line arguments for fence_kdump_send (it can contain
-# all valid arguments except hosts to send notification to).
+# all valid arguments except hosts to send notification to).
#
# fence_kdump_nodes <node(s)>
-# - List of cluster node(s) separated by space to send fence_kdump
-# notification to (this option is mandatory to enable fence_kdump).
+# - List of cluster node(s), separated by spaces, to send
+# fence_kdump notifications to (this option is mandatory to
+# enable fence_kdump).
#
#raw /dev/vg/lv_kdump
diff --git a/kdump.conf.5 b/kdump.conf.5
index f1c2a2c..bc8ea4f 100644
--- a/kdump.conf.5
+++ b/kdump.conf.5
@@ -10,14 +10,14 @@ collection service.
kdump.conf provides post-kexec instructions to the kdump kernel. It is
stored in the initrd file managed by the kdump service. If you change
-this file and do not want to restart before it takes effect, restart
-the kdump service to rebuild to initrd.
+this file and do not want to reboot in order for the changes to take
+effect, restart the kdump service to rebuild the initrd.
For most configurations, you can simply review the examples provided
in the stock /etc/kdump.conf.
.B NOTE:
-For filesystem dump the dump target must be mounted before building
+For filesystem dumps the dump target must be mounted before building
kdump initramfs.
kdump.conf only affects the behavior of the initramfs. Please read the
@@ -48,8 +48,8 @@ server and that a fqdn is used as the server name
.B sshkey <path>
.RS
-Specifies the path of the ssh key you want to use when do ssh dump,
-the default value is /root/.ssh/kdump_id_rsa.
+Specifies the path of the ssh key to use when dumping via ssh.
+The default value is /root/.ssh/kdump_id_rsa.
.RE
.B <fs type> <partition>
@@ -77,26 +77,25 @@ Ignored for raw device dumps. If unset, will default to /var/crash.
.B core_collector <command> <options>
.RS
This allows you to specify the command to copy the vmcore.
-You could use the dump filtering program makedumpfile, the default one,
-to retrieve your core, which on some arches can drastically reduce
-core file size. See /sbin/makedumpfile --help for a list of options.
+The default is makedumpfile, which on some architectures can drastically reduce
+core file size. See /sbin/makedumpfile --help for a list of options.
Note that the -i and -g options are not needed here, as the initrd
will automatically be populated with a config file appropriate
for the running kernel.
.PP
Note 1: About default core collector:
-Default core_collector for raw/ssh dump is:
+The default core_collector for raw/ssh dump is:
"makedumpfile -F -l --message-level 1 -d 31".
-Default core_collector for other targets is:
+The default core_collector for other targets is:
"makedumpfile -l --message-level 1 -d 31".
Even if core_collector option is commented out in kdump.conf, makedumpfile
-is default core collector and kdump uses it internally.
+is the default core collector and kdump uses it internally.
If one does not want makedumpfile as default core_collector, then they
need to specify one using core_collector option to change the behavior.
.PP
Note 2: If "makedumpfile -F" is used then you will get a flattened format
vmcore.flat, you will need to use "makedumpfile -R" to rearrange the
-dump data from stdard input to a normal dumpfile (readable with analysis
+dump data from standard input to a normal dumpfile (readable with analysis
tools).
ie. "makedumpfile -R vmcore < vmcore.flat"
@@ -108,11 +107,10 @@ This directive allows you to run a specified
executable just after the memory dump process
terminates. The exit status from the dump process
is fed to the kdump_post executable, which can be
-used to trigger different actions for success or
-failure.
+used to trigger actions for success or failure respectively.
.PP
Note that scripts written for use with this
-directive must use the /bin/bash interpreter
+directive must use the /bin/bash interpreter.
.RE
.B kdump_pre <binary | script>
@@ -127,7 +125,7 @@ as follows:
non 0 - reboot the system
.PP
Note that scripts written for this directive must use
-the /bin/bash interpreter
+the /bin/bash interpreter.
.RE
.B extra_bins <binaries | shell scripts>
@@ -146,36 +144,33 @@ modules that you want to be loaded in the kdump
initrd, typically used to set up access to
non-boot-path dump targets that might otherwise
not be accessible in the kdump environment. Multiple
-modules can be listed, separated by a space, and any
+modules can be listed, separated by spaces, and any
dependent modules will automatically be included.
.RE
.B default <reboot | halt | poweroff | shell | dump_to_rootfs>
.RS
-Action to preform in case dumping to intended target fails. If no default
-action is specified, "reboot" is assumed default.
-reboot: If the default action is reboot simply reboot the system (this is what
-most people will want, as it returns the system to a nominal state). shell: If the default
-action is shell, then drop to an shell session inside the initramfs from
-where you can manually preform additional recovery actions. Exiting this shell
-reboots the system. halt: bring the system to a halt, requiring manual reset
-poweroff: The system will be powered down. dump_to_rootfs:If the default action
-is dump_to_rootfs, specified root will be mounted and dump will be saved in "path"
-directory.
+Action to perform in case dumping to intended target fails. The default is "reboot".
+reboot: reboot the system (this is what most people will want, as it returns the
+system to a normal state). shell: drop to a shell session inside the initramfs,
+from where you can manually perform additional recovery actions. Exiting this shell
+reboots the system. halt: bring the system to a halt, requiring manual reset.
+poweroff: the system will be powered down. dump_to_rootfs: specified root will
+be mounted and dump will be saved in "path" directory.
Note: kdump uses bash as the default shell.
.RE
.B force_rebuild <0 | 1>
.RS
-By default, kdump initrd only will be rebuilt when necessary.
+By default, kdump initrd will only be rebuilt when necessary.
Specify 1 to force rebuilding kdump initrd every time when kdump service starts.
.RE
.B override_resettable <0 | 1>
.RS
-Usually a unresettable block device can't be dump target. Specifying 1 means
-though block target is unresettable, user understand this situation and want
-to try dumping. By default, it's set to 0, means not to try a destined failure.
+Usually an unresettable block device can't be a dump target. Specifying 1 means
+that even though block target is unresettable, user wants to try dumping anyway.
+By default, it's set to 0, which will not try something destined to failure.
.RE
@@ -195,7 +190,7 @@ arguments except hosts to send notification to).
.B fence_kdump_nodes <node(s)>
.RS
-List of cluster node(s) separated by space to send fence_kdump notification
+List of cluster node(s), separated by spaces, to send fence_kdump notification
to (this option is mandatory to enable fence_kdump).
.RE
@@ -210,26 +205,26 @@ directly.
.B options <module> <option list>
.RS
-Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump to add proper
-module option as kernel command line params. Such as append loop.max_loop=1
+Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump to add module options as
+kernel command line parameters. For example, specify append 'loop.max_loop=1'
to limit maximum loop devices to 1.
.RE
.B link_delay <seconds>
.RS
-link_delay was used to wait a network device to initialize before using it.
-Now dracut network module take care of this issue automaticlly.
+link_delay was used to wait for a network device to initialize before using it.
+Now dracut network module takes care of this issue automatically.
.RE
.B disk_timeout <seconds>
.RS
-Similar to link_delay, dracut ensures disks being ready before kdump uses them.
+Similar to link_delay, dracut ensures disks are ready before kdump uses them.
.RE
.B debug_mem_level <0-3>
.RS
-This was used to turns on debug/verbose output of kdump scripts regarding
-free/used memory at various points of execution. This feature has been
+Turn on verbose debug output of kdump scripts regarding free/used memory at
+various points of execution. This feature has been
moved to dracut now.
Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump and
append dracut cmdline param rd.memdebug=[0-3] to enable the debug output.
@@ -253,7 +248,7 @@ present in initramfs but it is not actually loaded in kernel. Hence
retaining blacklist option creates more confusing behavior. It has been
deprecated.
.PP
-Instead use rd.driver.blacklist option on second kernel to blacklist
+Instead, use rd.driver.blacklist option on second kernel to blacklist
a certain module. One can edit /etc/sysconfig/kdump.conf and edit
KDUMP_COMMANDLINE_APPEND to pass kernel command line options. Refer
to dracut.cmdline man page for more details on module blacklist option.
@@ -262,7 +257,7 @@ to dracut.cmdline man page for more details on module blacklist option.
.RE
.SH EXAMPLES
-Here is some examples for core_collector option:
+Here are some examples for core_collector option:
.PP
Core collector command format depends on dump target type. Typically for
filesystem (local/remote), core_collector should accept two arguments.
--
2.5.5
6 years, 11 months
[PATCH v3] documentation fix
by Freeman Zhang
There are lots of typos and incorrect expresstions in kdump.conf,
as well as its manpage kdump.conf.5. Fix them to make it less
confusing.
Signed-off-by: Freeman Zhang <zhezhang(a)redhat.com>
Reported-by: Donald Berry <dberry(a)redhat.com>
---
v2->v3:
break some lines in the patch in 80 limit.
"dracut network module take care of" -> "dracut network module takes care of"
kdump.conf | 89 ++++++++++++++++++++++++++++--------------------------------
kdump.conf.5 | 78 +++++++++++++++++++++++-----------------------------
2 files changed, 76 insertions(+), 91 deletions(-)
diff --git a/kdump.conf b/kdump.conf
index 54b581d..14b734a 100644
--- a/kdump.conf
+++ b/kdump.conf
@@ -3,14 +3,14 @@
# This file contains a series of commands to perform (in order) when a
# kernel crash has happened and the kdump kernel has been loaded. Directives in
# this file are only applicable to the kdump initramfs, and have no effect if
-# the root filesystem is mounted and the normal init scripts are processed
+# the root filesystem is mounted and the normal init scripts are processed.
#
-# Currently only one dump target and path may be configured at once
-# if the configured dump target fails, the default action will be preformed
-# the default action may be configured with the default directive below. If the
-# configured dump target succedes
+# Currently, only one dump target and path can be specified. If the configured
+# dump target fails, the default action will be performed. The default action
+# may be configured with the default directive.
+#
+# Supported options:
#
-# Basics commands supported are:
# raw <partition> - Will dd /proc/vmcore into <partition>.
# Use persistent device names for partition devices,
# such as /dev/vg/<devname>.
@@ -21,11 +21,11 @@
# ssh <user@server> - Will scp /proc/vmcore to
# <user@server>:/var/crash/%HOST-%DATE/, supports DNS
# NOTE: make sure user has necessary write
-# permissions on server
+# permissions on the server.
#
-# sshkey <path> - Will use the sshkey to do ssh dump
-# Specifies the path of the ssh key you want to use
-# when do ssh dump, the default value is
+# sshkey <path> - Will use the sshkey to do ssh dump.
+# Specifies the path of the ssh key to use
+# when dumping via ssh. The default value is
# /root/.ssh/kdump_id_rsa.
#
# <fs type> <partition> - Will mount -t <fs type> <partition> /mnt and copy
@@ -50,17 +50,15 @@
#
# core_collector <command> <options>
# - This allows you to specify the command to copy
-# the vmcore. You could use the dump filtering
-# program makedumpfile, the default one, to retrieve
-# your core, which on some arches can drastically
-# reduce core file size. See /sbin/makedumpfile --help
-# for a list of options. Note that the -i and -g
-# options are not needed here, as the initrd will
-# automatically be populated with a config file
-# appropriate for the running kernel.
-# Default core_collector for raw/ssh dump is:
+# the vmcore. The default is makedumpfile, which on
+# some architectures can drastically reduce core file size.
+# See /sbin/makedumpfile --help for a list of options.
+# Note that the -i and -g options are not needed here,
+# as the initrd will automatically be populated with a
+# config file appropriate for the running kernel.
+# The default core_collector for raw/ssh dump is:
# "makedumpfile -F -l --message-level 1 -d 31".
-# Default core_collector for other targets is:
+# The default core_collector for other targets is:
# "makedumpfile -l --message-level 1 -d 31".
# For core_collector format details please refer to
# kexec-kdump-howto.txt or kdump.conf manpage.
@@ -70,13 +68,13 @@
# executable just after the memory dump process
# terminates. The exit status from the dump process
# is fed to the kdump_post executable, which can be
-# used to trigger different actions for success or
-# failure.
+# used to trigger actions for success or failure
+# respectively.
#
# kdump_pre <binary | script>
-# - works just like the kdump_post directive, but instead
+# - Works just like the kdump_post directive, but instead
# of running after the dump process, runs immediately
-# before. Exit status of this binary is interpreted
+# before. Exit status of this binary is interpreted
# as follows:
# 0 - continue with dump process as usual
# non 0 - reboot the system
@@ -94,23 +92,18 @@
# initrd, typically used to set up access to
# non-boot-path dump targets that might otherwise
# not be accessible in the kdump environment. Multiple
-# modules can be listed, separated by a space, and any
+# modules can be listed, separated by spaces, and any
# dependent modules will automatically be included.
#
# default <reboot | halt | poweroff | shell | dump_to_rootfs>
-# - Action to preform in case dumping to intended target
-# fails. If no default action is specified, "reboot"
-# is assumed default.
-# reboot: If the default action is reboot simply reboot
-# the system and loose the core that you are
-# trying to retrieve.
-# halt: If the default action is halt, then simply
-# halt the system after attempting to capture
-# a vmcore, regardless of success or failure.
-# poweroff: The system will be powered down
-# shell: If the default action is shell, then drop to
-# an shell session inside the initramfs from
-# where you can try to record the core manually.
+# - Action to perform in case dumping to intended target
+# fails. The default is "reboot".
+# reboot: Reboot the system. The vmcore will be lost.
+# halt: Halt the system. The vmcore will be lost.
+# poweroff: The system will be powered down.
+# shell: Drop to a shell session inside the initramfs,
+# from which you can try to save the core
+# manually.
# Exiting this shell reboots the system.
# Note: kdump uses bash as the default shell.
# dump_to_rootfs: If non-root dump target is specified,
@@ -119,15 +112,16 @@
# to rootfs from initramfs context and reboot.
#
# force_rebuild <0 | 1>
-# - By default, kdump initrd only will be rebuilt when
+# - By default, kdump initrd will only be rebuilt when
# necessary. Specify 1 to force rebuilding kdump
# initrd every time when kdump service starts.
#
-#override_resettable <0 | 1>
-# - Usually a unresettable block device can't be dump target.
-# Specifying 1 means though block target is unresettable, user
-# understand this situation and want to try dumping. By default,
-# it's set to 0, means not to try a destined failure.
+# override_resettable <0 | 1>
+# - Usually an unresettable block device can't be a dump
+# target. Specifying 1 means that even though the block
+# target is unresettable, the user wants to try dumping
+# anyway. By default, it's set to 0, which will not try
+# something destined to failure.
#
# dracut_args <arg(s)>
# - Pass extra dracut options when rebuilding kdump
@@ -135,11 +129,12 @@
#
# fence_kdump_args <arg(s)>
# - Command line arguments for fence_kdump_send (it can contain
-# all valid arguments except hosts to send notification to).
+# all valid arguments except hosts to send notification to).
#
# fence_kdump_nodes <node(s)>
-# - List of cluster node(s) separated by space to send fence_kdump
-# notification to (this option is mandatory to enable fence_kdump).
+# - List of cluster node(s), separated by spaces, to send
+# fence_kdump notifications to (this option is mandatory to
+# enable fence_kdump).
#
#raw /dev/vg/lv_kdump
diff --git a/kdump.conf.5 b/kdump.conf.5
index f1c2a2c..499e71c 100644
--- a/kdump.conf.5
+++ b/kdump.conf.5
@@ -10,14 +10,14 @@ collection service.
kdump.conf provides post-kexec instructions to the kdump kernel. It is
stored in the initrd file managed by the kdump service. If you change
-this file and do not want to restart before it takes effect, restart
-the kdump service to rebuild to initrd.
+this file and do not want to have to reboot in order for the changes
+to take effect, restart the kdump service to rebuild the initrd.
For most configurations, you can simply review the examples provided
in the stock /etc/kdump.conf.
.B NOTE:
-For filesystem dump the dump target must be mounted before building
+For filesystem dumps the dump target must be mounted before building
kdump initramfs.
kdump.conf only affects the behavior of the initramfs. Please read the
@@ -48,8 +48,8 @@ server and that a fqdn is used as the server name
.B sshkey <path>
.RS
-Specifies the path of the ssh key you want to use when do ssh dump,
-the default value is /root/.ssh/kdump_id_rsa.
+Specifies the path of the ssh key to use when dumping via ssh.
+The default value is /root/.ssh/kdump_id_rsa.
.RE
.B <fs type> <partition>
@@ -77,26 +77,20 @@ Ignored for raw device dumps. If unset, will default to /var/crash.
.B core_collector <command> <options>
.RS
This allows you to specify the command to copy the vmcore.
-You could use the dump filtering program makedumpfile, the default one,
-to retrieve your core, which on some arches can drastically reduce
-core file size. See /sbin/makedumpfile --help for a list of options.
+The default is makedumpfile, which on some architectures can drastically reduce
+core file size. See /sbin/makedumpfile --help for a list of options.
Note that the -i and -g options are not needed here, as the initrd
will automatically be populated with a config file appropriate
for the running kernel.
.PP
-Note 1: About default core collector:
-Default core_collector for raw/ssh dump is:
+Note 1: The default core_collector for raw/ssh dump is:
"makedumpfile -F -l --message-level 1 -d 31".
-Default core_collector for other targets is:
+The default core_collector for other targets is:
"makedumpfile -l --message-level 1 -d 31".
-Even if core_collector option is commented out in kdump.conf, makedumpfile
-is default core collector and kdump uses it internally.
-If one does not want makedumpfile as default core_collector, then they
-need to specify one using core_collector option to change the behavior.
.PP
Note 2: If "makedumpfile -F" is used then you will get a flattened format
vmcore.flat, you will need to use "makedumpfile -R" to rearrange the
-dump data from stdard input to a normal dumpfile (readable with analysis
+dump data from standard input to a normal dumpfile (readable with analysis
tools).
ie. "makedumpfile -R vmcore < vmcore.flat"
@@ -108,11 +102,10 @@ This directive allows you to run a specified
executable just after the memory dump process
terminates. The exit status from the dump process
is fed to the kdump_post executable, which can be
-used to trigger different actions for success or
-failure.
+used to trigger actions for success or failure respectively.
.PP
Note that scripts written for use with this
-directive must use the /bin/bash interpreter
+directive must use the /bin/bash interpreter.
.RE
.B kdump_pre <binary | script>
@@ -127,7 +120,7 @@ as follows:
non 0 - reboot the system
.PP
Note that scripts written for this directive must use
-the /bin/bash interpreter
+the /bin/bash interpreter.
.RE
.B extra_bins <binaries | shell scripts>
@@ -146,36 +139,33 @@ modules that you want to be loaded in the kdump
initrd, typically used to set up access to
non-boot-path dump targets that might otherwise
not be accessible in the kdump environment. Multiple
-modules can be listed, separated by a space, and any
+modules can be listed, separated by spaces, and any
dependent modules will automatically be included.
.RE
.B default <reboot | halt | poweroff | shell | dump_to_rootfs>
.RS
-Action to preform in case dumping to intended target fails. If no default
-action is specified, "reboot" is assumed default.
-reboot: If the default action is reboot simply reboot the system (this is what
-most people will want, as it returns the system to a nominal state). shell: If the default
-action is shell, then drop to an shell session inside the initramfs from
-where you can manually preform additional recovery actions. Exiting this shell
-reboots the system. halt: bring the system to a halt, requiring manual reset
-poweroff: The system will be powered down. dump_to_rootfs:If the default action
-is dump_to_rootfs, specified root will be mounted and dump will be saved in "path"
-directory.
+Action to perform in case dumping to intended target fails. The default is "reboot".
+reboot: reboot the system (this is what most people will want, as it returns the
+system to a normal state). shell: drop to a shell session inside the initramfs,
+from where you can manually perform additional recovery actions. Exiting this shell
+reboots the system. halt: bring the system to a halt, requiring manual reset.
+poweroff: the system will be powered down. dump_to_rootfs: specified root will
+be mounted and dump will be saved in "path" directory.
Note: kdump uses bash as the default shell.
.RE
.B force_rebuild <0 | 1>
.RS
-By default, kdump initrd only will be rebuilt when necessary.
+By default, kdump initrd will only be rebuilt when necessary.
Specify 1 to force rebuilding kdump initrd every time when kdump service starts.
.RE
.B override_resettable <0 | 1>
.RS
-Usually a unresettable block device can't be dump target. Specifying 1 means
-though block target is unresettable, user understand this situation and want
-to try dumping. By default, it's set to 0, means not to try a destined failure.
+Usually an unresettable block device can't be dump target. Specifying 1 means
+that even though block target is unresettable, user wants to try dumping anyway.
+By default, it's set to 0, which will not try something destined to failure.
.RE
@@ -195,7 +185,7 @@ arguments except hosts to send notification to).
.B fence_kdump_nodes <node(s)>
.RS
-List of cluster node(s) separated by space to send fence_kdump notification
+List of cluster node(s), separated by spaces, to send fence_kdump notification
to (this option is mandatory to enable fence_kdump).
.RE
@@ -210,25 +200,25 @@ directly.
.B options <module> <option list>
.RS
-Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump to add proper
-module option as kernel command line params. Such as append loop.max_loop=1
+Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump to add module options as
+kernel command line parameters. For example, specify 'append loop.max_loop=1'
to limit maximum loop devices to 1.
.RE
.B link_delay <seconds>
.RS
-link_delay was used to wait a network device to initialize before using it.
-Now dracut network module take care of this issue automaticlly.
+link_delay was used to wait for a network device to initialize before using it.
+Now dracut network module takes care of this issue automatically.
.RE
.B disk_timeout <seconds>
.RS
-Similar to link_delay, dracut ensures disks being ready before kdump uses them.
+Similar to link_delay, dracut ensures disks are ready before kdump uses them.
.RE
.B debug_mem_level <0-3>
.RS
-This was used to turns on debug/verbose output of kdump scripts regarding
+This was used to turn on debug/verbose output of kdump scripts regarding
free/used memory at various points of execution. This feature has been
moved to dracut now.
Use KDUMP_COMMANDLINE_APPEND in /etc/sysconfig/kdump and
@@ -253,7 +243,7 @@ present in initramfs but it is not actually loaded in kernel. Hence
retaining blacklist option creates more confusing behavior. It has been
deprecated.
.PP
-Instead use rd.driver.blacklist option on second kernel to blacklist
+Instead, use rd.driver.blacklist option on second kernel to blacklist
a certain module. One can edit /etc/sysconfig/kdump.conf and edit
KDUMP_COMMANDLINE_APPEND to pass kernel command line options. Refer
to dracut.cmdline man page for more details on module blacklist option.
@@ -262,7 +252,7 @@ to dracut.cmdline man page for more details on module blacklist option.
.RE
.SH EXAMPLES
-Here is some examples for core_collector option:
+Here are some examples for core_collector option:
.PP
Core collector command format depends on dump target type. Typically for
filesystem (local/remote), core_collector should accept two arguments.
--
2.5.5
6 years, 11 months