On 24.01.2017 19:56, Valentin Plotkin wrote: > > Hi everyone, > > I looked at the "qemu-system-ppc -nographic -cpu G2leGP3 -M ppce500" on > the BiteSizedTasks page. The segfault was caused by machine > initialization code which expected a certain MMU model, checked, so > unused SPR were read, returning zeros. bamboo and virtex machines are > affected as well, but it doesn't always cause segfault, usually running > into unmapped memory and failing somewhat more nicely. > > I added the checks. It would be possible to add support for other MMU > models, but I'm not sure if there is any point (would any guest OS > support mutually exclusive CPU and machine)?
Hi, great to have a fix for this crash! I don't think it make much sense to add support for other MMU models here, so the simple checks should be good enough. However, your new code obviously does not follow the QEMU coding style. Could you please run your patch through scripts/checkpatch.pl and fix all issues that it reports? And when you resubmit, please make sure to copy the maintainers on CC: as well (scripts/get_maintainers.pl is your friend here). Thanks, Thomas
