Re: [PATCHv2 2/2] tracing,mm - add kernel pagefault tracepoint for x86 & x86_64

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Christoph Hellwig
Date: Thursday, November 11, 2010 - 5:51 am

Can you explain again who this is supposed to be used?  It's missing
a lot of essential information about the page fault, but on the other
hand traces totally useless information like the task struct pointer.

What's your motivation for posting this given that you obviously
haven't actually tried to use it in practice?

Btw, this doesn't mean I'm against a page faul trace point - quite
contrary.  It just seems you're not actually pushing something that's
been tested in real life or is generally useful.

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH 1/2] tracing,mm - add kernel pagefault tracepoi ..., Christoph Hellwig, (Wed Nov 10, 6:29 am)
Re: [PATCH 1/2] tracing,mm - add kernel pagefault tracepoi ..., Frederic Weisbecker, (Wed Nov 10, 8:00 am)
Re: [PATCH 1/2] tracing,mm - add kernel pagefault tracepoi ..., Christoph Hellwig, (Wed Nov 10, 8:20 am)
Re: [PATCH 1/2] tracing,mm - add kernel pagefault tracepoi ..., Frederic Weisbecker, (Wed Nov 10, 9:44 am)
Re: [PATCHv2 2/2] tracing,mm - add kernel pagefault tracep ..., Christoph Hellwig, (Thu Nov 11, 5:51 am)
Re: [PATCHv2 2/2] tracing,mm - add kernel pagefault tracep ..., Frederic Weisbecker, (Mon Nov 15, 6:43 am)
Re: [PATCHv2 2/2] tracing,mm - add kernel pagefault tracep ..., Frederic Weisbecker, (Mon Nov 15, 7:54 am)