From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1O5Q7U-00028m-Bo for qemu-devel@nongnu.org; Fri, 23 Apr 2010 17:09:04 -0400 Received: from [140.186.70.92] (port=43412 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1O5Q7S-00028N-Vg for qemu-devel@nongnu.org; Fri, 23 Apr 2010 17:09:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1O5Q7R-0005Ft-5V for qemu-devel@nongnu.org; Fri, 23 Apr 2010 17:09:02 -0400 Received: from mx1.redhat.com ([209.132.183.28]:15572) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1O5Q7Q-0005Fo-T1 for qemu-devel@nongnu.org; Fri, 23 Apr 2010 17:09:01 -0400 Date: Fri, 23 Apr 2010 18:08:52 -0300 From: Luiz Capitulino Message-ID: <20100423180852.67d5f68f@redhat.com> In-Reply-To: <1271340427-12579-7-git-send-email-miguel.filho@gmail.com> References: <1271340427-12579-1-git-send-email-miguel.filho@gmail.com> <1271340427-12579-7-git-send-email-miguel.filho@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH v3 06/12] net: tap/tap-win32: use info_dict instead of info_str List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Miguel Di Ciurcio Filho Cc: qemu-devel@nongnu.org, armbru@redhat.com On Thu, 15 Apr 2010 11:07:01 -0300 Miguel Di Ciurcio Filho wrote: > Signed-off-by: Miguel Di Ciurcio Filho > --- > net/tap-win32.c | 9 ++++++--- > net/tap.c | 18 +++++++++++++----- > 2 files changed, 19 insertions(+), 8 deletions(-) > > diff --git a/net/tap-win32.c b/net/tap-win32.c > index 74348da..a54cd31 100644 > --- a/net/tap-win32.c > +++ b/net/tap-win32.c > @@ -32,6 +32,8 @@ > #include "net.h" > #include "sysemu.h" > #include "qemu-error.h" > +#include "qdict.h" > +#include "qstring.h" > #include > #include > #include > @@ -688,10 +690,11 @@ static int tap_win32_init(VLANState *vlan, const char *model, > > nc = qemu_new_net_client(&net_tap_win32_info, vlan, NULL, model, name); > > - s = DO_UPCAST(TAPState, nc, nc); > + nc->info_dict = qdict_new(); > > - snprintf(s->nc.info_str, sizeof(s->nc.info_str), > - "tap: ifname=%s", ifname); > + qdict_put(nc->info_dict, "ifname", qstring_from_str(ifname)); Isn't it better/needed to also do: qdict_put(nc->info_dict, "tap", qbool_from_int(1)); So that you can reliably recognize this is a tap nic? Another option is to have model=tap, although I'm not sure if it has any undesirable implication. What do you think, Markus? Note that same applies for slirp,vde etc.. > + > + s = DO_UPCAST(TAPState, nc, nc); > > s->handle = handle; > > diff --git a/net/tap.c b/net/tap.c > index 303d69f..8ba7eed 100644 > --- a/net/tap.c > +++ b/net/tap.c > @@ -39,6 +39,9 @@ > #include "qemu-char.h" > #include "qemu-common.h" > #include "qemu-error.h" > +#include "qdict.h" > +#include "qint.h" > +#include "qstring.h" > > #include "net/tap-linux.h" > > @@ -447,18 +450,23 @@ int net_init_tap(QemuOpts *opts, Monitor *mon, const char *name, VLANState *vlan > } > > if (qemu_opt_get(opts, "fd")) { > - snprintf(s->nc.info_str, sizeof(s->nc.info_str), "fd=%d", fd); > + if (s->nc.info_dict == NULL) > + s->nc.info_dict = qdict_new(); Style: in QEMU if you to have to use braces. > + > + qdict_put(s->nc.info_dict, "fd", qint_from_int(fd)); > } else { > const char *ifname, *script, *downscript; > + if (s->nc.info_dict == NULL) > + s->nc.info_dict = qdict_new(); > > ifname = qemu_opt_get(opts, "ifname"); > script = qemu_opt_get(opts, "script"); > downscript = qemu_opt_get(opts, "downscript"); > > - snprintf(s->nc.info_str, sizeof(s->nc.info_str), > - "ifname=%s,script=%s,downscript=%s", > - ifname, script, downscript); > - > + qdict_put(s->nc.info_dict, "ifname", qstring_from_str(ifname)); > + qdict_put(s->nc.info_dict, "script", qstring_from_str(script)); > + qdict_put(s->nc.info_dict, "downscript", qstring_from_str(downscript)); Did you consider using qobject_from_jsonf()? > + > if (strcmp(downscript, "no") != 0) { > snprintf(s->down_script, sizeof(s->down_script), "%s", downscript); > snprintf(s->down_script_arg, sizeof(s->down_script_arg), "%s", ifname);