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