[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.13 v3 1/2] slirp: Add "query-usernet" QMP
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH for-2.13 v3 1/2] slirp: Add "query-usernet" QMP command |
Date: |
Wed, 2 May 2018 15:29:36 +0800 |
User-agent: |
Mutt/1.9.2 (2017-12-15) |
On Tue, 05/01 22:41, Samuel Thibault wrote:
> Hello,
>
> I'm sorry I didn't find the time to have a look at it before.
>
> In general it looks good, just a few things:
>
> Samuel
>
> Fam Zheng, le ven. 16 mars 2018 14:28:21 +0800, a ecrit:
> > + if (!net_hub_id_for_client(&s->nc, &vlan)) {
>
> That makes me think that IIUC, we shouldn't call them vlan, as mentioned
> in the recent vlan->hub renaming patches.
>
> > +{ 'struct': 'UsernetInfo',
> > + 'data': {
> > + 'id': 'str',
> > + 'vlan': 'int',
>
> and here as well, I guess.
OK, so rename this to hub? (I'm not familiar with the naming issue you pointed
out).
>
> > +void slirp_connection_info(Slirp *slirp, Monitor *mon)
> > +{
> > + const char *state;
> > + char buf[64];
> > + UsernetInfo info = { };
> > + UsernetConnectionList *cl;
> > +
> > + monitor_printf(mon, " Protocol[State] FD Source Address Port "
> > + "Dest. Address Port RecvQ SendQ\n");
> > +
> > + usernet_get_info(slirp, &info);
>
> Mmm, I don't see the content of info being freed here?
You're right, will fix.
Fam