Skip to content

Topic/thread framework2

Created by: shintaro-iwasaki

This PR has the following structure. Please don't merge it.

Could you please locally manipulate commits so that the first five commits become a single commit?

If so, the first collapsed commit has minimum changes, making it easier to review this PR. Specifically, the reviewer comments that modification in threads_pthreads_wait_sync.c. cannot be justified, so creating a separate clean-up commit (27d6dd378d8861d59af362bf2ac89fc8018976f6) would be helpful.

# The following addresses "3. REQUEST_PENDING/COMPLETE". We might need to rename them.
commit f6db5f06b5d9f136558bd5df81c34b5c4ca5d1eb
    ompi/request: move REQUEST constants from mca/threads to ompi/request

# These two commits address "1. TSD fix"
commit bc31877d24e670696feb9b83d0c50fe208d295f5
    mca/threads: efficiently reallocate opal_tsd_key_values array
commit 50146207cfd64535251b704e50ea764e126f870b
    mca/threads: fix tsd management

# These two commits address "2. Modification in threads_pthreads_wait_sync.c."
commit 27d6dd378d8861d59af362bf2ac89fc8018976f6
    mca/threads: code cleanup
commit f7d5324bbc64463a1a7e2a336bcaa60a96635df9
    mca/threads: fix error handling

# The following five commits should be collapsed into a single commit.
commit 0b420c3a6a3bd269d85928311cf00959b9a8c328
    Revert clean-up changes
# This fixes tests and addresses "4. Use MCA/threads in the tests"
commit e29f3148f19337bda99e14a9b353e86f1f140d84
    test/class: fix opal_fifo and opal_lifo
commit b39d27fbfba7fd00014e5c5977d840c14c849a7e
    Merge pull request #15 from shintaro-iwasaki/topic/thread_framework2
commit d8f1bb93429311b7a44038fd3c01c089140dd35b
    Fix threads framework
commit 205ee1664fcab2ea8c9e0db98e335f5a5b2e3c8c
    Add threads framework

Merge request reports