Re: [OMPI devel] Warnings in v1.7.4: rcache

2013-10-23 Thread Rolf vandeVaart
Yes, that is from one of my CMRs.  I always configure with -enable-picky  but 
that did not pick up this warning.
I will fix this in the trunk in the morning (watching the Red Sox right now :)) 
and then file CMR to bring over.
Rolf

From: devel [mailto:devel-boun...@open-mpi.org] On Behalf Of Ralph Castain
Sent: Wednesday, October 23, 2013 7:19 PM
To: Open MPI Developers
Subject: [OMPI devel] Warnings in v1.7.4: rcache

One of the recent CMRs has created new warnings in v1.7.4:

rcache_vma.c: In function 'mca_rcache_vma_find':
rcache_vma.c:58:23: warning: pointer of type 'void *' used in arithmetic 
[-Wpedantic]
 bound_addr = addr + size - 1;
   ^
rcache_vma.c:58:30: warning: pointer of type 'void *' used in arithmetic 
[-Wpedantic]
 bound_addr = addr + size - 1;
  ^
rcache_vma.c: In function 'mca_rcache_vma_find_all':
rcache_vma.c:84:23: warning: pointer of type 'void *' used in arithmetic 
[-Wpedantic]
 bound_addr = addr + size - 1;
   ^
rcache_vma.c:84:30: warning: pointer of type 'void *' used in arithmetic 
[-Wpedantic]
 bound_addr = addr + size - 1;
  ^

Does someone know where these came from, and how to correct them?
Ralph

---
This email message is for the sole use of the intended recipient(s) and may 
contain
confidential information.  Any unauthorized review, use, disclosure or 
distribution
is prohibited.  If you are not the intended recipient, please contact the 
sender by
reply email and destroy all copies of the original message.
---


[OMPI devel] Warnings in v1.7.4: rcache

2013-10-23 Thread Ralph Castain
One of the recent CMRs has created new warnings in v1.7.4:

rcache_vma.c: In function 'mca_rcache_vma_find':
rcache_vma.c:58:23: warning: pointer of type 'void *' used in arithmetic
[-Wpedantic]
 bound_addr = addr + size - 1;
   ^
rcache_vma.c:58:30: warning: pointer of type 'void *' used in arithmetic
[-Wpedantic]
 bound_addr = addr + size - 1;
  ^
rcache_vma.c: In function 'mca_rcache_vma_find_all':
rcache_vma.c:84:23: warning: pointer of type 'void *' used in arithmetic
[-Wpedantic]
 bound_addr = addr + size - 1;
   ^
rcache_vma.c:84:30: warning: pointer of type 'void *' used in arithmetic
[-Wpedantic]
 bound_addr = addr + size - 1;
  ^


Does someone know where these came from, and how to correct them?
Ralph


Re: [OMPI devel] http://newscenter.lbl.gov/wp-content/uploads/2008/07/yelick-berkeleyview-july081.pdf

2013-10-23 Thread Larry Baker
Ralph, et al.,

Kathy Yelick is featured in this month's ACM Member News sidebar (p. 17).  It 
also turns out she will be giving a lecture at SC13.

http://sc13.supercomputing.org/content/sc13-feature-acm-athena-lecturer-katherine-yelick

Check it out.

Larry Baker
US Geological Survey
650-329-5608
ba...@usgs.gov



On 6 Jun 2013, at 11:07 AM, Larry Baker wrote:

> Ralph,
> 
> I mentioned at dinner how much I like listening to UCB Prof. Kathy Yelick's 
> lectures.  Here's some slides I found of hers.  She is also associate lab 
> director of NERSC at LBNL.
> 
> http://newscenter.lbl.gov/wp-content/uploads/2008/07/yelick-berkeleyview-july081.pdf
> 
> http://www.lanl.gov/orgs/hpc/salishan/pdfs/Salishan%20slides/Yelick.pdf
> 
> http://www.hpcsw.org/presentations/wed/yelick.pdf
> 
> Here's the UCB web page about their view of HPC/Parallel computing.
> 
> http://view.eecs.berkeley.edu/wiki/Main_Page
> 
> I think the one-way (DMA) messaging model is very important, especially when 
> it can be hidden and automatically (and correctly) done by the compiler 
> (requires new languages?).
> 
> Any time you get a chance to hear her (or invite her to speak) you will enjoy 
> it.  The work they are doing there will inform you to benefit your own work.
> 
> Larry Baker
> US Geological Survey
> 650-329-5608
> ba...@usgs.gov
> 
> 
> 



