On Friday 11 August 2006 22:07, Alan Cox wrote: > > + printk(KERN_WARNING "%s: watchdog expired - resetting system\n", > > + wdt_gpi_name); > > + > > + *(volatile char *) flagaddr |= 0x01; > > + *(volatile char *) resetaddr = powercycle ? 0x01 : 0x2; > > + iob(); > > + while (1) continue; > > cpu_relax(); I tried to find out about the purpose of cpu_relax(). On MIPS, at least, it maps to barrier(). I do not quite understand why I would need a barrier() in this place. Would you, or someone else, care to enlighten me? I am sending a revised patch in a separate mail. Thomas -- Thomas Koeller thomas@xxxxxxxxxxxxxxxxxx