qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 07/14] usb-linux: If opening a device fails remo


From: Gerd Hoffmann
Subject: Re: [Qemu-devel] [PATCH 07/14] usb-linux: If opening a device fails remove it from our filter list
Date: Mon, 06 Jun 2011 12:24:25 +0200
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110419 Red Hat/3.1.10-1.el6_0 Thunderbird/3.1.10

On 06/01/11 16:37, Hans de Goede wrote:
Hi,

On 06/01/2011 02:32 PM, Gerd Hoffmann wrote:
On 05/31/11 11:35, Hans de Goede wrote:
So that we don't retry to open it every 2 seconds flooding stderr with
error messages.

The polling here is done intentionally, so the devices catched by the
filter show up in the guest automagically as soon as they are plugged
in. Just zapping the filter on failure isn't the right thing to do here.

Note I'm zapping the filter when we fail to open the device, not when it
is not present. This can happen for example when the qemu user does not
have rights on the usbfs device node.

It seems better to me to print the relevant error once, and then require
the user to redo the usb_add / device_add if necessary, then to flood
the monitor with repeating the same error every 2 seconds.

Devices magically disappearing is a problem for the management stack (such as libvirt) though. qemu does stuff like that in too many places already and we are trying to get rid of it.

Flooding the monitor (or stderr) indeed isn't nice though.

cheers,
  Gerd




reply via email to

[Prev in Thread] Current Thread [Next in Thread]