Skip to content

Commit

Permalink
wip: datapath-windows: Use the term hard stop
Browse files Browse the repository at this point in the history
Recently OVS adopted a policy of using the inclusive naming word list v1
[1, 2].

This patch addresses the use of the term abort within the scope of
the Windows datapath.  It does so by using the term hard stop instead.

This changes:
* Code comments
* One use-visible log message

[1] df5e5cf ("Documentation: Add section on inclusive language.")
[2] https://inclusivenaming.org/word-lists/

Signed-off-by: Simon Horman <horms@ovn.org>
  • Loading branch information
Simon Horman committed Feb 21, 2024
1 parent 11a3521 commit bae8e7b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions datapath-windows/ovsext/Netlink/NetlinkError.h
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ typedef enum _NL_ERROR_
NL_ERROR_BADMSG = ((ULONG)-104),
/* The operation was canceled */
NL_ERROR_CANCELED = ((ULONG)-105),
/* The software has caused a connection abort */
/* The software has caused a connection hard stop */
NL_ERROR_CONNABORTED = ((ULONG)-106),
/*The connection was refused */
NL_ERROR_CONNREFUSED = ((ULONG)-107),
Expand All @@ -148,7 +148,7 @@ typedef enum _NL_ERROR_
/* The network is down */
NL_ERROR_NETDOWN = ((ULONG)-116),
/* The network has dropped connection because of a reset (i.e. the
* connection was aborted by the network)
* connection was hard stopped by the network)
*/
NL_ERROR_NETRESET = ((ULONG)-117),
/* The network is unreachable */
Expand Down
2 changes: 1 addition & 1 deletion datapath-windows/ovsext/TunnelFilter.c
Original file line number Diff line number Diff line change
Expand Up @@ -1203,7 +1203,7 @@ OvsTunnelFilterRequestListProcess(POVS_TUNFLT_THREAD_CONTEXT threadCtx)
if (inTransaction) {
FwpmTransactionAbort(threadCtx->engineSession);
OVS_LOG_ERROR("Failed to execute request, status: %x.\
Transaction aborted.", status);
Transaction hard stop.", status);
}
}

Expand Down

0 comments on commit bae8e7b

Please sign in to comment.