[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: |
Paul Brook |
Subject: |
Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans |
Date: |
Mon, 7 Jun 2010 17:42:55 +0100 |
User-agent: |
KMail/1.13.3 (Linux/2.6.33-2-amd64; KDE/4.4.3; x86_64; ; ) |
> 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
- [Qemu-devel] [PATCH] configure: add an option to disable vlans, Michael S. Tsirkin, 2010/06/07
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Paul Brook, 2010/06/07
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Michael S. Tsirkin, 2010/06/07
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans,
Paul Brook <=
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Anthony Liguori, 2010/06/07
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Michael S. Tsirkin, 2010/06/07
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Anthony Liguori, 2010/06/07
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Michael S. Tsirkin, 2010/06/07
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Anthony Liguori, 2010/06/07
- Re: [Qemu-devel] [PATCH] configure: add an option to disable vlans, Anthony Liguori, 2010/06/07
- 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, Michael S. Tsirkin, 2010/06/08