> On Apr 2, 2015, at 2:14 AM, Sorin Vinturis <[email protected]> 
> wrote:
> 
> Added a new IOCTL in order to retrieve the PID from the kernel datapath.
> The new method uses a direct and cleaner way, as opposed to the old way
> of using a Netlink transaction, avoiding the unnecessary overhead.
> 
> Signed-off-by: Sorin Vinturis <[email protected]>
> 

hi Sorin,
Thanks for addressing the review comments.

I just had one super minor comment. Looks good otherwise.

Acked-by: Nithin Raju <[email protected]>

> static NTSTATUS HandleDpTransactionCommon(
>                     POVS_USER_PARAMS_CONTEXT usrParamsCtx, UINT32 *replyLen);
> +static NTSTATUS OvsGetPidHandler(POVS_USER_PARAMS_CONTEXT usrParamsCtx,
> +                                    UINT32 *replyLen);

Nit: indentation of ‘replyLen’.


-- Nithin
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to