Messages in this thread | | | Date | Wed, 18 Nov 2009 01:05:20 +0000 | From | Alan Cox <> | Subject | Re: yield() in i2c non-happy paths hits BUG under -rt patch |
| |
> I think the yield()s in the device driver code means "I need a small > delay before the hardware is ready" which might translate to some > arbitrary "let me msleep()" or "do not select this task in the next > scheduler run, EVEN IF this task is highest priority".
Yield() in a driver is almost always a bug. The reason for that is that doing
do { inb(); } while(!something);
which is what yield can end up as being if there is nothing else on that CPU is extremely bad for bus performance on most systems. It's almost always better to be using msleep() or even mdelay + a check to see if a reschedule is needed/schedule().
> I assume this is rather dirty and has too much overhead on the timer interfaces.
Our timers are very efficient and some day we will need to make jiffies a function and stop the timer ticking for best performance. At that point timers are probably the most efficient way to do much of this.
Be that as it may, yield() in a driver is almost always the wrong thing to do.
| |