Messages in this thread | | | From | "Zhaolei" <> | Subject | Re: [PATCH v3 1/1] ftrace, workqueuetrace: Make workqueuetracepoints use TRACE_EVENT macro | Date | Mon, 20 Apr 2009 09:43:47 +0800 |
| |
* From: "KOSAKI Motohiro" <kosaki.motohiro@jp.fujitsu.com> >> Thanks for your suggest. >> I read distinction between workqueue and worklet. >> In my schedule, this patch is first step of our target, my image of steps is: >> 1: Move current workqueuetracepoints into TRACEEVENT (this patch) >> 2: Make workqueuetrace support per-worklet output (doing) >> 3: Add time information to workqueuetrace's worklet stat (need above new TRACEPOINT) >> >> So, i prepared to add new worklet tracepoints in step3. >> What's your opinion? >
Hello, Kosaki-san
> Zhao-san, Maintainer's easy reviewability is very important. I agree it.
> Can you switch 2 and 3 ? Sorry for my poor English. I means 2 is to add worklet stat, 3 is to add more information to worklet stat, 3 is based on 2, switch 2 and 3 means merge 2 and 3.
Thanks Zhaolei > > it cause Ingo can review the patch fill his request or not directly. > > > > | |