- 15 Sep, 2017 40 commits
-
-
Alejandro Sanchez authored
_parse_resv_core_cnt has been moved to src/common/state_control.[ch]. Furthermore, it has been split in two functions: - _parse_resv_core_cnt - _is_corecnt_supported Bug 2329
-
Alejandro Sanchez authored
Bug 2329
-
Alejandro Sanchez authored
Bug was introduced in 22c9e25c. Stack trace: Thread 1 (Thread 0x7ff64228c300 (LWP 6652)): ../sysdeps/unix/sysv/linux/raise.c:58 fmt=fmt@entry=0x7ff6409bb000 "*** Error in `%s': %s: 0x%s ***\n") at ../sysdeps/posix/libc_fatal.c:175 ptr=<optimized out>, str=0x7ff6409bb110 "double free or corruption (out)", action=3) at malloc.c:5048 have_lock=<optimized out>) at malloc.c:3904 malloc.c:2984 new_text=0x561a4f8b4d10 "asfasdf", column=21) at ../../../../slurm/src/sview/resv_info.c:331 event=0x561a4f8a8f10, resv_msg=0x561a4f8b1410) at ../../../../slurm/src/sview/resv_info.c:515 Bug 2329
-
Alejandro Sanchez authored
resv_msg_ptr->node_cnt memory needed to be freed before returning, since it was previously allocated by xrealloc().
-
Alejandro Sanchez authored
The scontrol and sview clients print errors in different ways (i.e. error() vs global_edit_error_msg respectively). By adding this err_msg parameter to the common functions we decouple the message itself from the way we print it. Bug 2329
-
Alejandro Sanchez authored
And make it so src/scontrol/create_res.c makes use of it. Also set exit_code = 1 when there's an error in the calling function instead of the function itself. Bug 2329
-
Alejandro Sanchez authored
-
Alejandro Sanchez authored
Different clients provide similar functionality such as scontrol and sview. These new files are intended to hold common functions that can be used by these clients. This will help to avoid behavior divergence across them and stop duplicating code. src/common/Makefile.am libcommon_la_SOURCES have been modified accordingly, and a following autogen.sh execution should be committed right after to reflect the changes in the generated src/common/Makefile.in file. Bug 2329
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Letting the thread_id go out of context means that _safe_signal_while_allocating should be detached or we leak the return value.
-
Tim Wickberg authored
Since the thread_id is never referenced elsewhere, don't bother storing it in the struct.
-
Tim Wickberg authored
Letting the thread_id go out of scope leaks the thread's return value. So this should be a detached thread instead.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Remove references to deprecated/obsolete/removed MPI stacks, and generally tidy things up.
-
Tim Wickberg authored
-
Tim Wickberg authored
Remove plugins for: - lam - mpich1_p4 - mpich1_shmem - mvapich All modern MPI stacks should be talking PMI2 / PMIx, these are all obsolete and should have been retired long ago.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Rather than creating a detached thread and using pthread_kill to stop it (which could lead to data loss), make an attached thread, signal it as appropriate then wait on proper shutdown with pthread_join.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
While here change signatures of notice_thread_init() and notice_thread_fini() to void, as they aren't checked anywhere anyways.
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
-
Tim Wickberg authored
Also fix a wrong type use in sizeof calculation fed to xmalloc.
-
Tim Wickberg authored
Also fix a wrong type use in sizeof calculation fed to xmalloc.
-