| To: | Thomas Bogendoerfer <tsbogend@alpha.franken.de> |
|---|---|
| Subject: | Re: [PATCH] MIPS: tracing: Optimize the implementation |
| From: | Ralf Baechle <ralf@linux-mips.org> |
| Date: | Fri, 12 Mar 2010 11:14:13 +0100 |
| Cc: | Wu Zhangjin <wuzhangjin@gmail.com>, Steven Rostedt <srostedt@redhat.com>, linux-mips@linux-mips.org, David Daney <ddaney@caviumnetworks.com> |
| In-reply-to: | <20100312085053.GB6364@alpha.franken.de> |
| Original-recipient: | rfc822;linux-mips@linux-mips.org |
| References: | <8b93c417fefa4d446f801abfd718ba94fdcb1821.1268330348.git.wuzhangjin@gmail.com> <20100312085053.GB6364@alpha.franken.de> |
| Sender: | linux-mips-bounce@linux-mips.org |
| User-agent: | Mutt/1.5.20 (2009-08-17) |
On Fri, Mar 12, 2010 at 09:50:53AM +0100, Thomas Bogendoerfer wrote:
> On Fri, Mar 12, 2010 at 02:07:37AM +0800, Wu Zhangjin wrote:
> > +/*
> > + * If the Instruction Pointer is in module space (0xc0000000), return ture;
> > + * otherwise, it is in kernel space (0x80000000), return false.
> > + */
> > +#define in_module(ip) (unlikely((ip) & 0x40000000))
> > +
>
> looks broken for 64bit, but maybe this is a 32bit only feature...
This gem did already exist in the old code, so no regression:
if (ip & 0x40000000) {
Ralf
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] MIPS: tracing: Optimize the implementation, Thomas Bogendoerfer |
|---|---|
| Next by Date: | Re: [PATCH] MIPS: tracing: Optimize the implementation, Wu Zhangjin |
| Previous by Thread: | Re: [PATCH] MIPS: tracing: Optimize the implementation, Thomas Bogendoerfer |
| Next by Thread: | Re: [PATCH] MIPS: tracing: Optimize the implementation, Wu Zhangjin |
| Indexes: | [Date] [Thread] [Top] [All Lists] |