diff options
author | Mark Wenning <wenning@us.ibm.com> | 2011-07-11 15:43:13 -0500 |
---|---|---|
committer | Mark W. Wenning <wenning@us.ibm.com> | 2011-07-25 16:50:09 -0500 |
commit | 9c0e69b8cdf3aabcd77c119c3b3425fda66706da (patch) | |
tree | 77a5b7f1c4f58da3a74048e54575a5b9a7785f34 /src/include/usr/initservice/taskargs.H | |
parent | b679a1729d7aea0870544e886ddb4b03e1ecf4c4 (diff) | |
download | talos-hostboot-9c0e69b8cdf3aabcd77c119c3b3425fda66706da.tar.gz talos-hostboot-9c0e69b8cdf3aabcd77c119c3b3425fda66706da.zip |
Cleanup from previous reviews, add barrier for CxxTest
- add IStep1 for demo
- cleanup: remove inheritance, add errlog tags, etc.
- inadvertently ran copyright script, this is OK.
- more cleanup: indentation, etc.
- add barrier for CxxTests
- rename cxxtest_stub.C to cxxtest_data.C
- merge problems on gerrit
- fix review comments
Change-Id: Iee988b8595d8c6dd521fd94d4d58cb403898d948
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/207
Tested-by: Jenkins Server
Reviewed-by: MIKE J. JONES <mjjones@us.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Diffstat (limited to 'src/include/usr/initservice/taskargs.H')
-rw-r--r-- | src/include/usr/initservice/taskargs.H | 66 |
1 files changed, 61 insertions, 5 deletions
diff --git a/src/include/usr/initservice/taskargs.H b/src/include/usr/initservice/taskargs.H index 17e5cbce3..ea12f1800 100644 --- a/src/include/usr/initservice/taskargs.H +++ b/src/include/usr/initservice/taskargs.H @@ -1,3 +1,20 @@ +/**************************************************************************** + * $IBMCopyrightBlock: + * + * IBM Confidential + * + * Licensed Internal Code Source Materials + * + * IBM HostBoot Licensed Internal Code + * + * (C) Copyright IBM Corp. 2011 + * + * The source code for this program is not published or other- + * wise divested of its trade secrets, irrespective of what has + * been deposited with the U.S. Copyright Office. + * $ +****************************************************************************/ + #ifndef __TASKARGS_TASKARGS_H #define __TASKARGS_TASKARGS_H @@ -14,6 +31,9 @@ #include <sys/sync.h> #include <trace/interface.H> +#include <errl/errlentry.H> +#include <errl/errlmanager.H> + /** * @note macro to set up a _start() task entry routine. */ @@ -63,6 +83,7 @@ class TaskArgs public: + /** * @brief TaskArgs constructor * @@ -75,6 +96,7 @@ public: */ ~TaskArgs(); + /** * @brief waitParentSync() * @@ -107,7 +129,7 @@ public: * @param[in] i_returncode; * */ - void postReturnCode( const uint64_t &i_returncode ); + void postReturnCode( const uint64_t i_returncode ); /** @@ -128,7 +150,7 @@ public: * * @param[in] i_command; */ - void setCommand( const uint64_t &i_command ); + void setCommand( const uint64_t i_command ); /** @@ -144,6 +166,39 @@ public: uint64_t getCommand( ) const; + /** + * @brief postErrorLog + * + * Child task can use this to post an errorlog to InitServices + * Caller is responsible for creating a new errorlog, etc. + * No checking is done. + * + * @param[in] i_errl; + * + */ + void postErrorLog( errlHndl_t i_errl ); + + + /** + * @brief getErrorLog + * + * Parent task can use this to get an errorlog from the child + * + * @return iv_errl + * + */ + errlHndl_t getErrorLog( ); + + + /** + * @brief clear taskargs struct between uses + * - barrier is left alone, + * - if iv_errl is non zero, we commit it here just to avoid + * a memory leak. + */ + void clear(); + + private: /** @@ -153,14 +208,15 @@ private: TaskArgs& operator=(const TaskArgs& i_right); - barrier_t iv_sync_barrier; - + errlHndl_t iv_errl; uint64_t iv_taskreturncode; uint64_t iv_taskcommand; + barrier_t iv_sync_barrier; + }; -}; // namespace TASKARGS +}; // namespace #endif |