From: Paolo Bonzini <pbonzini@redhat.com>
To: Luiz Capitulino <lcapitulino@redhat.com>
Cc: Stefan Hajnoczi <stefanha@gmail.com>,
Markus Armbruster <armbru@redhat.com>,
Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>,
kvm@vger.kernel.org, jan.kiszka@siemens.com,
qemu-devel@nongnu.org, zwu.kernel@gmail.com,
wuzhy@linux.vnet.ibm.com
Subject: Re: [Qemu-devel] [PATCH v3 00/16] net: hub-based networking
Date: Fri, 25 May 2012 15:14:39 +0200 [thread overview]
Message-ID: <4FBF85BF.6050403@redhat.com> (raw)
In-Reply-To: <20120525100746.51d7bf28@doriath.home>
Il 25/05/2012 15:07, Luiz Capitulino ha scritto:
> On Fri, 25 May 2012 14:59:25 +0200
> Paolo Bonzini <pbonzini@redhat.com> wrote:
>
>> Il 25/05/2012 14:53, Luiz Capitulino ha scritto:
>>>>> I agree it would be nice to drop entirely but I don't feel happy doing
>>>>> that to users who might have QEMU buried in scripts somewhere. One
>>>>> day they upgrade packages and suddenly their stuff doesn't work
>>>>> anymore.
>>> This is very similar to kqemu and I don't think we regret having dropped it.
>>
>> It's not. kqemu was putting maintainance burden, the aim of this patch
>> is exactly to isolate the feature to command-line parsing and a magic
>> net client. If you don't use -net, the new code is absolutely dead,
>> unlike kqemu.
>
> Let me quote Stefan on this thread:
>
> """
> The point of this patch series is to remove the special-case net.c code
> for the legacy "vlan" feature. Today's code makes it harder to
> implement a clean QOM model and is a burden for the net subsystem in
> general
> """
Still not sure what you mean... we removed kqemu and didn't give an
alternative. This time we are providing an alternative.
Paolo
WARNING: multiple messages have this Message-ID (diff)
From: Paolo Bonzini <pbonzini@redhat.com>
To: Luiz Capitulino <lcapitulino@redhat.com>
Cc: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>,
kvm@vger.kernel.org, Stefan Hajnoczi <stefanha@gmail.com>,
qemu-devel@nongnu.org, Markus Armbruster <armbru@redhat.com>,
zwu.kernel@gmail.com, wuzhy@linux.vnet.ibm.com,
jan.kiszka@siemens.com
Subject: Re: [Qemu-devel] [PATCH v3 00/16] net: hub-based networking
Date: Fri, 25 May 2012 15:14:39 +0200 [thread overview]
Message-ID: <4FBF85BF.6050403@redhat.com> (raw)
In-Reply-To: <20120525100746.51d7bf28@doriath.home>
Il 25/05/2012 15:07, Luiz Capitulino ha scritto:
> On Fri, 25 May 2012 14:59:25 +0200
> Paolo Bonzini <pbonzini@redhat.com> wrote:
>
>> Il 25/05/2012 14:53, Luiz Capitulino ha scritto:
>>>>> I agree it would be nice to drop entirely but I don't feel happy doing
>>>>> that to users who might have QEMU buried in scripts somewhere. One
>>>>> day they upgrade packages and suddenly their stuff doesn't work
>>>>> anymore.
>>> This is very similar to kqemu and I don't think we regret having dropped it.
>>
>> It's not. kqemu was putting maintainance burden, the aim of this patch
>> is exactly to isolate the feature to command-line parsing and a magic
>> net client. If you don't use -net, the new code is absolutely dead,
>> unlike kqemu.
>
> Let me quote Stefan on this thread:
>
> """
> The point of this patch series is to remove the special-case net.c code
> for the legacy "vlan" feature. Today's code makes it harder to
> implement a clean QOM model and is a burden for the net subsystem in
> general
> """
Still not sure what you mean... we removed kqemu and didn't give an
alternative. This time we are providing an alternative.
Paolo
next prev parent reply other threads:[~2012-05-25 13:15 UTC|newest]
Thread overview: 103+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-24 17:59 [PATCH v3 00/16] net: hub-based networking zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 01/16] net: Add a hub net client zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 02/16] net: Use hubs for the vlan feature zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 03/16] net: Look up 'vlan' net clients using hubs zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 04/16] hub: Check that hubs are configured correctly zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 05/16] net: Drop vlan argument to qemu_new_net_client() zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 06/16] net: Remove vlan qdev property zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 07/16] net: Remove vlan code from net.c zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 08/16] net: Remove VLANState zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 09/16] net: Rename non_vlan_clients to net_clients zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 10/16] net: Rename VLANClientState to NetClientState zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 11/16] net: Rename vc local variables to nc zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 12/16] net: Rename qemu_del_vlan_client() to qemu_del_net_client() zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 13/16] net: Make the monitor output more reasonable hub info zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 20:34 ` Jan Kiszka
2012-05-24 20:34 ` [Qemu-devel] " Jan Kiszka
2012-05-25 0:48 ` Zhi Yong Wu
2012-05-25 0:48 ` [Qemu-devel] " Zhi Yong Wu
2012-05-25 12:00 ` Zhi Yong Wu
2012-05-25 12:00 ` [Qemu-devel] " Zhi Yong Wu
2012-05-25 13:49 ` Jan Kiszka
2012-05-25 13:49 ` [Qemu-devel] " Jan Kiszka
2012-05-25 13:58 ` Zhi Yong Wu
2012-05-25 13:58 ` [Qemu-devel] " Zhi Yong Wu
2012-05-24 17:59 ` [PATCH v3 14/16] net: cleanup deliver/deliver_iov func pointers zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 15/16] net: determine if packets can be sent before net queue deliver packets zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-24 17:59 ` [PATCH v3 16/16] hub: add the support for hub own flow control zwu.kernel
2012-05-24 17:59 ` [Qemu-devel] " zwu.kernel
2012-05-25 7:04 ` Paolo Bonzini
2012-05-25 7:04 ` [Qemu-devel] " Paolo Bonzini
2012-05-25 7:48 ` Zhi Yong Wu
2012-05-25 7:48 ` [Qemu-devel] " Zhi Yong Wu
2012-05-25 10:08 ` Paolo Bonzini
2012-05-25 10:08 ` [Qemu-devel] " Paolo Bonzini
2012-05-25 10:54 ` Zhi Yong Wu
2012-05-25 10:54 ` [Qemu-devel] " Zhi Yong Wu
2012-05-25 8:04 ` Zhi Yong Wu
2012-05-25 8:04 ` [Qemu-devel] " Zhi Yong Wu
2012-05-25 8:18 ` Zhi Yong Wu
2012-05-25 8:18 ` [Qemu-devel] " Zhi Yong Wu
2012-05-24 20:53 ` [Qemu-devel] [PATCH v3 00/16] net: hub-based networking Luiz Capitulino
2012-05-24 20:53 ` Luiz Capitulino
2012-05-25 0:47 ` Zhi Yong Wu
2012-05-25 0:47 ` [Qemu-devel] " Zhi Yong Wu
2012-05-25 12:49 ` Luiz Capitulino
2012-05-25 12:49 ` Luiz Capitulino
2012-05-25 10:07 ` Stefan Hajnoczi
2012-05-25 10:07 ` [Qemu-devel] " Stefan Hajnoczi
2012-05-25 11:18 ` Markus Armbruster
2012-05-25 11:18 ` [Qemu-devel] " Markus Armbruster
2012-05-25 12:01 ` Stefan Hajnoczi
2012-05-25 12:01 ` Stefan Hajnoczi
2012-05-25 12:30 ` Markus Armbruster
2012-05-25 12:53 ` Luiz Capitulino
2012-05-25 12:53 ` Luiz Capitulino
2012-05-25 12:59 ` Paolo Bonzini
2012-05-25 12:59 ` Paolo Bonzini
2012-05-25 13:07 ` Luiz Capitulino
2012-05-25 13:07 ` [Qemu-devel] " Luiz Capitulino
2012-05-25 13:14 ` Paolo Bonzini [this message]
2012-05-25 13:14 ` Paolo Bonzini
2012-05-25 13:18 ` Luiz Capitulino
2012-05-25 13:18 ` Luiz Capitulino
2012-05-25 13:19 ` Paolo Bonzini
2012-05-25 13:19 ` Paolo Bonzini
2012-05-25 13:30 ` Luiz Capitulino
2012-05-25 13:30 ` Luiz Capitulino
2012-05-25 13:37 ` Paolo Bonzini
2012-05-25 13:37 ` Paolo Bonzini
2012-05-25 13:43 ` Luiz Capitulino
2012-05-25 13:43 ` Luiz Capitulino
2012-05-25 13:47 ` Paolo Bonzini
2012-05-25 13:47 ` [Qemu-devel] " Paolo Bonzini
2012-05-25 13:56 ` Luiz Capitulino
2012-05-25 13:56 ` Luiz Capitulino
2012-05-28 11:17 ` Stefan Hajnoczi
2012-05-28 11:17 ` Stefan Hajnoczi
2012-05-28 13:25 ` Luiz Capitulino
2012-05-28 13:25 ` Luiz Capitulino
2012-05-29 8:14 ` Markus Armbruster
2012-06-04 4:48 ` Anthony Liguori
2012-06-04 4:48 ` Anthony Liguori
2012-06-04 7:24 ` Markus Armbruster
2012-06-04 4:56 ` Anthony Liguori
2012-06-04 4:56 ` [Qemu-devel] " Anthony Liguori
2012-06-04 13:09 ` Luiz Capitulino
2012-06-04 13:09 ` Luiz Capitulino
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FBF85BF.6050403@redhat.com \
--to=pbonzini@redhat.com \
--cc=armbru@redhat.com \
--cc=jan.kiszka@siemens.com \
--cc=kvm@vger.kernel.org \
--cc=lcapitulino@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@gmail.com \
--cc=stefanha@linux.vnet.ibm.com \
--cc=wuzhy@linux.vnet.ibm.com \
--cc=zwu.kernel@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.