[Crash-utility] [PATCH] scope: add ability to un-set scope

[HAGIO KAZUHITO(萩尾 一仁)]

-----Original Message-----
> Currently there is no way to un-set the scope without having to
> exit and re-enter crash.  The ability to un-set can come in very
> useful when running automated pykdump scripts and needing scope to
> be cleared between script runs.  Add the ability by allowing
> vaddr 0 to be passed through gdb_set_crash_scope() and
> gdb_command_funnel(), taking advantage of the !req->addr check in
> gdb_set_crash_block(), enabling 'set scope 0' as a viable command.
> 
> Signed-off-by: John Pittman <jpittman at redhat.com>

Looks good and tested OK.

Applied:
https://github.com/crash-utility/crash/commit/e4c1617e17ebf2d4e20ba59041de1536ec193be3

Thanks,
Kazu

> ---
>  gdb_interface.c | 30 ++++++++++++++++--------------
>  help.c          |  2 +-
>  2 files changed, 17 insertions(+), 15 deletions(-)
> 
> diff --git a/gdb_interface.c b/gdb_interface.c
> index 562d2ac..1172edd 100644
> --- a/gdb_interface.c
> +++ b/gdb_interface.c
> @@ -1012,23 +1012,25 @@ gdb_set_crash_scope(ulong vaddr, char *arg)
>  	char name[BUFSIZE];
>  	struct load_module *lm;
> 
> -	if (!is_kernel_text(vaddr)) {
> -		error(INFO, "invalid text address: %s\n", arg);
> -		return FALSE;
> -	}
> +	if (vaddr) {
> +		if (!is_kernel_text(vaddr)) {
> +			error(INFO, "invalid text address: %s\n", arg);
> +			return FALSE;
> +		}
> 
> -	if (module_symbol(vaddr, NULL, &lm, name, 0)) {
> -		if (!(lm->mod_flags & MOD_LOAD_SYMS)) {
> -			error(INFO, "attempting to find/load \"%s\" module debuginfo\n",
> -				lm->mod_name);
> -			if (!load_module_symbols_helper(lm->mod_name)) {
> -				error(INFO, "cannot find/load \"%s\" module debuginfo\n",
> +		if (module_symbol(vaddr, NULL, &lm, name, 0)) {
> +			if (!(lm->mod_flags & MOD_LOAD_SYMS)) {
> +				error(INFO, "attempting to find/load \"%s\" module debuginfo\n",
>  					lm->mod_name);
> -				return FALSE;
> +				if (!load_module_symbols_helper(lm->mod_name)) {
> +					error(INFO, "cannot find/load \"%s\" module debuginfo\n",
> +						lm->mod_name);
> +					return FALSE;
> +				}
>  			}
> -		}
> -	} else if (kt->flags2 & KASLR)
> -		vaddr -= (kt->relocate * -1);
> +		} else if (kt->flags2 & KASLR)
> +			vaddr -= (kt->relocate * -1);
> +	}
> 
>  	req->command = GNU_SET_CRASH_BLOCK;
>  	req->addr = vaddr;
> diff --git a/help.c b/help.c
> index d3427a3..7c9455f 100644
> --- a/help.c
> +++ b/help.c
> @@ -1088,7 +1088,7 @@ char *help_set[] = {
>  "                               of data structures; the \"text-addr\" argument",
>  "                               must be a kernel or module text address, which",
>  "                               may be expressed symbolically or as a hexadecimal",
> -"                               value.",
> +"                               value; set scope 0 to un-set.",
>  "         offline  show | hide  show or hide command output that is associated",
>  "                               with offline cpus.",
>  "         redzone  on | off     if on, CONFIG_SLUB object addresses displayed by",
> --
> 2.17.2