public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Chris Zankel <chris@zankel.net>
To: Oleg Nesterov <oleg@redhat.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Roland McGrath <roland@redhat.com>,
	David Howells <dhowells@redhat.com>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Greg Ungerer <gerg@uclinux.org>,
	Haavard Skinnemoen <hskinnemoen@atmel.com>,
	Helge Deller <deller@gmx.de>,
	Hirokazu Takata <takata@linux-m32r.org>,
	Jeff Dike <jdike@addtoit.com>, Kyle McMartin <kyle@mcmartin.ca>,
	Martin Schwidefsky <schwidefsky@de.ibm.com>,
	Mike Frysinger <vapier.adi@gmail.com>,
	Paul Mundt <lethal@linux-sh.org>,
	Roman Zippel <zippel@linux-m68k.org>,
	Yoshinori Sato <ysato@users.sourceforge.jp>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/4] ptrace: remove PT_DTRACE from avr32, mn10300, parisc, s390, sh, xtensa
Date: Fri, 24 Apr 2009 00:05:28 -0700	[thread overview]
Message-ID: <49F164B8.9000504@zankel.net> (raw)
In-Reply-To: <20090423193206.GA14885@redhat.com>

Acked-by: Chris Zankel <chris@zankel.net>

Cheers!
-Chris

Oleg Nesterov wrote:
> avr32, mn10300, parisc, s390, sh, xtensa:
> 
> they never set PT_DTRACE, but clear it after do_execve().
> 
> Signed-off-by: Oleg Nesterov <oleg@redhat.com>
> 
>  arch/xtensa/kernel/process.c      |    5 -----
> --- PTRACE/arch/xtensa/kernel/process.c~DT_2_CLEAR	2009-04-06 00:03:37.000000000 +0200
> +++ PTRACE/arch/xtensa/kernel/process.c	2009-04-22 21:48:13.000000000 +0200
> @@ -331,11 +331,6 @@ long xtensa_execve(char __user *name, ch
>  	if (IS_ERR(filename))
>  		goto out;
>  	error = do_execve(filename, argv, envp, regs);
> -	if (error == 0) {
> -		task_lock(current);
> -		current->ptrace &= ~PT_DTRACE;
> -		task_unlock(current);
> -	}
>  	putname(filename);
>  out:
>  	return error;


  parent reply	other threads:[~2009-04-24  7:05 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-23 19:32 [PATCH 2/4] ptrace: remove PT_DTRACE from avr32, mn10300, parisc, s390, sh, xtensa Oleg Nesterov
2009-04-24  0:18 ` Paul Mundt
2009-04-24  2:11 ` Kyle McMartin
2009-04-24  7:05 ` Chris Zankel [this message]
2009-04-24  7:34 ` Martin Schwidefsky
2009-04-24  8:13 ` Haavard Skinnemoen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=49F164B8.9000504@zankel.net \
    --to=chris@zankel.net \
    --cc=akpm@linux-foundation.org \
    --cc=deller@gmx.de \
    --cc=dhowells@redhat.com \
    --cc=geert@linux-m68k.org \
    --cc=gerg@uclinux.org \
    --cc=hskinnemoen@atmel.com \
    --cc=jdike@addtoit.com \
    --cc=kyle@mcmartin.ca \
    --cc=lethal@linux-sh.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=oleg@redhat.com \
    --cc=roland@redhat.com \
    --cc=schwidefsky@de.ibm.com \
    --cc=takata@linux-m32r.org \
    --cc=vapier.adi@gmail.com \
    --cc=ysato@users.sourceforge.jp \
    --cc=zippel@linux-m68k.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox