[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v17 15/24] hw/rx: Honor -accel qtest
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v17 15/24] hw/rx: Honor -accel qtest |
Date: |
Fri, 7 Jun 2019 15:40:57 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 |
On 6/7/19 11:11 AM, Yoshinori Sato wrote:
> From: Richard Henderson <address@hidden>
>
> Issue an error if no kernel, no bios, and not qtest'ing.
> Fixes make check-qtest-rx: test/qom-test.
>
> Signed-off-by: Richard Henderson <address@hidden>
> Signed-off-by: Yoshinori Sato <address@hidden>
Again:
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/rx/rx62n.c | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/hw/rx/rx62n.c b/hw/rx/rx62n.c
> index 3a8fe7b0bf..e55257c622 100644
> --- a/hw/rx/rx62n.c
> +++ b/hw/rx/rx62n.c
> @@ -21,11 +21,13 @@
>
> #include "qemu/osdep.h"
> #include "qapi/error.h"
> +#include "qemu/error-report.h"
> #include "hw/hw.h"
> #include "hw/rx/rx62n.h"
> #include "hw/loader.h"
> #include "hw/sysbus.h"
> #include "sysemu/sysemu.h"
> +#include "sysemu/qtest.h"
> #include "cpu.h"
>
> /*
> @@ -190,8 +192,14 @@ static void rx62n_realize(DeviceState *dev, Error **errp)
> memory_region_init_rom(&s->c_flash, NULL, "codeflash",
> RX62N_CFLASH_SIZE, errp);
> memory_region_add_subregion(s->sysmem, RX62N_CFLASH_BASE, &s->c_flash);
> +
> if (!s->kernel) {
> - rom_add_file_fixed(bios_name, RX62N_CFLASH_BASE, 0);
> + if (bios_name) {
> + rom_add_file_fixed(bios_name, RX62N_CFLASH_BASE, 0);
> + } else if (!qtest_enabled()) {
> + error_report("No bios or kernel specified");
> + exit(1);
> + }
> }
>
> object_initialize_child(OBJECT(s), "cpu", &s->cpu,
>
- Re: [Qemu-devel] [PATCH v17 20/24] target/rx: Use prt_ldmi for XCHG_mr disassembly, (continued)
- [Qemu-devel] [PATCH v17 17/24] MAINTAINERS: Add RX, Yoshinori Sato, 2019/06/07
- [Qemu-devel] [PATCH v17 03/24] target/rx: CPU definition, Yoshinori Sato, 2019/06/07
- [Qemu-devel] [PATCH v17 06/24] hw/timer: RX62N internal timer modules, Yoshinori Sato, 2019/06/07
- [Qemu-devel] [PATCH v17 02/24] target/rx: TCG helper, Yoshinori Sato, 2019/06/07
- [Qemu-devel] [PATCH v17 15/24] hw/rx: Honor -accel qtest, Yoshinori Sato, 2019/06/07
- Re: [Qemu-devel] [PATCH v17 15/24] hw/rx: Honor -accel qtest,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH v17 01/24] target/rx: TCG translation, Yoshinori Sato, 2019/06/07
- Re: [Qemu-devel] [PATCH v17 00/24] Add RX archtecture support, no-reply, 2019/06/07
- Re: [Qemu-devel] [PATCH v17 00/24] Add RX archtecture support, no-reply, 2019/06/07
- Re: [Qemu-devel] [PATCH v17 00/24] Add RX archtecture support, Philippe Mathieu-Daudé, 2019/06/07
- Re: [Qemu-devel] [PATCH v17 00/24] Add RX archtecture support, no-reply, 2019/06/07