Thanks Nithin. Will fix the parameter alignment in the first opportunity.
Eitan

-----Original Message-----
From: Nithin Raju 
Sent: Wednesday, September 10, 2014 6:25 AM
To: Eitan Eliahu
Cc: <dev@openvswitch.org>
Subject: Re: [ovs-dev] [PATCH v4] Windows NetLink Socket - Support for 
asynchronous event notification

This is already committed, but LG to me as well. Will look forward to the 
kernel implementation.

Only comment I had was w.r.t alignment in the following code:

> +    int ovs_msg_size = sizeof (struct nlmsghdr) + sizeof (struct genlmsghdr) 
> +
> +                               sizeof (struct ovs_header);


sizeof (struct ovs_header); should be aligned with sizeof (struct nlmsghdr).

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

Reply via email to