Thread (9 messages) 9 messages, 4 authors, 2018-12-04

Re: [PATCH 3/3] arm64: ftrace: add cond_resched() to func ftrace_make_(call|nop)

From: Steven Rostedt <rostedt@goodmis.org>
Date: 2018-12-04 18:23:50
Also in: lkml

On Tue, 4 Dec 2018 19:07:16 +0100
Anders Roxell [off-list ref] wrote:

quoted
quoted
quoted
+   schedulable = !irqs_disabled() & !preempt_count();  
Looks suspiciously like a bitwise preemptible() to me!  
Ah, thanks. Yeah, that should have been &&. But what did you expect.
I didn't even compile this ;-)
 
quoted
If it does, then I'll add it. Or take a patch for it ;-)  
I tested your patch. it worked.

I'll send a patch shortly.
Thanks. Please add a comment above the schedulable test stating that
some archs call this with interrupts or preemption disabled, but
other archs don't and this can cause a tremendous unneeded latency.

-- Steve
 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help