From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Keith Owens <kaos@ocs.com.au>
Cc: Sam Ravnborg <sam@ravnborg.org>,
Linux Kernel list <linux-kernel@vger.kernel.org>
Subject: Re: [patch 0/3] kallsyms: Add gate page and all symbols support
Date: Wed, 19 Jan 2005 09:03:04 +1100 [thread overview]
Message-ID: <1106085784.4500.95.camel@gaston> (raw)
In-Reply-To: <15379.1106053651@ocs3.ocs.com.au>
> > - I dislike the fact that you now define the prototype of the function
> >in the __HAVE_ARCH_GATE_AREA case. I want my arch .h to be the one doing
> >so, since i want to inline it
>
> Maybe. I dislike copying definitions to multiple asm headers. If you
> think that the win of inlining the ppc64 version of these functions
> outweighs the header duplication then send a patch. Don't forget to
> duplicate the definition in include/asm-x86_64 as well.
Well, I just want to inline it as en empty function ...
>(to nothing in the ppc64 case since the
> >vDSO I'm implementing doesn't need any special treatement of the gate
> >area, it's a normal VMA added to the mm's at exec time).
>
> Added to specific task's mm or to all tasks? If the gate VMA varies
> according to the task then you have to support the kallsyms "is this a
> possible gate address for any task?" question, like x86-64.
each task mm can have it differently. why would I need kallsym support ?
the ppc64 vDSO support will not be part of the kernel address ranges,
but will be somewhere in the normal userland areas.
Ben.
prev parent reply other threads:[~2005-01-18 22:03 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-11-28 3:38 [patch 0/3] kallsyms: Add gate page and all symbols support Keith Owens
2004-11-28 3:54 ` [patch 1/3] kallsyms: Clean up x86-64 special casing of in_gate_area() Keith Owens
2004-11-28 3:55 ` [patch 2/3] kallsyms: Add in_gate_area_no_task() Keith Owens
2004-11-28 3:55 ` [patch 3/3] kallsyms: gate page is part of the kernel, honour CONFIG_KALLSYMS_ALL Keith Owens
2004-12-28 21:17 ` [patch 0/3] kallsyms: Add gate page and all symbols support Sam Ravnborg
2005-01-18 7:52 ` Benjamin Herrenschmidt
2005-01-18 13:07 ` Keith Owens
2005-01-18 22:03 ` Benjamin Herrenschmidt [this message]
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=1106085784.4500.95.camel@gaston \
--to=benh@kernel.crashing.org \
--cc=kaos@ocs.com.au \
--cc=linux-kernel@vger.kernel.org \
--cc=sam@ravnborg.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.