From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kiszka Subject: Re: [PATCH v2 13/15] net: Remove obsolete vlan info Date: Wed, 23 May 2012 12:41:25 -0300 Message-ID: <4FBD0525.4040107@siemens.com> References: <1337786045-2277-1-git-send-email-zwu.kernel@gmail.com> <1337786045-2277-14-git-send-email-zwu.kernel@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: stefanha@linux.vnet.ibm.com, kvm@vger.kernel.org, linuxram@us.ibm.com, qemu-devel@nongnu.org, wuzhy@linux.vnet.ibm.com, pbonzini@redhat.com To: zwu.kernel@gmail.com Return-path: In-Reply-To: <1337786045-2277-14-git-send-email-zwu.kernel@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org List-Id: kvm.vger.kernel.org On 2012-05-23 12:14, zwu.kernel@gmail.com wrote: > From: Zhi Yong Wu > > Signed-off-by: Zhi Yong Wu > --- > net.c | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/net.c b/net.c > index 61dc28d..8c8e703 100644 > --- a/net.c > +++ b/net.c > @@ -1079,7 +1079,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; This looks suspicious - or the patch description is improvable. This is really just about removing that headline? And what about the indention of the lines printed afterward? It also leads me to the question how hub-based networks will be visualized on "info network", specifically when there are multiple hubs. Can you provide some more complex example of an info network output? Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux