[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [PATCH v1] hw/s390x: Allow to configure the consoles wi
From: |
David Hildenbrand |
Subject: |
Re: [qemu-s390x] [PATCH v1] hw/s390x: Allow to configure the consoles with the "-serial" parameter |
Date: |
Wed, 25 Apr 2018 14:47:27 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
>>> int ret;
>>> @@ -346,6 +355,14 @@ static void ccw_init(MachineState *machine)
>>> /* Create VirtIO network adapters */
>>> s390_create_virtio_net(BUS(css_bus), "virtio-net-ccw");
>>>
>>> + /* init consoles */
>>> + if (serial_hds[0]) {
>>> + s390_create_sclpconsole("sclpconsole", serial_hds[0]);
>>> + }
>>> + if (serial_hds[1]) {
>>> + s390_create_sclpconsole("sclplmconsole", serial_hds[1]);
>>> + }
>>
>> What happens if more -serial are defined? An error? Silently ignored?
>
> Silently ignored, since this is also what almost all other machines are
> doing (look for serial_hds in hw/ and you'll see what I mean).
>
>> (e.g. do we have to redefine MAX_SERIAL_PORTS on s390x or add checking
>> code here?)
>
> As all the other machines are also not redefining MAX_SERIAL_PORTS, I
> think we should also not do this on s390x now, should we?
Then I guess if we would introduce it, we should do it for all other
implementations. So I think we can just leave it as it is.
>
> Thomas
>
--
Thanks,
David / dhildenb
Re: [qemu-s390x] [PATCH v1] hw/s390x: Allow to configure the consoles with the "-serial" parameter, Christian Borntraeger, 2018/04/25
Re: [qemu-s390x] [PATCH v1] hw/s390x: Allow to configure the consoles with the "-serial" parameter, Thomas Huth, 2018/04/25