From dca3516765a4b5927b1877ca59d952baec42bc4a Mon Sep 17 00:00:00 2001 From: Nathan Hjelm Date: Mon, 13 Aug 2018 10:14:34 -0600 Subject: [PATCH] btl/vader: move memory barrier to where it belongs The write memory barrier was intended to precede setting a fast-box header but instead follows it. This commit moves the memory barrier to the intended location. Signed-off-by: Nathan Hjelm --- opal/mca/btl/vader/btl_vader_fbox.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/opal/mca/btl/vader/btl_vader_fbox.h b/opal/mca/btl/vader/btl_vader_fbox.h index abaf12811e4..17239ce8ef0 100644 --- a/opal/mca/btl/vader/btl_vader_fbox.h +++ b/opal/mca/btl/vader/btl_vader_fbox.h @@ -1,6 +1,6 @@ /* -*- Mode: C; c-basic-offset:4 ; indent-tabs-mode:nil -*- */ /* - * Copyright (c) 2011-2015 Los Alamos National Security, LLC. All rights + * Copyright (c) 2011-2018 Los Alamos National Security, LLC. All rights * reserved. * $COPYRIGHT$ * @@ -51,8 +51,8 @@ static inline void mca_btl_vader_fbox_set_header (mca_btl_vader_fbox_hdr_t *hdr, uint16_t seq, uint32_t size) { mca_btl_vader_fbox_hdr_t tmp = {.data = {.tag = tag, .seq = seq, .size = size}}; - hdr->ival = tmp.ival; opal_atomic_wmb (); + hdr->ival = tmp.ival; } /* attempt to reserve a contiguous segment from the remote ep */