Re: [Qemu-devel] [PATCH] net: provide a friendly message when a user passes a bad -net tap, fd=X

[Anthony Liguori]

On 10/08/2010 06:28 PM, Alexander Graf wrote:
> On 09.10.2010, at 00:04, Anthony Liguori wrote:
>
>    
> > A lot of people copy libvirt's command line from ps -ef and then wonder why the
> > VM isn't working correctly.  Let's be kind and tell them what they should do
> > instead.
> >
> > Without this patch, if you run with an invalid -net tap,fd=X, the guest still
> > runs and we poll 100% on a bad file descriptor.  With this patch, you get:
> >
> > qemu: -net tap,fd=42: invalid fd= for tap network device.  If you're copying
> > from libvirt, use `virsh dom2xml-to-native' instead
> > qemu: -net tap,fd=42: Device 'tap' could not be initialized
> >
> > Signed-off-by: Anthony Liguori<address@hidden>
> >
> > diff --git a/hw/usb-uhci.c b/hw/usb-uhci.c
> > index 1d83400..5f9f749 100644
> > --- a/hw/usb-uhci.c
> > +++ b/hw/usb-uhci.c
> > @@ -64,7 +64,7 @@
> > #define UHCI_PORT_CSC   (1<<  1)
> > #define UHCI_PORT_CCS   (1<<  0)
> >
> > -#define FRAME_TIMER_FREQ 1000
> > +#define FRAME_TIMER_FREQ 500
> >
> > #define FRAME_MAX_LOOPS  100
> >
> > @@ -1054,7 +1054,7 @@ static void uhci_frame_timer(void *opaque)
> >      UHCIState *s = opaque;
> >
> >      /* prepare the timer for the next frame */
> > -    s->expire_time += (get_ticks_per_sec() / FRAME_TIMER_FREQ);
> > +    s->expire_time = qemu_get_clock(vm_clock) + (get_ticks_per_sec() / 
> > FRAME_TIMER_FREQ);
> >      
> How exactly is this related?
>    

So, didn't have a clean working directory.

Regards,

Anthony Liguori

> Alex
>
>
>