[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] QMP: Introduce the documentation for query-n
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH v2] QMP: Introduce the documentation for query-netdev and info netdev |
Date: |
Mon, 21 Jun 2010 11:34:11 -0300 |
On Mon, 21 Jun 2010 10:27:11 +0200
Markus Armbruster <address@hidden> wrote:
> Luiz Capitulino <address@hidden> writes:
>
> > On Fri, 18 Jun 2010 15:28:42 -0500
> > Anthony Liguori <address@hidden> wrote:
> >
> >> On 06/18/2010 11:26 AM, Miguel Di Ciurcio Filho wrote:
> >> > These commands show the information about active backend network devices.
> >> >
> >> > Signed-off-by: Miguel Di Ciurcio Filho<address@hidden>
> >> > ---
> >> > qemu-monitor.hx | 105
> >> > +++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >> > 1 files changed, 105 insertions(+), 0 deletions(-)
> >> >
> >> > diff --git a/qemu-monitor.hx b/qemu-monitor.hx
> >> > index 9f62b94..8fc5ed6 100644
> >> > --- a/qemu-monitor.hx
> >> > +++ b/qemu-monitor.hx
> >> > @@ -1674,6 +1674,111 @@ show the various VLANs and the associated devices
> >> > ETEXI
> >> >
> >> > STEXI
> >> > address@hidden info netdev
> >> > +show information about the current backend network devices
> >> > +ETEXI
> >> > +SQMP
> >> > +query-netdev
> >> > +------------
> >> > +
> >> > +Each device is represented by a json-object. The returned value is a
> >> > json-array
> >> > +of all devices.
> >> > +
> >> > +Each json-object contains the following:
> >> > +
> >> > +- "id": the device's ID, must be unique (json-string)
> >> > +- "type": device type (json-string)
> >> > + - Possible values: "tap", "user", "vde", "socket"
> >> > +- "vlan": QEMU's internal vlan identification. Only present if the
> >> > device is
> >> > + attached to a VLAN (json-int, optional)
> >> > +- "peer": ID of the frontend device when on a 1:1 relationship
> >> > (json-string,
> >> > + optional)
> >> >
> >>
> >> I think we should only return items with a valid peer property and drop
> >> anything attached to vlans.
> >
> > The reason is that we're going to completely drop the vlan stuff, right?
> >
> >> The current info network already provides vlan information.
> >
> > What did you mean by that? info network is not going to be converted to
> > QMP,
> > that's why we're doing query-netdev.
>
> If we want to cover VLANs in QMP, we can either cover them in
> query-netdev, or provide a separate query, say query-vlans. The latter
> has the advantage that we won't have to mess with query-netdev when we
> get rid of VLANs.
>
> In my personal opinion, we should just ditch VLANs and be done with it.
Agreed.
>
> > If I got it right:
> >
> > - query-netdev: backend info
> > - query-qdm: device info
>
> I think you mean query-qtree. query-qdm is about available device
> models.
Yes.
[Qemu-devel] Re: [PATCH v2] QMP: Introduce the documentation for query-netdev and info netdev, Miguel Di Ciurcio Filho, 2010/06/21