[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH qemu 6/6] virtio-input: evdev passthrough
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH qemu 6/6] virtio-input: evdev passthrough |
Date: |
Thu, 10 Apr 2014 13:57:03 +0200 |
Hi,
> > +static void virtio_input_host_event(void *opaque)
> > +{
> > + VirtIOInputHost *vhost = opaque;
>
> I'd prefer a name that does not imply
> vhost infrastructure
ok.
> > + rc = ioctl(vhost->fd, EVIOCGVERSION, &ver);
> > + if (rc < 0) {
> > + error_setg(errp, "%s: is not an evdev device", vhost->evdev);
> > + goto err_close;
> > + }
>
> Hmm is that all?
> Don't we want to report versioning info to guests?
I guess the only thing we might want to here is bail out in case the
major version is != 1 (which implies a incompatible change). The major
version didn't change so far though, and I think it is highly unlikely
that it'll ever happen.
cheers,
Gerd
- [Qemu-devel] [PATCH qemu 1/6] pci: add virtio input pci device id, (continued)
[Qemu-devel] [PATCH qemu 6/6] virtio-input: evdev passthrough, Gerd Hoffmann, 2014/04/10
[Qemu-devel] [PATCH qemu 4/6] virtio-input: emulated devices, Gerd Hoffmann, 2014/04/10
[Qemu-devel] [PATCH qemu 3/6] virtio-input: core code & base class, Gerd Hoffmann, 2014/04/10