[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event |
Date: |
Thu, 23 May 2013 17:43:25 +0300 |
On Thu, May 23, 2013 at 10:28:42AM -0400, Luiz Capitulino wrote:
> On Thu, 23 May 2013 13:24:59 +0300
> "Michael S. Tsirkin" <address@hidden> wrote:
>
> > On Thu, May 23, 2013 at 05:07:59PM +0800, Amos Kong wrote:
> > > Introduce this new QMP event to notify management after guest changes
> > > rx-filter configuration.
> > >
> > > Signed-off-by: Amos Kong <address@hidden>
> > > ---
> > > QMP/qmp-events.txt | 14 ++++++++++++++
> > > include/monitor/monitor.h | 1 +
> > > monitor.c | 1 +
> > > 3 files changed, 16 insertions(+)
> > >
> > > diff --git a/QMP/qmp-events.txt b/QMP/qmp-events.txt
> > > index 92fe5fb..ad6612b 100644
> > > --- a/QMP/qmp-events.txt
> > > +++ b/QMP/qmp-events.txt
> > > @@ -154,6 +154,20 @@ Data:
> > > "path": "/machine/peripheral/virtio-net-pci-0" },
> > > "timestamp": { "seconds": 1265044230, "microseconds": 450486 } }
> > >
> > > +RX_FILTER_CHANGED
> > > +-----------------
> > > +
> > > +Emitted when rx-filter configuration is changed by the guest.
> >
> > Please stress this is only for the NIC. It does not apply
> > to non-NIC netclients.
>
> Stress it in the event name too, please. I find RX_FILTER_CHANGED a bit
> generic.
What do you suggest?
NIC_RX_FILTER_CHANGED ?
> Also, although I haven't reviewed the next patch yet, I think you
> should move the event trigger to this patch.
That's hard because of the flag that's shared with the query
command.
> >
> > > +
> > > +Data:
> > > +
> > > +- "name": net client name (json-string)
> >
> > Maybe a device path here as well?
> >
> > > +
> > > +{ "event": "RX_FILTER_CHANGED",
> > > + "data": { "name": "vnet0" },
> > > + "timestamp": { "seconds": 1368697518, "microseconds": 326866 }}
> > > +}
> > > +
> > > DEVICE_TRAY_MOVED
> > > -----------------
> > >
> > > diff --git a/include/monitor/monitor.h b/include/monitor/monitor.h
> > > index 1a6cfcf..c495a67 100644
> > > --- a/include/monitor/monitor.h
> > > +++ b/include/monitor/monitor.h
> > > @@ -40,6 +40,7 @@ typedef enum MonitorEvent {
> > > QEVENT_BLOCK_JOB_ERROR,
> > > QEVENT_BLOCK_JOB_READY,
> > > QEVENT_DEVICE_DELETED,
> > > + QEVENT_RX_FILTER_CHANGED,
> > > QEVENT_DEVICE_TRAY_MOVED,
> > > QEVENT_SUSPEND,
> > > QEVENT_SUSPEND_DISK,
> > > diff --git a/monitor.c b/monitor.c
> > > index 6ce2a4e..4f7bd48 100644
> > > --- a/monitor.c
> > > +++ b/monitor.c
> > > @@ -489,6 +489,7 @@ static const char *monitor_event_names[] = {
> > > [QEVENT_BLOCK_JOB_ERROR] = "BLOCK_JOB_ERROR",
> > > [QEVENT_BLOCK_JOB_READY] = "BLOCK_JOB_READY",
> > > [QEVENT_DEVICE_DELETED] = "DEVICE_DELETED",
> > > + [QEVENT_RX_FILTER_CHANGED] = "RX_FILTER_CHANGED",
> > > [QEVENT_DEVICE_TRAY_MOVED] = "DEVICE_TRAY_MOVED",
> > > [QEVENT_SUSPEND] = "SUSPEND",
> > > [QEVENT_SUSPEND_DISK] = "SUSPEND_DISK",
> > > --
> > > 1.8.1.4
> >
- [Qemu-devel] [PATCH v3 0/2] mac programming over macvtap, Amos Kong, 2013/05/23
- [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event, Amos Kong, 2013/05/23
- Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event, Michael S. Tsirkin, 2013/05/23
- Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event, Luiz Capitulino, 2013/05/23
- Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event, Eric Blake, 2013/05/23
- Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event, Michael S. Tsirkin, 2013/05/23
- Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event, Luiz Capitulino, 2013/05/23
- Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event, Amos Kong, 2013/05/23
Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event, Eric Blake, 2013/05/23
[Qemu-devel] [PATCH v3 2/2] net: introduce command to query rx-filter information, Amos Kong, 2013/05/23