Re: [OMPI devel] [OMPI svn] svn:open-mpi r29489 - trunk/orte/mca/plm/base

2013-10-23 Thread Ralph Castain
I reverted this changeset from the trunk as it incorrectly re-added the
local coprocessor data to the HNP's hash table. This is already being done
in the ess/hnp module, and there is no value in duplicating it again every
time a daemon calls back.

As noted in the revert comment, if we want host daemons to retain their
coprocessor info in a hash table, then we need to do that somewhere else,
not where this was done. At this time, I don't see the daemons using that
info anywhere.



On Wed, Oct 23, 2013 at 8:56 AM,  wrote:

> Author: hjelmn (Nathan Hjelm)
> Date: 2013-10-23 11:56:23 EDT (Wed, 23 Oct 2013)
> New Revision: 29489
> URL: https://svn.open-mpi.org/trac/ompi/changeset/29489
>
> Log:
> Fix coprocessor detection by always adding the local daemon's co-processors
> to the hash table.
>
> Tested and working on a system with 2 Xeon Phi co-processors.
>
> cmr=v1.7.4:ticket=3847:reviewer=ompi-rm1.7
>
> Text files modified:
>trunk/orte/mca/plm/base/plm_base_launch_support.c |40
> +---
>1 files changed, 37 insertions(+), 3 deletions(-)
>
> Modified: trunk/orte/mca/plm/base/plm_base_launch_support.c
>
> ==
> --- trunk/orte/mca/plm/base/plm_base_launch_support.c   Wed Oct 23
> 11:52:05 2013(r29488)
> +++ trunk/orte/mca/plm/base/plm_base_launch_support.c   2013-10-23
> 11:56:23 EDT (Wed, 23 Oct 2013)  (r29489)
> @@ -1,3 +1,4 @@
> +/* -*- Mode: C; c-basic-offset:4 ; indent-tabs-mode:nil -*- */
>  /*
>   * Copyright (c) 2004-2010 The Trustees of Indiana University and Indiana
>   * University Research and Technology
> @@ -12,7 +13,8 @@
>   * Copyright (c) 2007-2011 Cisco Systems, Inc.  All rights reserved.
>   * Copyright (c) 2009  Institut National de Recherche en Informatique
>   * et Automatique. All rights reserved.
> - * Copyright (c) 2011-2012 Los Alamos National Security, LLC.
> + * Copyright (c) 2011-2013 Los Alamos National Security, LLC. All rights
> + * reserved.
>   * Copyright (c) 2013  Intel, Inc. All rights reserved.
>   * $COPYRIGHT$
>   *
> @@ -677,6 +679,38 @@
>  jdatorted = orte_get_job_data_object(ORTE_PROC_MY_NAME->jobid);
>  }
>
> +#if OPAL_HAVE_HWLOC
> +{
> +char *coprocessors, **sns;
> +
> +/* detect and add any of my coprocessors to the hash table */
> +coprocessors =
> opal_hwloc_base_find_coprocessors(opal_hwloc_topology);
> +
> +if (NULL != coprocessors) {
> +/* init the hash table, if necessary */
> +if (NULL == orte_coprocessors) {
> +orte_coprocessors = OBJ_NEW(opal_hash_table_t);
> +opal_hash_table_init(orte_coprocessors,
> orte_process_info.num_procs);
> +}
> +/* separate the serial numbers of the coprocessors
> + * on this host
> + */
> +sns = opal_argv_split(coprocessors, ',');
> +for (int idx = 0 ; NULL != sns[idx] ; ++idx) {
> +uint32_t h;
> +
> +/* compute the hash */
> +OPAL_HASH_STR(sns[idx], h);
> +/* mark that this coprocessor is hosted by this daemon */
> +opal_hash_table_set_value_uint32(orte_coprocessors, h,
> (void*)&ORTE_PROC_MY_NAME->vpid);
> +}
> +opal_argv_free(sns);
> +free(coprocessors);
> +orte_coprocessors_detected = true;
> +}
> +}
> +#endif
> +
>  /* multiple daemons could be in this buffer, so unpack until we
> exhaust the data */
>  idx = 1;
>  while (OPAL_SUCCESS == (rc = opal_dss.unpack(buffer, &dname, &idx,
> ORTE_NAME))) {
> @@ -1271,7 +1305,7 @@
>  /* check for duplicate */
>  ignore = false;
>  for (j=0; j < *argc; j++) {
> - if (0 == strcmp((*argv)[j], orted_cmd_line[i+1])) {
> +  if (0 == strcmp((*argv)[j], orted_cmd_line[i+1])) {
>  ignore = true;
>  break;
>  }
> @@ -1589,7 +1623,7 @@
>  OBJ_DESTRUCT(&nodes);
>  /* mark that the daemons have reported so we can proceed */
>  daemons->state = ORTE_JOB_STATE_DAEMONS_REPORTED;
> -   daemons->updated = false;
> +daemons->updated = false;
>  return ORTE_SUCCESS;
>  }
>
> ___
> svn mailing list
> s...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/svn
>


[OMPI devel] .mailmap added for github mirror

2013-10-23 Thread Dave Goodell (dgoodell)
Based on discussion with Mellanox around the recent GitHub mirror updates, I've 
added a .mailmap file in r29494.  It helps address two goals:

1) To be able to fix misspelled names without rewinding the Git history.

2) To be able to add email addresses incrementally without rewinding Git 
history.  Mellanox would like email addresses so that they can automatically 
send build failure emails from their Jenkins continuous integration server.

