Messages in this thread | | | Date | Tue, 23 Dec 2014 13:21:18 +0900 | From | Namhyung Kim <> | Subject | Re: [PATCH] ftracetest: Replace usleep with sleep 0.000001 |
| |
On Sat, Dec 20, 2014 at 05:07:25PM -0500, Steven Rostedt wrote: > On Sat, 20 Dec 2014 21:42:13 +0100 > Pavel Machek <pavel@ucw.cz> wrote: > > > On Wed 2014-12-10 15:38:27, Michael Ellerman wrote: > > > usleep is a Fedoraism, it's not generally available on Debian based > > > systems AFAICS. > > > > > > GNU sleep accepts a floating point argument, so use that instead. > > > > > > Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> > > > > Could ftrace user interface be enhanced not to need random delays? > > > > Not sure what you are talking about. These "random delays" are not for > the interface, but instead to force some events to happen and to make > sure they did. > > As the subject states, this is for "ftracetest" which is the selftests > for ftrace. > > The usleep is basically a "do something to trigger events", and we > don't want to wait long in doing it.
Right. AFAIK what ftracetest expects is a scheduler event so it doesn't need to be the [u]sleep.
Thanks, Namhyung
| |