Thread (1 message) 1 message, 1 author, 2013-02-25

Re: [PATCH] Add fix to _do_madrpc to set the errno after umad_recv.

From: Sarat Kakarla <hidden>
Date: 2013-02-25 19:34:23

FOlks,

Just a gentle reminder, whenever you get a chance please review the following fix.

Thanks
Sarat



On 2/18/2013 2:10 PM, Sarat Kakarla wrote:
quoted hunk
  _do_madrpc() (in libibmad-1.3.3/src/rpc.c) sends the mad request through umad_send() and waits for a response from umad_recv() with the timeout passed  by caller. Linux error
number sent by peer in 'status' is not copied over to the process/thread local 'errno'.

Signed-off-by: Sarat Kakarla <redacted>

$diff -Naru rpc.c rpc_fix.c
--- rpc.c       2013-02-07 10:28:06.000000000 -0800
+++ rpc_fix.c   2013-02-18 13:57:15.000000000 -0800
@@ -175,6 +175,7 @@
                                          IB_MAD_TRID_F) != trid);

                 status = umad_status(rcvbuf);
+               errno = status;
                 if (!status)
                         return length;  /* done */
                 if (status == ENOMEM)
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help