The initial version only contains email addresses for Cisco contributors.  I've 
put commented-out templates for all other contributors listed in the AUTHORS 
file.  If you are a contributor and you would like to opt-in to having your 
email address listed in this format, please follow the editing instructions 
listed in that file on the trunk.

At some point I'll CMR the latest state of the file to the v1.7 branch as well.

For more information about git's usage of the .mailmap file, see the man page 
for git-shortlog: 
https://www.kernel.org/pub/software/scm/git/docs/git-shortlog.html

-Dave



Re: [OMPI devel] Open MPI shirts and more

2013-10-23 Thread Kenneth A. Lloyd
+1 - and my family has been notified for the holidays.

-Original Message-
From: devel [mailto:devel-boun...@open-mpi.org] On Behalf Of Jeff Squyres
(jsquyres)
Sent: Friday, October 18, 2013 1:42 PM
To: Open MPI Developers List
Subject: [OMPI devel] Open MPI shirts and more

OMPI Developer Community --

Per the upcoming 10th anniversary of OMPI's SVN r1 (and our discussion on
the call this past Tuesday), Ralph and I setup a Cafe Press story with a
pre-selected bundle of Open MPI-branded stuff (mostly shirts, but some cups
and other stuff, too):

http://www.cafepress.com/openmpi

We set the markup to $0.00 on all the items, so hypothetically we won't earn
any money from this.  If we do, we'll pick some charity and donate
everything (maybe the EFF?).

What does everyone think?  If the community likes it, we'll publish this on
the user list and a few other places.

-- 
Jeff Squyres
jsquy...@cisco.com
For corporate legal information go to:
http://www.cisco.com/web/about/doing_business/legal/cri/

___
devel mailing list
de...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/devel


-
No virus found in this message.
Checked by AVG - www.avg.com
Version: 2014.0.4158 / Virus Database: 3609/6739 - Release Date: 10/10/13