]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
rx: Lock call for KeepAliveOn/KeepAliveOff
authorAndrew Deason <adeason@sinenomine.net>
Wed, 14 Nov 2012 05:30:41 +0000 (23:30 -0600)
committerPaul Smeddle <paul.smeddle@gmail.com>
Wed, 5 Dec 2012 16:10:02 +0000 (08:10 -0800)
rxi_KeepAliveOn/Off expect the call lock to be held after the call has
been initialized. So, hold it in the rx_KeepAliveOn/Off callers.

Reviewed-on: http://gerrit.openafs.org/8463
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Simon Wilkinson <simonxwilkinson@gmail.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
(cherry picked from commit 866c05825c9964aa92740bcb0f20f26b451e65ea)

Change-Id: I2df1937f4098dde59a7fed344e1c975fdf0cdf1b
Reviewed-on: http://gerrit.openafs.org/8551
Reviewed-by: Ken Dreyer <ktdreyer@ktdreyer.com>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Paul Smeddle <paul.smeddle@gmail.com>
src/rx/rx.c

index f35bf837fe98d719215c101da3a535ea40241843..d4f3bd346c30936eabf16ef3e04b384942cef169 100644 (file)
@@ -6676,12 +6676,16 @@ rxi_KeepAliveOn(struct rx_call *call)
 void
 rx_KeepAliveOff(struct rx_call *call)
 {
+    MUTEX_ENTER(&call->lock);
     rxi_KeepAliveOff(call);
+    MUTEX_EXIT(&call->lock);
 }
 void
 rx_KeepAliveOn(struct rx_call *call)
 {
+    MUTEX_ENTER(&call->lock);
     rxi_KeepAliveOn(call);
+    MUTEX_EXIT(&call->lock);
 }
 
 void