Re: [Qemu-devel] [PATCH v3 1/2] net: introduce RX_FILTER_CHANGED event

[Luiz Capitulino]

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.

Also, although I haven't reviewed the next patch yet, I think you
should move the event trigger to this patch.

> 
> > +
> > +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
>