Open MPI logo

Hardware Locality Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Hardware Locality Development mailing list

Subject: Re: [hwloc-devel] want 1.0rc4?
From: Jeff Squyres (jsquyres_at_[hidden])
Date: 2010-05-03 22:28:23


Awesome; thanks Chris!

On May 3, 2010, at 8:57 PM, Christopher Samuel wrote:

> On 03/05/10 09:57, Jeff Squyres wrote:
>
> > 1.0rc4 is up.
>
> Running coccicheck on 1.0rc4 flags up this construct, I presume
> as an ambiguous construction:
>
> if (!topology->flags & HWLOC_TOPOLOGY_FLAG_WHOLE_SYSTEM) {
>
> That's at line 1518 of src/topology.c in hwloc_discover(). The
> check file simply says:
>
> // !x&y combines boolean negation with bitwise and
>
>
> It's also flagged some unchecked malloc()'s in various bits:
>
>
> line 41 of src/misc.c in hwloc_snprintf():
>
> str = malloc(size);
>
>
> line 321 of src/topology-linux.c in hwloc_linux_get_proc_tids():
>
> tids = malloc(max_tids*sizeof(pid_t));
>
>
> line 328 of src/topology-linux.c in hwloc_linux_get_proc_tids():
>
> tids = realloc(tids, max_tids*sizeof(pid_t));
>
>
> line 1561 of src/topology.c in hwloc_discover():
>
> objs = malloc(n_objs * sizeof(objs[0]));
>
>
> Hope these are helpful!
> Chris
> --
> Christopher Samuel - Senior Systems Administrator
> VLSCI - Victorian Life Sciences Computational Initiative
> Email: samuel_at_[hidden] Phone: +61 (0)3 903 55545
> http://www.vlsci.unimelb.edu.au/
>
> _______________________________________________
> hwloc-devel mailing list
> hwloc-devel_at_[hidden]
> http://www.open-mpi.org/mailman/listinfo.cgi/hwloc-devel

-- 
Jeff Squyres
jsquyres_at_[hidden]
For corporate legal information go to:
http://www.cisco.com/web/about/doing_business/legal/cri/