From: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: kosaki.motohiro@jp.fujitsu.com, Bryan Donlan <bdonlan@gmail.com>,
Ulrich Drepper <drepper@redhat.com>, Timo Sirainen <tss@iki.fi>,
WANG Cong <xiyou.wangcong@gmail.com>,
Oleg Nesterov <oleg@redhat.com>,
LKML <linux-kernel@vger.kernel.org>,
Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH v6] Added PR_SET_PROCTITLE_AREA option for prctl()
Date: Tue, 8 Dec 2009 15:53:21 +0900 (JST) [thread overview]
Message-ID: <20091208155202.B5B7.A69D9226@jp.fujitsu.com> (raw)
In-Reply-To: <20091208144004.B5AB.A69D9226@jp.fujitsu.com>
> >
> > * KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> wrote:
> >
> > > > The feature looks useful, but the choice of a prctl as an API is strange
> > > > - it limits us to the current task only - while the ability to set
> > > > arguments for another task looks a more generic (and potentially more
> > > > useful) solution.
> > >
> > > No. It's impossible.
> > > /proc/{pid}/cmdline read user process's memory. iow, this prctl() don't
> > > receive string, it receive virtual address itself. [...]
> >
> > it's not 'impossible' at all, you yourself mention ptrace:
>
> Ah yes, 'impossible' was wrong word. but it doesn't works intentionally.
>
> 1. setproctitle() unaware application continue to see argv[0] directly.
> it makes some inconsistent behavior.
> 2. proc title (i.e. string) injection need to map new page as process title area.
> implicit mapping increasing makes new trouble
> - mihgt cause to exceed max_map_count awhile after.
> - might cause leak proc title area (who know when it should be freed?)
>
> I think reasonable way is 1. send signal (or use another inter process
> communication way) to target process 2. target process change own proc title
> themself.
>
> Plus, I haven't seen the use-case of changin another task. iow I doubt
> it's worth to change lots code.
if your mention is strongly, can you please explain your expected use case?
next prev parent reply other threads:[~2009-12-08 6:53 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-08 3:15 KOSAKI Motohiro
2009-12-08 4:46 ` Ingo Molnar
2009-12-08 5:10 ` KOSAKI Motohiro
2009-12-08 5:38 ` Ingo Molnar
2009-12-08 5:56 ` KOSAKI Motohiro
2009-12-08 6:53 ` KOSAKI Motohiro [this message]
2009-12-08 6:58 ` Ingo Molnar
2009-12-08 6:28 ` Bryan Donlan
2009-12-08 6:57 ` Ingo Molnar
2009-12-08 7:19 ` KOSAKI Motohiro
2009-12-09 9:10 ` Ingo Molnar
2009-12-10 0:16 ` KOSAKI Motohiro
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=20091208155202.B5B7.A69D9226@jp.fujitsu.com \
--to=kosaki.motohiro@jp.fujitsu.com \
--cc=akpm@linux-foundation.org \
--cc=bdonlan@gmail.com \
--cc=drepper@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=oleg@redhat.com \
--cc=tss@iki.fi \
--cc=xiyou.wangcong@gmail.com \
/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
all inboxes | Powered by JetHome®