| commit | 89d6ab0de8d5b1d959667b8c7a07239c6936efe7 | |
| author | Sam White <white67@illinois.edu> | |
| Thu, 12 Jan 2017 14:59:07 +0000 (12 08:59 -0600) | ||
| committer | Sam White <white67@illinois.edu> | |
| Fri, 20 Jan 2017 21:02:35 +0000 (20 15:02 -0600) | ||
| tree | 4095b2282e35357c1ca1c8b477860c3ac8de222d | treesnapshot (tar.gz zip) |
| parent | c078af96eeeb3210740b6fd2d93466487d70f187 | commitdiff |
AMPI: fix leak by not inserting send requests into posted request queue
Send requests are matched by their request number, not the full
(tag,src,comm) tuple, so they were never being looked up in and deleted
from the posted request queue.
This also improves message matching performance, since the queue no
longer grows indefinitely long over time. AMPI used to use NULL requests
for sends, so this wasn't a problem before change #1811.
Change-Id: I60f016655e9a85d8758c5ccc543b15d4bb7ce8f5
Send requests are matched by their request number, not the full
(tag,src,comm) tuple, so they were never being looked up in and deleted
from the posted request queue.
This also improves message matching performance, since the queue no
longer grows indefinitely long over time. AMPI used to use NULL requests
for sends, so this wasn't a problem before change #1811.
Change-Id: I60f016655e9a85d8758c5ccc543b15d4bb7ce8f5
| src/libs/ck-libs/ampi/ampi.C | diffblobblamehistory | |
| src/libs/ck-libs/ampi/ampiimpl.h | diffblobblamehistory |