Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans

[Anthony Liguori]

On 06/07/2010 04:37 PM, Anthony Liguori wrote:
> On 06/07/2010 03:58 PM, Michael S. Tsirkin wrote:
> > On Mon, Jun 07, 2010 at 03:40:57PM -0500, Anthony Liguori wrote:
> > > On 06/07/2010 02:21 PM, Michael S. Tsirkin wrote:
> > > > So I see two ways to go forward: switch default value in my patch,
> > > > or disable vlans unconditionally.
> > > The problem with disabling vlans unconditionally is that you break -net
> > > socket and -net dump.
> > >
> > > If we can come up with an alternative way to do these things, I'm all
> > > for removing it.
> > Hmm, I'll try to look at supporting -net socket in netdev.
> > Does -net dump do anything that can't be done with tap+tcpdump?
>
> tap+tcpdump requires root privileges (even if you have a tap helper).
>
> Plus tcpdump doesn't help with slirp and -net dump is very useful for 
> debugging slirp.

Of course, you could add this functionality to netdev.  It's arguably 
better there too because then you can debug virtio-net+tap with full 
offload enabled (which you cannot do today).

Regards,

Anthony Liguori