Merge pull request #10607 from luzpaz/typos/ompi

Fix typos in ompi/ subdirectory

Author: jsquyres

ompi/attribute/attribute.c

@@ -1333,7 +1333,7 @@ static int set_value(ompi_attribute_type_t type, void *object,
     ret = opal_hash_table_set_value_uint32(*attr_hash, key, new_attr);
 
     /* Increase the reference count of the object, only if there was no
-       old atribute/no old entry in the object's key hash */
+       old attribute/no old entry in the object's key hash */
     if (OMPI_SUCCESS == ret && !had_old) {
         OBJ_RETAIN(keyval);
     }

ompi/attribute/attribute_predefined.c

@@ -52,7 +52,7 @@
  *
  * MPI_HOST is set to MPI_PROC_NULL (per MPI-1, see 7.1.1, p192).
  *
- * MPI_IO is set to MPI_ANY_SOURCE.  We may need to revist this.
+ * MPI_IO is set to MPI_ANY_SOURCE.  We may need to revisit this.
  *
  * MPI_WTIME_IS_GLOBAL is set to 0 (a conservative answer).
  *
@@ -78,7 +78,7 @@
  * integer-valued attributes, *BUT* at least the MPI_WIN_SIZE is an
  * MPI_Aint, so in terms of consistency, both should be the same --
  * hence, we treat them as MPI-2 Fortran integer-valued attributes.
- * All three of these atrributes have NULL_COPY_FN copy functions; it
+ * All three of these attributes have NULL_COPY_FN copy functions; it
  * doesn't make sense to copy them to new windows (because they're
  * values specific and unique to each window) -- especially when
  * WIN_CREATE will explicitly set them on new windows anyway.

ompi/communicator/comm.c

@@ -2049,7 +2049,7 @@ int ompi_comm_get_rprocs (ompi_communicator_t *local_comm, ompi_communicator_t *
         PMIX_DATA_BUFFER_RELEASE(sbuf);
     }
 
-    /* broadcast name list to all proceses in local_comm */
+    /* broadcast name list to all processes in local_comm */
     rc = local_comm->c_coll->coll_bcast( recvbuf, rlen, MPI_BYTE,
                                         local_leader, local_comm,
                                         local_comm->c_coll->coll_bcast_module);

ompi/communicator/comm_cid.c

