On Fri, May 03, 2013 at 11:07:24PM +0100, Mark Cave-Ayland wrote:
Fortunately the use of the "volatile"
quantifier in the patch makes
sure that this doesn't happen. Both myself and Artyom have tested
the patch and it works for both of us, so I'm reasonably confident.
Oh yes of course. I missed that detail. Oops.
The reason that the udelay() implementation is a
trivial one is
because we need something simple in order to get a working OpenBIOS
(and hopefully 1.1 release) into QEMU git master in time for feature
freeze on Monday. If anyone else would like to investigate using
timers for udelay() on SPARC32/64, I'd be very happy to accept
patches from next week onwards.
Makes sense now.