[ovs-dev] [PATCH 4/4 v2] netlink-socket.c: implement get pid support on Windows
nithin at vmware.com
Tue Aug 19 15:55:41 UTC 2014
On Aug 18, 2014, at 5:08 PM, Alin Serdean <aserdean at cloudbasesolutions.com>
> Hi Nithin,
> Just a few thoughts:
> Since get PID is a special case would it not be better to treat it as a DeviceIoControl;dwIoControlCode like OVS_IOCTL_TRANSACT (i.e. OVS_IOCTL_GET_PID)?
> We can drop the extra headers and the whole Netlink message building/parsing since the goal is just to get a number.
Thanks for taking a look.
I want to keep the message exchange to be in netlink format. Though, the possibility of changing the format of the get_pid message is very remote, I want to stick with netlink.
As regards to including header files, I do need odp-netlink-ext.h for the definition of the ioctl. odp-netlink.h was needed for ovs_header, since there is an assumption that each message is at least (nl header + genlheader + ovs_header). I can get rid of the usage of ovs_header and remove the dependency on odp-netlink.h. This will involve some kernel changes too since there's a length check there.
I wand to postpone this task for this future since the commands are evolving. I'll keep a note of it in an issue.
More information about the dev