[ofw] [PATCH] winverbs/nd: do not convert wv_io_timeout to nd_timeout

[Sean Hefty]

wv_io_timeout is an error value, but nd_timeout is a
success value.  This means that an overlapped request
can be completed in error, but GetOverlappedResult can
return a success value.  Instead, if a timeout is fatal,
report the status as wv_io_timeout, but if the timeout
can be retried, then report the status as wv_timeout.

Signed-off-by: Sean Hefty <sean.hefty at intel.com>
---
This should be added to the 2.2 release.  It passes all ND logo tests.

 trunk/ulp/netdirect/user/nd_base.cpp |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/trunk/ulp/netdirect/user/nd_base.cpp b/trunk/ulp/netdirect/user/nd_base.cpp
index e51bc59..5dfaf11 100644
--- a/trunk/ulp/netdirect/user/nd_base.cpp
+++ b/trunk/ulp/netdirect/user/nd_base.cpp
@@ -58,8 +58,6 @@ HRESULT NDConvertWVStatus(HRESULT hr)
 	switch (hr) {
 	case WV_IO_PENDING:
 		return ND_PENDING;
-	case WV_IO_TIMEOUT:
-		return ND_TIMEOUT;
 	default:
 		return hr;
 	}