On 28.04.2012, at 17:52, Stefan Weil wrote: > The default case in function spin_read should never be reached, > therefore the old code used assert(0) to abort QEMU. > > This does not work when QEMU is compiled with macro NDEBUG defined. > In this case (and also when the compiler does not know that assert > never returns), there is a compiler warning because of the missing > return value. > > Using hw_error allows an improved error message and aborts always. > > Signed-off-by: Stefan Weil <s...@weilnetz.de>
Thanks, applied to ppc-next. The patch didn't show up on patchworks btw, not sure what went wrong there. Alex