Looks good. Just a few questions from my side so I get up to speed with the 
changes in events.
Why do we need a lock per eventid ?

> +typedef struct _OVS_CT_EVENT_ENTRY {
> +    OVS_CT_ENTRY entry;
> +    UINT8 type;
> +    UINT64 pad[10];
> +} OVS_CT_EVENT_ENTRY, *POVS_CT_EVENT_ENTRY;

Why such a big pad?

Thanks,
Alin.

> -----Mesaj original-----
> De la: dev [mailto:dev-boun...@openvswitch.org] În numele Sairam
> Venugopal
> Trimis: Thursday, July 14, 2016 2:39 AM
> Către: dev@openvswitch.org
> Subiect: [ovs-dev] [PATCH 7/9] datapath-windows: Add support for multiple
> event queue in Event.c
> 
> Update Event.c to have multiple event queues and mechanism to retrieve
> the associated queue. Introduce OvsPostCtEvent and
> OvsRemoveCtEventEntry similar to OvsPostVportEvent and
> OvsRemoveVportEventEntry.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to