Re: [Qemu-trivial] [PATCH] hw/input/pckbd: The i8042 device should not be user_creatable

[Paolo Bonzini]

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 = {
>