diff options
| author | Richard J. Knight <rjknight@us.ibm.com> | 2018-03-08 09:16:22 -0600 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-03-12 14:23:33 -0400 |
| commit | 967e9a084bbe2c714bdb7b436e5ca265bd4d5985 (patch) | |
| tree | dc3dd719be4776267586da672049c2ded5ec65f8 /src/usr/runtime | |
| parent | 24929fd8ab969f22986c132f83238d959c6ce642 (diff) | |
| download | talos-hostboot-967e9a084bbe2c714bdb7b436e5ca265bd4d5985.tar.gz talos-hostboot-967e9a084bbe2c714bdb7b436e5ca265bd4d5985.zip | |
Wait for responses from all nodes for IPC_POPULATE_ATTRIBUTES msg
-In the message handling for IPC_POPULATE_ATTRIBUTES the master
node should wait for a response from all messages, the current
code only waits for a single response. The change in this
commit adds a loop to capture a response from each message sent.
Change-Id: Iec3d3f666a3f00e01b850a822674d3f479f8bc89
RTC:189354
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/55249
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Reviewed-by: Richard Ward <rward15@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/runtime')
| -rw-r--r-- | src/usr/runtime/populate_hbruntime.C | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/src/usr/runtime/populate_hbruntime.C b/src/usr/runtime/populate_hbruntime.C index d222edc4d..3068fbd02 100644 --- a/src/usr/runtime/populate_hbruntime.C +++ b/src/usr/runtime/populate_hbruntime.C @@ -3000,12 +3000,16 @@ errlHndl_t populate_hbRuntimeData( void ) // wait for a response to each message we sent if( l_elog == nullptr ) { - msg_t * response = msg_wait(msgQ); - TRACFCOMP(g_trac_runtime, - "IPC_POPULATE_ATTRIBUTES : drawer %d completed", - response->data[0]); - msg_free(response); - --msg_count; + //$TODO RTC:189356 - need timeout here + while(msg_count) + { + msg_t * response = msg_wait(msgQ); + TRACFCOMP(g_trac_runtime, + "IPC_POPULATE_ATTRIBUTES : drawer %d completed", + response->data[0]); + msg_free(response); + --msg_count; + } } MBOX::msgq_unregister(MBOX::HB_POP_ATTR_MSGQ); |
