[Qemu-devel] [Bug 1624726] Re: Integrator/CP regression after QOM'ification of integratorcp.c
Patch has been included here: http://git.qemu.org/?p=qemu.git;a=commitdiff;h=e9d9ee234f852026d58 ... and been released with QEMU version 2.8 ** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1624726 Title: Integrator/CP regression after QOM'ification of integratorcp.c Status in HelenOS branches: New Status in QEMU: Fix Released Bug description: The following command line no longer works (i.e. the guest does not boot) with QEMU 2.7.0: qemu-system-arm -M integratorcp -m 128M -kernel HelenOS-0.6.0-arm32-integratorcp.boot The HelenOS image can be downloaded here: http://www.helenos.org/releases/HelenOS-0.6.0-arm32-integratorcp.boot I did git bisect and came to this revision: a1f42e0c9abc1028a8bb8686dbb3749fcd2d18e8 is the first bad commit commit a1f42e0c9abc1028a8bb8686dbb3749fcd2d18e8 Author: xiaoqiang.zhaoDate: Mon Mar 7 15:05:44 2016 +0800 hw/arm: QOM'ify integratorcp.c * Drop the use of old SysBus init function and use instance_init * Remove the empty 'icp_pic_class_init' from Typeinfo Signed-off-by: xiaoqiang zhao Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell :04 04 b73418ea3fb69ed72438776e78786456fe4c414c b483e8579037fdae7d136b2f4ada3147bdde92f1 M hw Upon closer inspection, I discovered that for some reason s->memsz in integratorcm_init() is zero. In the last good revision, this value was 128. As a temporary workaround, hardcoding it to this expected value fixes the problem. To manage notifications about this bug go to: https://bugs.launchpad.net/helenos/+bug/1624726/+subscriptions
[Qemu-devel] [Bug 1624726] Re: Integrator/CP regression after QOM'ification of integratorcp.c
Turns out integratorcm_init() depends on the memsz property being already set, but that unfortunately is not the case as setting of memsz depends on integratorcm_init() having completed: dev = qdev_create(NULL, TYPE_INTEGRATOR_CM);<= calls integratorcm_init(), needs memsz qdev_prop_set_uint32(dev, "memsz", ram_size >> 20); <= memsz set here, needs dev -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1624726 Title: Integrator/CP regression after QOM'ification of integratorcp.c Status in HelenOS branches: New Status in QEMU: New Bug description: The following command line no longer works (i.e. the guest does not boot) with QEMU 2.7.0: qemu-system-arm -M integratorcp -m 128M -kernel HelenOS-0.6.0-arm32-integratorcp.boot The HelenOS image can be downloaded here: http://www.helenos.org/releases/HelenOS-0.6.0-arm32-integratorcp.boot I did git bisect and came to this revision: a1f42e0c9abc1028a8bb8686dbb3749fcd2d18e8 is the first bad commit commit a1f42e0c9abc1028a8bb8686dbb3749fcd2d18e8 Author: xiaoqiang.zhaoDate: Mon Mar 7 15:05:44 2016 +0800 hw/arm: QOM'ify integratorcp.c * Drop the use of old SysBus init function and use instance_init * Remove the empty 'icp_pic_class_init' from Typeinfo Signed-off-by: xiaoqiang zhao Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell :04 04 b73418ea3fb69ed72438776e78786456fe4c414c b483e8579037fdae7d136b2f4ada3147bdde92f1 M hw Upon closer inspection, I discovered that for some reason s->memsz in integratorcm_init() is zero. In the last good revision, this value was 128. As a temporary workaround, hardcoding it to this expected value fixes the problem. To manage notifications about this bug go to: https://bugs.launchpad.net/helenos/+bug/1624726/+subscriptions
[Qemu-devel] [Bug 1624726] Re: Integrator/CP regression after QOM'ification of integratorcp.c
** Bug watch added: Email to helenos-devel@lists # mailto:helenos-de...@lists.modry.cz ** Also affects: helenos via mailto:helenos-de...@lists.modry.cz Importance: Undecided Status: New -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1624726 Title: Integrator/CP regression after QOM'ification of integratorcp.c Status in HelenOS branches: New Status in QEMU: New Bug description: The following command line no longer works (i.e. the guest does not boot) with QEMU 2.7.0: qemu-system-arm -M integratorcp -m 128M -kernel HelenOS-0.6.0-arm32-integratorcp.boot The HelenOS image can be downloaded here: http://www.helenos.org/releases/HelenOS-0.6.0-arm32-integratorcp.boot I did git bisect and came to this revision: a1f42e0c9abc1028a8bb8686dbb3749fcd2d18e8 is the first bad commit commit a1f42e0c9abc1028a8bb8686dbb3749fcd2d18e8 Author: xiaoqiang.zhaoDate: Mon Mar 7 15:05:44 2016 +0800 hw/arm: QOM'ify integratorcp.c * Drop the use of old SysBus init function and use instance_init * Remove the empty 'icp_pic_class_init' from Typeinfo Signed-off-by: xiaoqiang zhao Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell :04 04 b73418ea3fb69ed72438776e78786456fe4c414c b483e8579037fdae7d136b2f4ada3147bdde92f1 M hw Upon closer inspection, I discovered that for some reason s->memsz in integratorcm_init() is zero. In the last good revision, this value was 128. As a temporary workaround, hardcoding it to this expected value fixes the problem. To manage notifications about this bug go to: https://bugs.launchpad.net/helenos/+bug/1624726/+subscriptions