>     NL_ERROR_TXTBSY = ((ULONG)-139),
>     /*the operation would block */
>     NL_ERROR_WOULDBLOCK = ((ULONG)-140),
> +    /* parsing failed */
> +    NL_ERROR_PARSEFAIL = ((ULONG)-141)

One more thing is, I don't know the basis of the netlink transactional error 
numbers that got defined in a previous commit. Except for the top few, they 
don't really map to errno_t.

In the nl_transact() function, we copy this error code as a errno_t, and then 
the errno doesn't make any sense at all.

So, maybe we should not define NL_ERROR_PARSEFAIL for now, and just fail the 
ioctl. We can figure out what to do with the NL_ERROR* codes later.

-- Nithin

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

Reply via email to