[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] hw/isa/superio: Fix inconsistent use of Cha
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] hw/isa/superio: Fix inconsistent use of Chardev->be |
Date: |
Fri, 20 Apr 2018 11:29:12 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
>>> On 04/20/2018 05:43 AM, Peter Maydell wrote:
>>> Since we can now safely call serial_mm_init(..., chr=NULL, ...), I'll
>>> work on simplifying and fixing this.
>>
>> Correct. There are a bunch of dubious workarounds in the
>> codebase which try to handle the problem at different levels,
>> but we've decided that the right thing is for the NULL pointer
>> to be dealt with in one place at the bottom (in the qemu_chr_fe_*
>> functions).
>
> OK, I'll happily clean that :)
Hmmm removing the "if (serial_hds[0])" check I reach:
void serial_realize_core(SerialState *s, Error **errp)
{
if (!qemu_chr_fe_backend_connected(&s->chr)) {
error_setg(errp, "Can't create serial device, empty char device");
return;
}
- [Qemu-devel] [PATCH 0/2] hw/superio: Fix inconsistent use of Chardev->be, Philippe Mathieu-Daudé, 2018/04/19
- [Qemu-devel] [PATCH 1/2] hw/isa/superio: Fix inconsistent use of Chardev->be, Philippe Mathieu-Daudé, 2018/04/19
- Re: [Qemu-devel] [PATCH 1/2] hw/isa/superio: Fix inconsistent use of Chardev->be, Peter Maydell, 2018/04/20
- Re: [Qemu-devel] [PATCH 1/2] hw/isa/superio: Fix inconsistent use of Chardev->be, Philippe Mathieu-Daudé, 2018/04/20
- Re: [Qemu-devel] [PATCH 1/2] hw/isa/superio: Fix inconsistent use of Chardev->be, Peter Maydell, 2018/04/20
- Re: [Qemu-devel] [PATCH 1/2] hw/isa/superio: Fix inconsistent use of Chardev->be, Philippe Mathieu-Daudé, 2018/04/20
- Re: [Qemu-devel] [PATCH 1/2] hw/isa/superio: Fix inconsistent use of Chardev->be, Peter Maydell, 2018/04/20
- Re: [Qemu-devel] [PATCH 1/2] hw/isa/superio: Fix inconsistent use of Chardev->be,
Philippe Mathieu-Daudé <=
- Re: [Qemu-devel] [PATCH 1/2] hw/isa/superio: Fix inconsistent use of Chardev->be, Peter Maydell, 2018/04/20
[Qemu-devel] [PATCH 2/2] Revert "Revert "mux: fix ctrl-a b again"" again, Philippe Mathieu-Daudé, 2018/04/19