[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH] hw/input/pckbd: The i8042 device should not b
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-trivial] [PATCH] hw/input/pckbd: The i8042 device should not be user_creatable |
Date: |
Thu, 4 Apr 2019 12:07:50 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.0 |
On 04/04/19 09:14, Thomas Huth wrote:
> The i8042 PS/2 controller is part of the chipset on the motherboard.
> It is instantiated by the machine init code, and it does not make sense
> to allow the user to plug an additional i8042 in any of the free ISA slots.
> Thus let's mark the device with user_creatable = false.
>
> Signed-off-by: Thomas Huth <address@hidden>
> ---
> hw/input/pckbd.c | 2 ++
> 1 file changed, 2 insertions(+)
user_creatable is not for devices that are not pluggable in real life;
it is for devices that crash QEMU (!) or always fail if plugged by the user.
So the question to ask is: would it make sense, and especially work, to
add an i8042 to machines that do have an ISA bridge (for example the Alpha?)
Paolo
> diff --git a/hw/input/pckbd.c b/hw/input/pckbd.c
> index 47a606f5e3..af393818fc 100644
> --- a/hw/input/pckbd.c
> +++ b/hw/input/pckbd.c
> @@ -568,6 +568,8 @@ static void i8042_class_initfn(ObjectClass *klass, void
> *data)
> dc->realize = i8042_realizefn;
> dc->vmsd = &vmstate_kbd_isa;
> set_bit(DEVICE_CATEGORY_INPUT, dc->categories);
> + /* i8042 is a device on the motherboard, and not pluggable by the user */
> + dc->user_creatable = false;
> }
>
> static const TypeInfo i8042_info = {
>