[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info
From: |
Zhi Yong Wu |
Subject: |
Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info |
Date: |
Thu, 24 May 2012 22:51:11 +0800 |
On Thu, May 24, 2012 at 10:43 PM, Jan Kiszka <address@hidden> wrote:
> On 2012-05-24 11:38, Zhi Yong Wu wrote:
>> On Thu, May 24, 2012 at 10:31 PM, Jan Kiszka <address@hidden> wrote:
>>> On 2012-05-24 11:27, Zhi Yong Wu wrote:
>>>> On Thu, May 24, 2012 at 10:25 PM, Jan Kiszka <address@hidden> wrote:
>>>>> Something mangled your reply and made it unreadable. Please retry.
>>>> Sorry. let it look like below. Do you think of it? typ=hubport
>>>>
>>>> (qemu) info network
>>>> virtio-net-pci.0: type=nic,model=virtio-net-pci,macaddr=52:54:00:12:34:56
>>>> \ hub0port0: type=hubport,
>>>> virtio-net-pci.1: type=nic,model=virtio-net-pci,macaddr=52:54:00:12:34:57
>>>> \ hub1port0: type=hubport,
>>>> virtio-net-pci.2: type=nic,model=virtio-net-pci,macaddr=52:54:00:12:34:58
>>>> \ u: type=user,net=10.0.2.0,restrict=off
>>>> e1000.0: type=nic,model=e1000,macaddr=52:54:00:12:34:59
>>>> \ ur: type=user,net=10.0.2.0,restrict=off
>>>> hub 1
>>>> port 1 peer user.1
>>>> port 0 peer virtio-net-pci.1
>>>> hub 0
>>>> port 1 peer user.0
>>>> port 0 peer virtio-net-pci.0
>>>
>>> My question remains: What added value we get from listing the hubs with
>>> its ports separately from the port connections? Also, how would this be
>>> printed:
>>>
>>> -net user -net dump -net nic
>> (qemu) info network
>> virtio-net-pci.0: type=nic,model=virtio-net-pci,macaddr=52:54:00:12:34:56
>> \ hub0port0: type=hubport,
>> virtio-net-pci.1: type=nic,model=virtio-net-pci,macaddr=52:54:00:12:34:57
>> \ hub1port0: type=hubport,
>> hub 1
>> port 2 peer dump.0
>> port 1 peer user.1
>> port 0 peer virtio-net-pci.1
>> hub 0
>> port 1 peer user.0
>> port 0 peer virtio-net-pci.0
>> (qemu)
>>
>>>
>>> The user should only be interested in the fact that user.0, dump.0 and
>>> <some_nic>.0 are attached to the same hub, not to which port of that hub.
>> OK, then let it seem like below. right?
>>
>> (qemu) info network
>> virtio-net-pci.0: type=nic,model=virtio-net-pci,macaddr=52:54:00:12:34:56
>> \ hub0port0: type=hubport,
>> virtio-net-pci.1: type=nic,model=virtio-net-pci,macaddr=52:54:00:12:34:57
>> \ hub1port0: type=hubport,
>> hub 1
>> \ dump.0
>> \ user.1
>> \ virtio-net-pci.1
>> hub 0
>> \ user.0
>> \ virtio-net-pci.0
>> (qemu)
>
> And, still, what is the added value of this verbose form compared to my
They are same, i think.
> compact proposal? Please don't remark that it's easier to implement. ;)
The implementation is not one difficult thing, if we reach agreement
about its layout.
For those NIC which aren't in one hub, they should been kept compact
with old qemu form.
>
> Jan
>
> --
> Siemens AG, Corporate Technology, CT T DE IT 1
> Corporate Competence Center Embedded Linux
--
Regards,
Zhi Yong Wu
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, (continued)
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Zhi Yong Wu, 2012/05/24
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Jan Kiszka, 2012/05/24
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Zhi Yong Wu, 2012/05/24
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Jan Kiszka, 2012/05/24
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Zhi Yong Wu, 2012/05/24
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Jan Kiszka, 2012/05/24
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Zhi Yong Wu, 2012/05/24
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Jan Kiszka, 2012/05/24
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info,
Zhi Yong Wu <=
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Jan Kiszka, 2012/05/24
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Zhi Yong Wu, 2012/05/24
- Re: [Qemu-devel] [PATCH v2 13/15] net: Remove obsolete vlan info, Zhi Yong Wu, 2012/05/24
[Qemu-devel] [PATCH v2 09/15] net: Rename non_vlan_clients to net_clients, zwu . kernel, 2012/05/23
[Qemu-devel] [PATCH v2 01/15] net: Add a hub net client, zwu . kernel, 2012/05/23
[Qemu-devel] [PATCH v2 02/15] net: Use hubs for the vlan feature, zwu . kernel, 2012/05/23
[Qemu-devel] [PATCH v2 12/15] net: Rename qemu_del_vlan_client() to qemu_del_net_client(), zwu . kernel, 2012/05/23
[Qemu-devel] [PATCH v2 03/15] net: Look up 'vlan' net clients using hubs, zwu . kernel, 2012/05/23
[Qemu-devel] [PATCH v2 04/15] hub: Check that hubs are configured correctly, zwu . kernel, 2012/05/23
[Qemu-devel] [PATCH v2 07/15] net: Remove vlan code from net.c, zwu . kernel, 2012/05/23