qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 3/3] monitor: add usb_attach and usb_detach


From: Alon Levy
Subject: Re: [Qemu-devel] [PATCH 3/3] monitor: add usb_attach and usb_detach
Date: Thu, 21 Oct 2010 12:58:20 +0200
User-agent: Mutt/1.5.21 (2010-09-15)

On Thu, Oct 21, 2010 at 12:00:12PM +0200, Gerd Hoffmann wrote:
> On 10/20/10 18:24, Luiz Capitulino wrote:
> >On Tue, 19 Oct 2010 15:35:01 +0200
> >Gerd Hoffmann<address@hidden>  wrote:
> >
> >>    Hi,
> >>
> >>>+        .help       = "attach USB device 'bus.addr'",
> >>
> >>>address@hidden usb_attach @var{devname}
> >>
> >>/me sees a mismatch here.
> >>
> >>There is still the use case question.  Also note that this might have
> >>unwanted side effects when drivers automagically attach/detach devices
> >>like usb-host.
> >
> >Alon, did you check this? I hope it doesn't blowup.
> >
> >>Having this purely for debugging/troubleshooting purposes would be fine
> >>with me, but the documentation should clearly say so.
> >
> >Is this useful in production or only in the development of new devices?
> 
> I'd say more a development tool.
> 
> >If
> >it's the letter, then I would even prefer enabling them only when DEBUG
> >is enabled.
> 
> Makes sense indeed.

I've sent a third version with fixes for the args_type you pointed out and the 
suggested default disabled behavior (I also added a flag to enable just this 
command, but made sure --enable-debug enables it too).

> 
> cheers,
>   Gerd



reply via email to

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