On 8/11/21 17:43, Paolo Valerio wrote:
> During a netlink transaction, in case of replies of type NLMSG_ERROR,
> the current behavior includes the translation of the error number
> received into a string that describes the error code.
> 
> Netlink replies may carry a more descriptive error message, and
> although it is possible to read those messages using the existing perf
> tracepoint, it is more convenient to retrieve them directly from ovs.
> 
> This patch extends nl_msg_nlmsgerr() so that it retrieves the message
> that later, if present, will be used by nl_sock_transact_multiple__()
> in place of the generic descriptive form of the error number.  This is
> particularly useful with tc that makes use of such kind of mechanism.
> 
> As an example, with this patch applied, the following generic message:
> 
> ovs|00239|netlink_socket|DBG|received NAK error=0 (Operation not supported)
> 
> becomes:
> 
> ovs|00239|netlink_socket|DBG|received NAK error=0 - Conntrack isn't enabled
> 
> The layout has been slightly modified to avoid nested parentheses.
> 
> Suggested-by: Marcelo Ricardo Leitner <[email protected]>
> Signed-off-by: Paolo Valerio <[email protected]>
> ---

Thanks, Paolo and Marcelo!
Sorry, this one fell through the cracks.  Applied now.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to