OASIS Mailing List ArchivesView the OASIS mailing list archive below
or browse/search using MarkMail.

 


Help: OASIS Mailing Lists Help | MarkMail Help

virtio-dev message

[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]


Subject: Re: [PATCH qemu 3/6] virtio-input: core code & base class


On Thu, Apr 10, 2014 at 02:22:17PM +0200, Gerd Hoffmann wrote:
> On Do, 2014-04-10 at 14:06 +0300, Michael S. Tsirkin wrote:
> > > +void virtio_input_send(VirtIOInput *vinput, virtio_input_event
> > *event)
> > > +{
> > > +    VirtQueueElement elem;
> > > +    int len;
> > > +
> > > +    if (!virtqueue_pop(vinput->evt, &elem)) {
> > > +        fprintf(stderr, "%s: virtqueue empty, dropping event\n",
> > __func__);
> > > +        return;
> > 
> > Looks scary.
> > 
> 
> It's not different from other input devices.  No buffer space -> drop
> event.  What else do you think should happen?  We could signal "you lost
> events" to the guest, but I suspect that buys us nothing.  Other input
> devices don't have that capability, so guests are likely not prepared to
> handle the situation.

For assigned device input events, how about we don't read events off the
input device file if there's nowhere to put them?

For things like sync that qemu generates, I suspect it's a good idea
to buffer them in QEMU otherwise guest will get out of sync, right?

I'm also pretty sure whoever's running the hypervisor does not
want to see the fprintf.

> Also, there isn't much they can actually do about
> it.
> 
> cheers,
>   Gerd



-- 
MST


[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]