summary refs log tree commit diff stats
path: root/results/classifier/118/kernel/1624726
diff options
context:
space:
mode:
Diffstat (limited to 'results/classifier/118/kernel/1624726')
-rw-r--r--results/classifier/118/kernel/162472670
1 files changed, 70 insertions, 0 deletions
diff --git a/results/classifier/118/kernel/1624726 b/results/classifier/118/kernel/1624726
new file mode 100644
index 000000000..76636a558
--- /dev/null
+++ b/results/classifier/118/kernel/1624726
@@ -0,0 +1,70 @@
+kernel: 0.938
+architecture: 0.916
+boot: 0.775
+ppc: 0.765
+graphic: 0.763
+arm: 0.760
+mistranslation: 0.757
+performance: 0.743
+hypervisor: 0.712
+user-level: 0.663
+socket: 0.660
+peripherals: 0.639
+x86: 0.638
+network: 0.638
+debug: 0.624
+semantic: 0.606
+files: 0.602
+device: 0.588
+KVM: 0.485
+vnc: 0.451
+virtual: 0.447
+i386: 0.427
+PID: 0.419
+TCG: 0.393
+register: 0.388
+VMM: 0.377
+permissions: 0.375
+risc-v: 0.356
+assembly: 0.295
+
+Integrator/CP regression after QOM'ification of integratorcp.c
+
+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.zhao <zxq_yx_007@163.com>
+Date:   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 <zxq_yx_007@163.com>
+    Reviewed-by: Peter Maydell <email address hidden>
+    Signed-off-by: Peter Maydell <email address hidden>
+
+:040000 040000 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.
+
+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
+
+
+Patch has been included here:
+http://git.qemu.org/?p=qemu.git;a=commitdiff;h=e9d9ee234f852026d58
+... and been released with QEMU version 2.8
+