Messages in this thread | | | Date | Thu, 05 Mar 2015 08:14:42 -0700 | From | Shuah Khan <> | Subject | Re: [PATCH] Remove redhat'ism from ftrace selftests. |
| |
On 03/04/2015 06:18 PM, Michael Ellerman wrote: > On Wed, 2015-03-04 at 21:44 +0900, Namhyung Kim wrote: >> >> I think that there's no need to even call true or echo.. >> >> From 0549544e8e982df6478f11e2b4fe419f94c22434 Mon Sep 17 00:00:00 2001 >> From: Namhyung Kim <namhyung@kernel.org> >> Date: Wed, 4 Mar 2015 21:26:38 +0900 >> Subject: [PATCH] ftracetest: Do not use usleep >> >> The usleep is only provided on distros from Redhat so running ftracetest >> on other distro resulted in failures due to the missing usleep. >> >> The reason of using [u]sleep in the test was to generate (scheduler) >> events. But as we use 'cat trace | grep | wc -l' to read the events, >> the command themselves already generate some events before reading the >> trace file so no need to call [u]sleep explicitly. >> >> Reported-by: Michael Ellerman <mpe@ellerman.id.au> >> Reported-by: Dave Jones <davej@codemonkey.org.uk> >> Signed-off-by: Namhyung Kim <namhyung@kernel.org> > > Thanks, this is working for me. > > Tested-by: Michael Ellerman <mpe@ellerman.id.au> >
Steve,
Do you want to make a call on which one you want me to take it through between these two patches - one from Dave and the other from Namhyung Kim.
Please send me the patch I have to pull in. Namhyung Kim's patch isn't in the format I can apply and I don't have Dave's patch in my inbox.
thanks, -- Shuah
-- Shuah Khan Sr. Linux Kernel Developer Open Source Innovation Group Samsung Research America (Silicon Valley) shuahkh@osg.samsung.com | (970) 217-8978
| |