linux-kernel - Re: [PATCH -mm] introduce tracehook_finish

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20090729155007.f4f5a793.akpm@linux-foundation.org>
Date:	Wed, 29 Jul 2009 15:50:07 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Oleg Nesterov <oleg@...hat.com>
Cc:	roland@...hat.com, utrace-devel@...hat.com,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH -mm] introduce tracehook_finish_jctl() helper

On Wed, 29 Jul 2009 04:01:40 +0200
Oleg Nesterov <oleg@...hat.com> wrote:

> (textually depends on signals-tracehook_notify_jctl-change.patch)
> 
> Introduce the empty inline tracehook_finish_jctl() helper called by
> do_signal_stop() after wakeup.
> 
> Currently we lack the ability to report this state change.
> 
> Also fix the comment, it should be placed before schedule().
> 
> Signed-off-by: Oleg Nesterov <oleg@...hat.com>
> ---
> 
>  include/linux/tracehook.h |    9 +++++++++
>  kernel/signal.c           |    2 ++
>  2 files changed, 11 insertions(+)
> 
> --- __UTRACE/include/linux/tracehook.h~2_FINISH_JCTL	2009-07-29 03:10:10.000000000 +0200
> +++ __UTRACE/include/linux/tracehook.h	2009-07-29 03:51:42.000000000 +0200
> @@ -521,6 +521,15 @@ static inline int tracehook_notify_jctl(
>  	return notify ?: (current->ptrace & PT_PTRACED) ? why : 0;
>  }
>  
> +/**
> + * tracehook_finish_jctl - report about return from job control stop
> + *
> + * This is called by do_signal_stop() after wakeup.
> + */
> +static inline void tracehook_finish_jctl(void)
> +{
> +}
> +
>  #define DEATH_REAP			-1
>  #define DEATH_DELAYED_GROUP_LEADER	-2
>  
> --- __UTRACE/kernel/signal.c~2_FINISH_JCTL	2009-07-29 03:17:49.000000000 +0200
> +++ __UTRACE/kernel/signal.c	2009-07-29 03:57:07.000000000 +0200
> @@ -1735,12 +1735,12 @@ static int do_signal_stop(int signr)
>  		read_unlock(&tasklist_lock);
>  	}
>  
> +	/* Now we don't run again until woken by SIGCONT or SIGKILL */
>  	do {
>  		schedule();
>  	} while (try_to_freeze());
> -	/*
> -	 * Now we don't run again until continued.
> -	 */
> +
> +	tracehook_finish_jctl();
>  	current->exit_code = 0;
>  

Confused.  This patch has no effect.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/