6b68d1cfc8
The current versions of these functions have a fatal flaw. If a errhandler set and free call is made by another thread while the thread calling get is between the cmpset and retain then we will retain an invalid object. Fixing this by just using locking. This is not a critical path so this should be ok. Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
76 строки
2.6 KiB
C
76 строки
2.6 KiB
C
/* -*- Mode: C; c-basic-offset:4 ; indent-tabs-mode:nil -*- */
|
|
/*
|
|
* Copyright (c) 2004-2007 The Trustees of Indiana University and Indiana
|
|
* University Research and Technology
|
|
* Corporation. All rights reserved.
|
|
* Copyright (c) 2004-2005 The University of Tennessee and The University
|
|
* of Tennessee Research Foundation. All rights
|
|
* reserved.
|
|
* Copyright (c) 2004-2005 High Performance Computing Center Stuttgart,
|
|
* University of Stuttgart. All rights reserved.
|
|
* Copyright (c) 2004-2005 The Regents of the University of California.
|
|
* All rights reserved.
|
|
* Copyright (c) 2008-2009 Cisco Systems, Inc. All rights reserved.
|
|
* Copyright (c) 2015 Research Organization for Information Science
|
|
* and Technology (RIST). All rights reserved.
|
|
* Copyright (c) 2016-2017 Los Alamos National Security, LLC. All rights
|
|
* reserved.
|
|
* $COPYRIGHT$
|
|
*
|
|
* Additional copyrights may follow
|
|
*
|
|
* $HEADER$
|
|
*/
|
|
#include "ompi_config.h"
|
|
#include <stdio.h>
|
|
|
|
#include "ompi/mpi/c/bindings.h"
|
|
#include "ompi/runtime/params.h"
|
|
#include "ompi/communicator/communicator.h"
|
|
#include "ompi/errhandler/errhandler.h"
|
|
#include "ompi/win/win.h"
|
|
|
|
#if OMPI_BUILD_MPI_PROFILING
|
|
#if OPAL_HAVE_WEAK_SYMBOLS
|
|
#pragma weak MPI_Win_set_errhandler = PMPI_Win_set_errhandler
|
|
#endif
|
|
#define MPI_Win_set_errhandler PMPI_Win_set_errhandler
|
|
#endif
|
|
|
|
static const char FUNC_NAME[] = "MPI_Win_set_errhandler";
|
|
|
|
|
|
int MPI_Win_set_errhandler(MPI_Win win, MPI_Errhandler errhandler)
|
|
{
|
|
MPI_Errhandler tmp;
|
|
|
|
OPAL_CR_NOOP_PROGRESS();
|
|
|
|
if (MPI_PARAM_CHECK) {
|
|
OMPI_ERR_INIT_FINALIZE(FUNC_NAME);
|
|
|
|
if (ompi_win_invalid(win)) {
|
|
return OMPI_ERRHANDLER_INVOKE(MPI_COMM_WORLD, MPI_ERR_WIN,
|
|
FUNC_NAME);
|
|
} else if (NULL == errhandler ||
|
|
MPI_ERRHANDLER_NULL == errhandler ||
|
|
(OMPI_ERRHANDLER_TYPE_WIN != errhandler->eh_mpi_object_type &&
|
|
OMPI_ERRHANDLER_TYPE_PREDEFINED != errhandler->eh_mpi_object_type) ) {
|
|
return OMPI_ERRHANDLER_INVOKE(win, MPI_ERR_ARG, FUNC_NAME);
|
|
}
|
|
}
|
|
|
|
/* Prepare the new error handler */
|
|
OBJ_RETAIN(errhandler);
|
|
|
|
opal_mutex_lock (&win->w_lock);
|
|
/* Ditch the old errhandler, and decrement its refcount. */
|
|
tmp = win->error_handler;
|
|
win->error_handler = errhandler;
|
|
OBJ_RELEASE(tmp);
|
|
opal_mutex_unlock (&win->w_lock);
|
|
|
|
/* All done */
|
|
return MPI_SUCCESS;
|
|
}
|