diff options
Diffstat (limited to 'results/classifier/118/all/1572329')
| -rw-r--r-- | results/classifier/118/all/1572329 | 121 |
1 files changed, 121 insertions, 0 deletions
diff --git a/results/classifier/118/all/1572329 b/results/classifier/118/all/1572329 new file mode 100644 index 000000000..ce99abda9 --- /dev/null +++ b/results/classifier/118/all/1572329 @@ -0,0 +1,121 @@ +debug: 0.961 +socket: 0.961 +arm: 0.961 +semantic: 0.956 +risc-v: 0.953 +architecture: 0.951 +assembly: 0.947 +kernel: 0.946 +network: 0.945 +boot: 0.945 +register: 0.944 +permissions: 0.943 +PID: 0.942 +graphic: 0.939 +performance: 0.938 +files: 0.934 +virtual: 0.934 +peripherals: 0.930 +user-level: 0.927 +mistranslation: 0.925 +ppc: 0.922 +device: 0.921 +VMM: 0.918 +hypervisor: 0.918 +vnc: 0.893 +KVM: 0.883 +x86: 0.870 +TCG: 0.849 +i386: 0.841 + +ARM bootloader does not set r0 to 0 + +# arm-softmmu/qemu-system-arm -M raspi2 -m 1024 -smp 4 -kernel kernel.bin -serial stdio -dtb rpi2.dtb + +My code shows r0 = 0x31 while it should be 0. + + + +On 19 April 2016 at 23:34, Sylvain <email address hidden> wrote: +> # arm-softmmu/qemu-system-arm -M raspi2 -m 1024 -smp 4 -kernel +> kernel.bin -serial stdio -dtb rpi2.dtb +> +> My code shows r0 = 0x31 while it should be 0. + +Hi. Thanks for this bug report and the patch. In order +for us to be able to use your patch, we'll need you to +provide a signed-off-by line, which tells us you have +the legal right to submit it and are happy for us to +include it in QEMU under QEMU's licensing terms. +(You can look at +http://wiki.qemu.org/Contribute/SubmitAPatch#Patch_emails_must_include_a_Signed-off-by:_line +if you want more details.) + +You can provide a signoff just by replying to this email +with a line which reads +"Signed-off-by: Your Name <your.email@here>". + +A couple of notes: this bug only affects boards which +have a write_board_setup function, which means only +highbank/midway, raspi2, and xilinx_zynq; that's probably +why we didn't spot it earlier. + +thanks +-- PMM + + +Signed-off-by: Sylvain Garrigues <email address hidden> + +Fix link register patch follows: + +diff --git a/hw/arm/boot.c b/hw/arm/boot.c +index 5975fbf..5876945 100644 +--- a/hw/arm/boot.c ++++ b/hw/arm/boot.c +@@ -68,7 +68,7 @@ static const ARMInsnFixup bootloader_aarch64[] = { + */ + + static const ARMInsnFixup bootloader[] = { +- { 0xe28fe008 }, /* add lr, pc, #8 */ ++ { 0xe28fe004 }, /* add lr, pc, #4 */ + { 0xe51ff004 }, /* ldr pc, [pc, #-4] */ + { 0, FIXUP_BOARD_SETUP }, + #define BOOTLOADER_NO_BOARD_SETUP_OFFSET 3 + + +> Le 20 avr. 2016 à 16:21, Peter Maydell <email address hidden> a écrit : +> +> On 19 April 2016 at 23:34, Sylvain <email address hidden> wrote: +>> # arm-softmmu/qemu-system-arm -M raspi2 -m 1024 -smp 4 -kernel +>> kernel.bin -serial stdio -dtb rpi2.dtb +>> +>> My code shows r0 = 0x31 while it should be 0. +> +> Hi. Thanks for this bug report and the patch. In order +> for us to be able to use your patch, we'll need you to +> provide a signed-off-by line, which tells us you have +> the legal right to submit it and are happy for us to +> include it in QEMU under QEMU's licensing terms. +> (You can look at +> http://wiki.qemu.org/Contribute/SubmitAPatch#Patch_emails_must_include_a_Signed-off-by:_line +> if you want more details.) +> +> You can provide a signoff just by replying to this email +> with a line which reads +> "Signed-off-by: Your Name <your.email@here>". +> +> A couple of notes: this bug only affects boards which +> have a write_board_setup function, which means only +> highbank/midway, raspi2, and xilinx_zynq; that's probably +> why we didn't spot it earlier. +> +> thanks +> -- PMM + + + +Fix committed: b4850e5ae9607f9f31932 + + +Should be part of QEMU 2.6 ==> Fix released + |