[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 13/16] net: Make the monitor output more reas
From: |
Zhi Yong Wu |
Subject: |
Re: [Qemu-devel] [PATCH v3 13/16] net: Make the monitor output more reasonable hub info |
Date: |
Fri, 25 May 2012 21:58:41 +0800 |
On Fri, May 25, 2012 at 9:49 PM, Jan Kiszka <address@hidden> wrote:
> On 2012-05-25 09:00, Zhi Yong Wu wrote:
>> On Fri, May 25, 2012 at 4:34 AM, Jan Kiszka <address@hidden> wrote:
>>> On 2012-05-24 14:59, address@hidden wrote:
>>>> From: Zhi Yong Wu <address@hidden>
>>>>
>>>> Signed-off-by: Zhi Yong Wu <address@hidden>
>>>> ---
>>>> net.c | 7 ++++++-
>>>> net/hub.c | 2 +-
>>>> 2 files changed, 7 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/net.c b/net.c
>>>> index 61dc28d..79ac51f 100644
>>>> --- a/net.c
>>>> +++ b/net.c
>>>> @@ -887,6 +887,12 @@ static const struct {
>>>> },
>>>> },
>>>> #endif /* CONFIG_NET_BRIDGE */
>>>> + [NET_CLIENT_TYPE_HUB] = {
>>>> + .type = "hubport",
>>>> + .desc = {
>>>> + { /* end of list */ }
>>>> + },
>>>> + },
>>>> };
>>>>
>>>> int net_client_init(Monitor *mon, QemuOpts *opts, int is_netdev)
>>>> @@ -1079,7 +1085,6 @@ void do_info_network(Monitor *mon)
>>>> NetClientState *nc, *peer;
>>>> net_client_type type;
>>>>
>>>> - monitor_printf(mon, "Devices not on any VLAN:\n");
>>>> QTAILQ_FOREACH(nc, &net_clients, next) {
>>>> peer = nc->peer;
>>>> type = nc->info->type;
>>>> diff --git a/net/hub.c b/net/hub.c
>>>> index 0cc385e..8a583ab 100644
>>>> --- a/net/hub.c
>>>> +++ b/net/hub.c
>>>> @@ -193,7 +193,7 @@ void net_hub_info(Monitor *mon)
>>>> QLIST_FOREACH(hub, &hubs, next) {
>>>> monitor_printf(mon, "hub %u\n", hub->id);
>>>> QLIST_FOREACH(port, &hub->ports, next) {
>>>> - monitor_printf(mon, " port %u peer %s\n", port->id,
>>>> + monitor_printf(mon, " \\ %s\n",
>>>> port->nc.peer ? port->nc.peer->name :
>>>> "<none>");
>>>> }
>>>> }
>>>
>>> I still do not agree with this formatting (peer -> hubport + hub ->
>>> abbreviated peers instead of just hub -> peers). But the series has a
>>> higher value than this, and we can fix on top - unless there is a need
>>> for another round anyway.
>> Can the following formatting let you be happy?
>>
>> (qemu) info network
>> hub 1
>> \ dump.0: type=dump,dump to qemu-vlan1.pcap (len=65536)
>> \ user.1: type=user,net=10.0.2.0,restrict=off
>> \ virtio-net-pci.0:
>> type=nic,model=virtio-net-pci,macaddr=52:54:00:12:34:58
>> hub 0
>> \ user.0: type=user,net=10.0.2.0,restrict=off
>> \ e1000.0: type=nic,model=e1000,macaddr=52:54:00:12:34:57
>> virtio-net-pci.1: type=nic,model=virtio-net-pci,macaddr=52:54:00:12:34:56
>> \ ur: type=tap,ifname=tap0,script=/etc/qemu-ifup,downscript=/etc/qemu-ifdown
>
> Yes, this is what I was propose. You can just save the peer indentions
> in the hub case.
OK. sent out v4 for this patch.
>
> Thanks!
> Jan
>
> --
> Siemens AG, Corporate Technology, CT T DE IT 1
> Corporate Competence Center Embedded Linux
--
Regards,
Zhi Yong Wu
- [Qemu-devel] [PATCH v3 09/16] net: Rename non_vlan_clients to net_clients, (continued)
- [Qemu-devel] [PATCH v3 09/16] net: Rename non_vlan_clients to net_clients, zwu . kernel, 2012/05/24
- [Qemu-devel] [PATCH v3 08/16] net: Remove VLANState, zwu . kernel, 2012/05/24
- [Qemu-devel] [PATCH v3 14/16] net: cleanup deliver/deliver_iov func pointers, zwu . kernel, 2012/05/24
- [Qemu-devel] [PATCH v3 15/16] net: determine if packets can be sent before net queue deliver packets, zwu . kernel, 2012/05/24
- [Qemu-devel] [PATCH v3 11/16] net: Rename vc local variables to nc, zwu . kernel, 2012/05/24
- [Qemu-devel] [PATCH v3 13/16] net: Make the monitor output more reasonable hub info, zwu . kernel, 2012/05/24
[Qemu-devel] [PATCH v3 10/16] net: Rename VLANClientState to NetClientState, zwu . kernel, 2012/05/24
[Qemu-devel] [PATCH v3 16/16] hub: add the support for hub own flow control, zwu . kernel, 2012/05/24
Re: [Qemu-devel] [PATCH v3 16/16] hub: add the support for hub own flow control, Zhi Yong Wu, 2012/05/25
Re: [Qemu-devel] [PATCH v3 16/16] hub: add the support for hub own flow control, Zhi Yong Wu, 2012/05/25
[Qemu-devel] [PATCH v3 12/16] net: Rename qemu_del_vlan_client() to qemu_del_net_client(), zwu . kernel, 2012/05/24
Re: [Qemu-devel] [PATCH v3 00/16] net: hub-based networking, Luiz Capitulino, 2012/05/24