memory: fix typo #1106
memory: fix typo #1106
Conversation
This is a 2.x specific change. No master equivalent. Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
|
One more commit coming. Apparently MEMORY_LINUX_PTMALLOC2 is being defined to 1 even though memory/linux disabled itself. Tracking down now. |
|
Test PASSed. |
|
Found the other issue. The MCA build priorities for the memory framework. Should probably be removed on master but I have a tweak that should help here. |
This commit ensures that is --with-memory-manager=linux is specified then the linux memory component is actually enabled. The lower priority was preventing this. Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
|
@hppritcha Also updating the LANL platform files to not force ptmalloc2. |
|
@jsquyres Ready for review. |
|
The gap between 40 and 41 is a little small. 😄 But good enough for now. 👍 |
|
@hppritcha Good to go. |
|
Test PASSed. |
|
On master I will remove the memory component priorities. They were only there because we could have only one memory component before. Now we can have multiple. |
|
I'm going to merge these so that they can get into tonight's snapshot. |
4 participants
This is a 2.x specific change. No master equivalent.
Signed-off-by: Nathan Hjelm hjelmn@lanl.gov
PEBKAC. Wasn't caught by MTT or CI.
:bot🏷️bug
:bot:milestone:v2.0.0
:bot:assign: @jsquyres