On Thu, Jul 30, 2020 at 06:14:15PM +0300, Eli Billauer wrote: > On 30/07/20 17:46, Alan Stern wrote: > > This loop ought to have a cpu_relax() somewhere, probably here. Same > > for the other two functions. Otherwise this looks okay to me. > This function isn't so well documented. Does it go with an include file? > asm/processor.h, for example? Or just add the call? asm/processor.h is the appropriate include file, but you can just add the call without worrying about it. Among other places, asm/processor.h is included by linux/mutex.h, for example. Alan Stern