[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefault
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults" |
Date: |
Fri, 23 Mar 2018 15:13:30 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 23/03/2018 15:04, Peter Maydell wrote:
>> However, if the machine is emulating a real-world board with a fixed SoC
>> and fixed hardware in the SoC, it makes more sense to create a null backend.
>
> That's a lot of duplicate code to say "oh, this is NULL, create the
> null backend" in lots of different boards :-(
Note it's a null "backend", not necessarily a null "character device".
Your proposal, namely ensuring that be->chr == NULL is handled properly
in qemu_chr_fe_init, would be just fine.
The main point was that the choice of whether to create the device is up
to the board, and there isn't a single answer valid for all boards.
Paolo
- [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults", Thomas Huth, 2018/03/23
- Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults", Bastian Koppelmann, 2018/03/23
- Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults", Peter Maydell, 2018/03/23
- Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults", Thomas Huth, 2018/03/23
- Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults", Peter Maydell, 2018/03/23
- Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults", Paolo Bonzini, 2018/03/23
- Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults", Peter Maydell, 2018/03/23
- Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults",
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults", Peter Maydell, 2018/03/23
- Re: [Qemu-devel] [PATCH for-2.12] hw/riscv: Fix crashes with "-nodefaults", Paolo Bonzini, 2018/03/23