Author: kib
Date: Tue Jul 28 06:58:10 2015
New Revision: 285933
URL: https://svnweb.freebsd.org/changeset/base/285933
Log:
Remove useless acquire semantic from the atomic_add operation before
sosend(). The only release on the xp_snt_cnt is done after sosend(),
with an intent to synchronize with load_acq in svc_vc_ack().
Reviewed by: alc
Tested by: pho
Sponsored by: The FreeBSD Foundation
MFC after: 2 weeks
Modified:
head/sys/rpc/svc_vc.c
Modified: head/sys/rpc/svc_vc.c
==============================================================================
--- head/sys/rpc/svc_vc.c Tue Jul 28 06:55:08 2015 (r285932)
+++ head/sys/rpc/svc_vc.c Tue Jul 28 06:58:10 2015 (r285933)
@@ -860,7 +860,7 @@ svc_vc_reply(SVCXPRT *xprt, struct rpc_m
len = mrep->m_pkthdr.len;
*mtod(mrep, uint32_t *) =
htonl(0x80000000 | (len - sizeof(uint32_t)));
- atomic_add_acq_32(&xprt->xp_snd_cnt, len);
+ atomic_add_32(&xprt->xp_snd_cnt, len);
error = sosend(xprt->xp_socket, NULL, NULL, mrep, NULL,
0, curthread);
if (!error) {
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