@@ -725,7 +725,7 @@ static int ompi_comm_activate_complete (ompi_communicator_t **newcomm, ompi_comm
     /**
      * Check to see if this process is in the new communicator.
      *
-     * Specifically, this function is invoked by all proceses in the
+     * Specifically, this function is invoked by all processes in the
      * old communicator, regardless of whether they are in the new
      * communicator or not.  This is because it is far simpler to use
      * MPI collective functions on the old communicator to determine
@@ -736,7 +736,7 @@ static int ompi_comm_activate_complete (ompi_communicator_t **newcomm, ompi_comm
      *
      * That being said, only processes in the new communicator need to
      * select a coll module for the new communicator.  More
-     * specifically, proceses who are not in the new communicator
+     * specifically, processes who are not in the new communicator
      * should *not* select a coll module -- for example,
      * ompi_comm_rank(newcomm) returns MPI_UNDEFINED for processes who
      * are not in the new communicator.  This can cause errors in the

ompi/communicator/comm_init.c

@@ -84,7 +84,7 @@ OBJ_CLASS_INSTANCE(ompi_communicator_t, opal_infosubscriber_t,
                    ompi_comm_destruct);
 
 /* This is the counter for the number of communicators, which contain
-   process with more than one jobid. This counter is a usefull
+   process with more than one jobid. This counter is a useful
    shortcut for finalize and abort. */
 int ompi_comm_num_dyncomm=0;
 
@@ -231,7 +231,7 @@ int ompi_comm_init_mpi3 (void)
     ompi_attr_get_ref();
 
     /* We have to create a hash (although it is legal to leave this
-       filed NULL -- the attribute accessor functions will intepret
+       filed NULL -- the attribute accessor functions will interpret
        this as "there are no attributes cached on this object")
        because MPI_COMM_WORLD has some predefined attributes. */
     ompi_attr_hash_init(&ompi_mpi_comm_world.comm.c_keyhash);

ompi/communicator/communicator.h

@@ -273,7 +273,7 @@ struct ompi_communicator_t {
                to a child*/
     int c_id_start_index; /* the starting index of the block of cids
                  allocated to this communicator*/
-    uint32_t c_epoch;  /* Identifier used to differenciate between two communicators
+    uint32_t c_epoch;  /* Identifier used to differentiate between two communicators
                           using the same c_contextid (not at the same time, obviously) */
 
     ompi_group_t        *c_local_group;
@@ -346,7 +346,7 @@ struct ompi_communicator_t {
 typedef struct ompi_communicator_t ompi_communicator_t;
 
 /**
- * Padded struct to maintain back compatibiltiy.
+ * Padded struct to maintain back compatibility.
  *
  * The following ompi_predefined_xxx_t structure is used to maintain
  * backwards binary compatibility for MPI applications compiled
@@ -395,7 +395,7 @@ typedef struct ompi_communicator_t ompi_communicator_t;
  *
  * - union of struct and padding - Similar to current implementation
  *   except using a union for the parent.  This worked except in cases
- *   where the compilers did not support C99 union static initalizers.
+ *   where the compilers did not support C99 union static initializers.
  *   It would have been a pain to convert a bunch of the code to use
  *   non-static initializers (e.g., MPI datatypes).
  */
@@ -452,7 +452,7 @@ OMPI_DECLSPEC extern ompi_predefined_communicator_t ompi_mpi_comm_null;
 
 /*
  * These variables are for the MPI F03 bindings (F03 must bind Fortran
- * varaiables to symbols; it cannot bind Fortran variables to the
+ * variables to symbols; it cannot bind Fortran variables to the
  * address of a C variable).
  */
 OMPI_DECLSPEC extern ompi_predefined_communicator_t *ompi_mpi_comm_world_addr;
@@ -480,7 +480,7 @@ OMPI_DECLSPEC extern ompi_predefined_communicator_t *ompi_mpi_comm_null_addr;
  * ompi_comm_invalid() as originally coded -- per the MPI-1
  * definition, where MPI_COMM_NULL is an invalid communicator.
  * The MPI_Comm_c2f() function, therefore, calls
- * ompi_comm_invalid() but also explictily checks to see if the
+ * ompi_comm_invalid() but also explicitly checks to see if the
  * handle is MPI_COMM_NULL.
  */
 static inline int ompi_comm_invalid (const ompi_communicator_t* comm)
@@ -1072,7 +1072,7 @@ int ompi_comm_determine_first ( ompi_communicator_t *intercomm,
                                 int high );
 
 /**
- * This is a routine determining wether the local or the
+ * This is a routine determining whether the local or the
  * remote group will be first in the new intra-comm.
  * It does not communicate to exchange the "high" values; used in Agree
  */

ompi/communicator/ft/comm_ft_reliable_bcast.c

@@ -170,7 +170,7 @@ static void ompi_comm_rbcast_bml_recv_cb(
     msg = (ompi_comm_rbcast_message_t*) descriptor->des_segments->seg_addr.pval;
 
     OPAL_OUTPUT_VERBOSE((10, ompi_ftmpi_output_handle,
-                         "%s %s: Info: Recieved rbcast tag for communicator with CID %3d:%d",
+                         "%s %s: Info: Received rbcast tag for communicator with CID %3d:%d",
                          OMPI_NAME_PRINT(OMPI_PROC_MY_NAME), __func__, msg->cid, msg->epoch));
 
     /* Find the target comm */
@@ -190,7 +190,7 @@ static void ompi_comm_rbcast_bml_recv_cb(
     /* Check if this is a delayed rbcast for an old communicator whose CID has been reused */
     if(OPAL_UNLIKELY( comm->c_epoch != msg->epoch )) {
         OPAL_OUTPUT_VERBOSE((2, ompi_ftmpi_output_handle,
-                             "%s %s: Info: Received a late rbcast order for the communicator with CID %3d:%d when is is now at epoch %d - ignoring, nothing to do",
+                             "%s %s: Info: Received a late rbcast order for the communicator with CID %3d:%d when it is now at epoch %d - ignoring, nothing to do",
                              OMPI_NAME_PRINT(OMPI_PROC_MY_NAME), __func__, msg->cid, msg->epoch, comm->c_epoch));
         return;
     }
@@ -204,7 +204,7 @@ static void ompi_comm_rbcast_bml_recv_cb(
         }
     }
     else {
-        /* During finalize, we loosly synchronize so it may happen 
+        /* During finalize, we loosely synchronize so it may happen 
          * that we keep receiving messages after we deregistered the type.
          * Any other time, this is indicative of a problem.
          */

ompi/datatype/ompi_datatype.h

@@ -91,7 +91,7 @@ typedef struct ompi_datatype_t ompi_datatype_t;
 OMPI_DECLSPEC OBJ_CLASS_DECLARATION(ompi_datatype_t);
 
 /**
- * Padded struct to maintain back compatibiltiy.
+ * Padded struct to maintain back compatibility.
  * See opal/communicator/communicator.h comments with struct opal_communicator_t
  * for full explanation why we chose the following padding construct for predefines.
  */
@@ -423,7 +423,7 @@ OMPI_DECLSPEC int ompi_datatype_pack_external_size( const char datarep[], int in
  * converted to (1,larger_datatype).  This comes up in pack/unpack if
  * the datatype is [int4b,empty4b] for example.  With that datatype the
  * (count,datatype) path has to loop over the count processing each
- * occurrance of the datatype, but a larger type created via
+ * occurrence of the datatype, but a larger type created via
  * MPI_Type_contiguous(count,datatype,) will have a single description
  * entry describing the whole vector and go through pack/unpack much
  * faster.

ompi/datatype/ompi_datatype_args.c

@@ -365,8 +365,8 @@ int32_t ompi_datatype_copy_args( const ompi_datatype_t* source_data,
 {
     ompi_datatype_args_t* pArgs = (ompi_datatype_args_t*)source_data->args;
 
-    /* Increase the reference count of the datatype enveloppe. This
-     * prevent us from making extra copies for the enveloppe (which is mostly
+    /* Increase the reference count of the datatype envelope. This
+     * prevent us from making extra copies for the envelope (which is mostly
      * a read only memory).
      */
     if( NULL != pArgs ) {
@@ -391,7 +391,7 @@ int32_t ompi_datatype_release_args( ompi_datatype_t* pData )
     OPAL_THREAD_ADD_FETCH32(&pArgs->ref_count, -1);
     if( 0 == pArgs->ref_count ) {
         /* There are some duplicated datatypes around that have a pointer to this
-         * args. We will release them only when the last datatype will dissapear.
+         * args. We will release them only when the last datatype will disappear.
          */
         for( i = 0; i < pArgs->cd; i++ ) {
             if( !(ompi_datatype_is_predefined(pArgs->d[i])) ) {
@@ -448,7 +448,7 @@ static inline int __ompi_datatype_pack_description( ompi_datatype_t* datatype,
     position = (int*)next_packed;
     next_packed += sizeof(int) * args->cd;
 
-    /* copy the aray of counts (32 bits aligned) */
+    /* copy the array of counts (32 bits aligned) */
     memcpy( next_packed, args->i, sizeof(int) * args->ci );
     next_packed += args->ci * sizeof(int);
 

ompi/datatype/ompi_datatype_external.c

@@ -57,7 +57,7 @@ int ompi_datatype_pack_external(const char datarep[], const void *inbuf, int inc
         return MPI_ERR_TRUNCATE;
     }
 
-    /* Prepare the iovec with all informations */
+    /* Prepare the iovec with all information */
     invec.iov_base = (char*) outbuf + (*position);
     invec.iov_len = size;
 
@@ -97,7 +97,7 @@ int ompi_datatype_unpack_external (const char datarep[], const void *inbuf, MPI_
         return MPI_ERR_TRUNCATE;
     }
 
-    /* Prepare the iovec with all informations */
+    /* Prepare the iovec with all information */
     outvec.iov_base = (char*) inbuf + (*position);
     outvec.iov_len = size;
 

ompi/datatype/ompi_datatype_internal.h

@@ -455,7 +455,7 @@ extern const ompi_datatype_t* ompi_datatype_basicDatatypes[OMPI_DATATYPE_MPI_MAX
 /*
  * Initialization for these types is deferred until runtime.
  *
- * Using this macro implies that at this point not all informations needed
+ * Using this macro implies that at this point not all information needed
  * to fill up the datatype are known. We fill them with zeros and then later
  * when the datatype engine will be initialized we complete with the
  * correct information. This macro should be used for all composed types.

ompi/debuggers/MPI_Handles_interface.txt

@@ -507,7 +507,7 @@ create their own requests.
   - Whether the underlying object has been "freed" by the MPI
     implementation (i.e., made inactive, and would have actually been
     freed if not running under a debugger)
-  - Peer process(es) invovled with the request (if available)
+  - Peer process(es) involved with the request (if available)
   - If pt2pt, communicator associated with the request
   - If file, file associated with the request
   - If pt2pt or file, whether the data transfer has started yet
@@ -570,7 +570,7 @@ C++: MPI::Status
 
 A user-accessible struct that contains information about a completed
 communication.  The MPI status is a little different from other MPI
-handles in that it is the object itselt; not a handle to an underlying
+handles in that it is the object itself; not a handle to an underlying
 MPI status.  For example, if a point-to-point communication was
 started with a wildcard receive, the status will contain information
 about the peer to whom the communication completed.  There are no

ompi/debuggers/dlopen_test.c

@@ -117,7 +117,7 @@ static int do_test(void)
     /* If we're using libltdl, it will find the .la file and may
        discover that it needs to open the actual file in the .libs
        directory.  If we're not using libltdl, then we won't know
-       about the magic .la file / .libs directory.  Hueristic: if we
+       about the magic .la file / .libs directory.  Heuristic: if we
        get here, manually prefix the filename with .libs/ and try
        again. */
     char *rel_filename;

ompi/debuggers/mpihandles_interface.h

@@ -153,7 +153,7 @@ struct mpidbg_process_t {
     int mpi_comm_world_rank;
 };
 /* ==> JMS Should we just use mqs_process_location instead?  George
-   thinks that this is unncessary -- perhaps due to the fact that we
+   thinks that this is unnecessary -- perhaps due to the fact that we
    could use mqs_process_location...?  Need to get some feedback from
    others on this one.  Need to check Euro PVM/MPI '06 paper... */
 
@@ -503,8 +503,8 @@ extern char mpidbg_dll_is_big_endian;
    the DLL. */
 extern char mpidbg_dll_bitness;
 
-/* Global variable *in the DLL* describing the DLL's capabilties with
-   regards to communicators.  This value is valid after a successfull
+/* Global variable *in the DLL* describing the DLL's capabilities with
+   regards to communicators.  This value is valid after a successful
    call to mpidbg_init_per_process(). */
 extern enum mpidbg_comm_capabilities_t mpidbg_comm_capabilities;
 
@@ -515,12 +515,12 @@ extern enum mpidbg_comm_capabilities_t mpidbg_comm_capabilities;
    be able to map this name to a valid handle.  MPI implementations
    not requiring this mapping can either have a NULL value for this
    variable or have a single entry that has a NULL string value.  This
-   variable is not valid until after a successfull call to
+   variable is not valid until after a successful call to
    mpidbg_init_per_process().  */
 extern struct mpidbg_name_map_t *mpidbg_comm_name_map;
 
-/* Global variable *in the DLL* describing the DLL's capabilties with
-   regards to error handlers.  This value is valid after a successfull
+/* Global variable *in the DLL* describing the DLL's capabilities with
+   regards to error handlers.  This value is valid after a successful
    call to mpidbg_init_per_process(). */
 extern enum mpidbg_errhandler_capabilities_t mpidbg_errhandler_capabilities;
 

ompi/debuggers/ompi_common_dll.c

@@ -505,7 +505,7 @@ int ompi_fill_in_type_info(mqs_image *image, char **message)
         }
     }
 
-    /* All the types are here. Let's succesfully return. */
+    /* All the types are here. Let's successfully return. */
     *message = NULL;
     return mqs_ok;
 

ompi/debuggers/ompi_debuggers.c

@@ -58,7 +58,7 @@
  * information on optimized build, and as a result we are unable to access
  * the fields structure (i.e. to get their displacement). This file is
  * included in the optimized build just to provide us with this missing
- * informations. Therefore, it always have to be compiled with the -g flag,
+ * information. Therefore, it always have to be compiled with the -g flag,
  * otherwise the type information will be missing and the parallel
  * debuggers will be unable to initialize the Open MPI debug library.
  */

ompi/debuggers/ompi_mpihandles_dll.c

@@ -286,7 +286,7 @@ int mpidbg_init_per_image(mqs_image *image, const mqs_image_callbacks *icb,
     handle_types->hi_cxx_win =
         mqs_find_type(image, "MPI::Win", mqs_lang_cplus);
 
-    /* Tell the debuger what capabilities we have */
+    /* Tell the debugger what capabilities we have */
     mpidbg_comm_capabilities =
         MPIDBG_COMM_CAP_BASIC |
         MPIDBG_COMM_CAP_STRING_NAMES |

ompi/debuggers/ompi_msgq_dll.c

@@ -79,7 +79,7 @@
 
 /* Notice to developers!!!!
  * The following include files with _dbg.h suffixes contains definitions
- * that are shared between the debuggger plugins and the OMPI code base.
+ * that are shared between the debugger plugins and the OMPI code base.
  * This is done instead of including the non-_dbg suffixed files because
  * of the different way compilers may handle extern definitions. The
  * particular case that is causing problems is when there is an extern

ompi/dpm/dpm.c

@@ -2065,7 +2065,7 @@ static int start_dvm(char **hostfiles, char **dash_host)
         set_handler_default(SIGCHLD);
 
         /* Unblock all signals, for many of the same reasons that
-         we set the default handlers, above.  This is noticable
+         we set the default handlers, above.  This is noticeable
          on Linux where the event library blocks SIGTERM, but we
          don't want that blocked by the prted (or, more
          specifically, we don't want it to be blocked by the

ompi/dpm/help-dpm.txt

@@ -48,7 +48,7 @@ We are not able to proceed. Please correct your application.
 #
 [execv-error]
 
-The singleton application was not able to find an executabe in your
+The singleton application was not able to find an executable in your
 PATH or in the directory where PRTE was initially installed, and
 therefore cannot continue.
 

ompi/errhandler/errcode.c

@@ -189,7 +189,7 @@ int ompi_mpi_errcode_init (void)
     CONSTRUCT_ERRCODE( ompi_err_conversion, MPI_ERR_CONVERSION, "MPI_ERR_CONVERSION: error in data conversion" );
     CONSTRUCT_ERRCODE( ompi_err_disp, MPI_ERR_DISP, "MPI_ERR_DISP: invalid displacement" );
     CONSTRUCT_ERRCODE( ompi_err_dup_datarep, MPI_ERR_DUP_DATAREP, "MPI_ERR_DUP_DATAREP: error duplicating data representation" );
-    CONSTRUCT_ERRCODE( ompi_err_file_exists, MPI_ERR_FILE_EXISTS, "MPI_ERR_FILE_EXISTS: file exists alreay" );
+    CONSTRUCT_ERRCODE( ompi_err_file_exists, MPI_ERR_FILE_EXISTS, "MPI_ERR_FILE_EXISTS: file exists already" );
     CONSTRUCT_ERRCODE( ompi_err_file_in_use, MPI_ERR_FILE_IN_USE, "MPI_ERR_FILE_IN_USE: file already in use" );
     CONSTRUCT_ERRCODE( ompi_err_file, MPI_ERR_FILE, "MPI_ERR_FILE: invalid file" );
     CONSTRUCT_ERRCODE( ompi_err_info_key, MPI_ERR_INFO_KEY, "MPI_ERR_INFO_KEY: invalid key argument for info object" );

ompi/errhandler/errhandler.c

@@ -420,7 +420,7 @@ int ompi_errhandler_proc_failed_internal(ompi_proc_t* ompi_proc, int status, boo
      * forward through errors in collectives" as this is less intrusive to the
      * code base.) */
     if( forward ) {
-        /* TODO: this to become redundand when pmix has rbcast */
+        /* TODO: this to become redundant when pmix has rbcast */
         ompi_comm_failure_propagate(&ompi_mpi_comm_world.comm, ompi_proc, status);
         /* Let pmix know: flush modex information, propagate to connect/accept
          * jobs; we will tell our local daemon, and it will do the proper thing */

ompi/errhandler/errhandler.h

@@ -121,7 +121,7 @@ struct ompi_errhandler_t {
 typedef struct ompi_errhandler_t ompi_errhandler_t;
 
 /**
- * Padded struct to maintain back compatibiltiy.
+ * Padded struct to maintain back compatibility.
  * See ompi/communicator/communicator.h comments with struct ompi_communicator_t
  * for full explanation why we chose the following padding construct for predefines.
  */

ompi/etc/Makefile.am

@@ -47,7 +47,7 @@ install-data-local:
 
 # Only remove if exactly the same as what in our tree
 # NOTE TO READER: Bourne shell if ... fi evaluates the body if
-#    the return of the evaluted command is 0 (as opposed to non-zero
+#    the return of the evaluated command is 0 (as opposed to non-zero
 #    as used by everyone else)
 uninstall-local:
 	@ p="$(ompi_config_files)"; \

ompi/file/file.c

@@ -69,7 +69,7 @@ OBJ_CLASS_INSTANCE(ompi_file_t,
 
 
 /*
- * Initialize file handling bookeeping
+ * Initialize file handling bookkeeping
  */
 int ompi_file_init(void)
 {