Messages in this thread | | | Date | Tue, 29 Apr 2008 22:29:01 +0300 | From | "Pekka Enberg" <> | Subject | Re: [2/2] vmallocinfo: Add caller information |
| |
On Tue, Apr 29, 2008 at 10:09 PM, Christoph Lameter <clameter@sgi.com> wrote: > > Well so we display out of whack backtraces? There are also issues on > > platforms that do not have a stack in the classic sense (rotating register > > file on IA64 and Sparc64 f.e.). Determining a backtrace can be very > > expensive.
On Tue, Apr 29, 2008 at 10:23 PM, Pekka Enberg <penberg@cs.helsinki.fi> wrote: > I think that's the key question here whether we need to enable this on > production systems? If yes, why? If it's just a debugging aid, then I > see Ingo's point of save_stack_trace(); otherwise the low-overhead > __builtin_return_address() makes more sense.
Actually, this is vmalloc() so why do we even care? If there are callers in the tree that use vmalloc() for performance sensitive stuff, they ought to be converted to kmalloc() anyway, no?
| |