[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/2] net: introduce command to query mac-tabl
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/2] net: introduce command to query mac-table information |
Date: |
Tue, 21 May 2013 09:38:01 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, May 21, 2013 at 12:46:09PM +0800, Amos Kong wrote:
> On Fri, May 17, 2013 at 09:39:31AM +0200, Stefan Hajnoczi wrote:
> > On Thu, May 16, 2013 at 07:07:25PM +0800, Amos Kong wrote:
>
> Hi Stefan,
>
> > > @@ -961,6 +961,44 @@ void print_net_client(Monitor *mon, NetClientState
> > > *nc)
> > > nc->info_str);
> > > }
> > >
> > > +MacTableInfoList *qmp_query_mac_table(bool has_name, const char *name,
> > > + Error **errp)
> > > +{
> > > + NetClientState *nc;
> > > + MacTableInfoList *table_list = NULL, *last_entry = NULL;
> > > +
> > > + QTAILQ_FOREACH(nc, &net_clients, next) {
> > > + MacTableInfoList *entry;
> > > + MacTableInfo *info;
> > > +
> > > + if (nc->info->type != NET_CLIENT_OPTIONS_KIND_NIC) {
> > > + continue;
> > > + }
>
> > > + if (has_name && strcmp(nc->name, name) != 0) {
> > > + continue;
> > > + }
>
> if (has_name && strcmp(nc->name, name) != 0) {
> error_setg(errp, "invalid net client name: %s", name);
> break;
> }
We still need to search the other NICs. Bailing early doesn't work:
imagine we have nic1 and nic2. If the user invokes query-mac-table nic2
then this code would return an error!
Stefan
- Re: [Qemu-devel] [PATCH v2 1/2] net: introduce MAC_TABLE_CHANGED event, (continued)