[Crash-utility] [PATCH] s390x: Add support for CONFIG_THREAD_INFO_IN_TASK

Dave Anderson anderson at redhat.com
Thu Nov 10 20:07:33 UTC 2016



----- Original Message -----
> For s390x we still use OFFSET(task_struct_thread_info) to find out
> the stack base. Since crash commit c9f932440bd06f0 which added support
> for the new CONFIG_THREAD_INFO_IN_TASK configuration (Linux 4.9-rc1)
> this is no longer possible.
> 
> So fix this and use the generic task_to_stackbase() to find the stack
> base instead.
> 
> Without the patch the stack backtrace for running tasks (bt -a) can be
> incomplete.
> 
> Signed-off-by: Michael Holzheu <holzheu at linux.vnet.ibm.com>
> ---
>  s390x.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/s390x.c b/s390x.c
> index 8e9cafa..e13bd59 100644
> --- a/s390x.c
> +++ b/s390x.c
> @@ -1304,8 +1304,7 @@ static void s390x_back_trace_cmd(struct bt_info *bt)
>  	 * Print task stack
>  	 */
>  	if (THIS_KERNEL_VERSION >= LINUX(2, 6, 0)) {
> -		readmem(bt->task + OFFSET(task_struct_thread_info), KVADDR,
> -			&low, sizeof(long), "thread info", FAULT_ON_ERROR);
> +		low = task_to_stackbase(bt->task);
>  	} else {
>  		low = bt->task;
>  	}
> --
> 2.8.4

Thanks Michael, queued for crash-7.1.7:

  https://github.com/crash-utility/crash/commit/13dc25c923adec235c5ca27149bc008b6fd6f298

Dave


  
 
 




More information about the Crash-utility mailing list