[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans |
Date: |
Mon, 7 Jun 2010 20:41:40 +0300 |
User-agent: |
Mutt/1.5.19 (2009-01-05) |
On Mon, Jun 07, 2010 at 05:42:55PM +0100, Paul Brook wrote:
> > On Mon, Jun 07, 2010 at 05:16:30PM +0100, Paul Brook wrote:
> > > > With -netdev, there now seems to be little need to support vlans,
> > > > enabling them leads to user confusion and bad performance.
> > > > Disable support for vlans by default, add config option to enable.
> > >
> > > No. If you want to remove vlans, then actually do that.
> >
> > How is this not what this patch does? You mean kill the code
> > completely, not just --contigure option?
>
> Yes. Configure options are bad. If code isn't worth enabling by default then
> you've got to have a very good reason why it exists at all.
>
> Paul
Everyone ok with disabling vlans with no config option?
--
MST
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, (continued)
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Michael S. Tsirkin, 2010/06/08
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Anthony Liguori, 2010/06/08
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Michael S. Tsirkin, 2010/06/08
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Gerd Hoffmann, 2010/06/08
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Paul Brook, 2010/06/08
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Anthony Liguori, 2010/06/08
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Markus Armbruster, 2010/06/09
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Chris Webb, 2010/06/10
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Michael S. Tsirkin, 2010/06/10
Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Michael S. Tsirkin, 2010/06/08