Eitan,
Thanks for doing this.

I had some minor comments. LG otherwise.

Acked-by: Nithin Raju <nit...@vmware.com>

> +    OVS_NL_ATTR_MCAST_JOIN,  /*1/0 - Join/Unjoin */

minor: need a space before '1/0'.

> +    nl_msg_put_genlmsghdr(&request, 0, OVS_WIN_NL_CTRL_FAMILY_ID, 0,
> +                       OVS_CTRL_CMD_MC_SUBSCRIBE_REQ, 
> OVS_WIN_CONTROL_VERSION);

minor: alignment of 'OVS_CTRL_CMD_MC_SUBSCRIBE_REQ' is off.

thanks,
-- Nithin
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to