Re: [PATCH v2] sched,tracing: Convert to sched_set_fifo()

From: Steven Rostedt
Date: Mon Jul 27 2020 - 16:56:20 EST


On Sat, 25 Jul 2020 18:58:16 +0200
Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:

> On Fri, Jul 24, 2020 at 06:18:46PM -0400, Steven Rostedt wrote:
> > On Fri, 24 Jul 2020 23:50:03 +0200
> > peterz@xxxxxxxxxxxxx wrote:
> >
> > > - if (producer_fifo < 0)
> > > + if (producer_fifo)
> > > + trace_printk("Running Producer at SCHED_FIFO %s\n",
> > > + consumer_fifo == 1 ? "low" : "high");
> >
> > I'm going to take cut-and-paste away from you!
>
> Well, yes, I said so, I also already have it fixed.

I thought you're comment on IRC was about v1, as I read it before
seeing v2.

>
> Aside from that, is the patch ok?

Beside this, yes.

Reviewed-by: Steven Rostedt (VMware) <rostedt@xxxxxxxxxxx>

-- Steve