On 26 March 2013 14:35, Andreas Färber <afaer...@suse.de> wrote: > Am 26.03.2013 13:20, schrieb Peter Maydell: >> Commit b350ab75 causes segfaults on accesses to PVR/PRR/CVR because >> it tries to call SUPERH_CPU_GET_CLASS() on a pointer that isn't a >> QOM object. Fix this by getting the actual QOM CPU object first. >> >> Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> >> --- >> Tested with the r2d image/instructions from >> https://oss.renesas.com/modules/document/?Getting%20Started%20with%20SH4%20and%20QEMU > > Oops, reproduces with the test image from the QEMU Wiki as well. Seems > to be a result of cherry-picking this commit before the full SH7750 > QOM'ification. > > Is this blocking any work of yours? Otherwise I would try rebasing my > SH7750 patches so that s->cpu becomes a SuperHCPU as expected here. > (Not sure if simply reverting my offending patch still works?)
Well, I have a workaround so it doesn't affect me now :-) I wasn't really doing anything much with the sh4 board, I was just looking at whether we could get rid of its use of taddr properties. (Oddly it uses qdev_prop_set_taddr() to set a property which isn't defined as a TADDR property, but this works anyway...) -- PMM
