On 23 May 2016 at 17:04, Christophe Milard <christophe.mil...@linaro.org>
wrote:
> Signed-off-by: Christophe Milard <christophe.mil...@linaro.org>
> ---
> helper/include/odp/helper/linux.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/helper/include/odp/helper/linux.h
> b/helper/include/odp/helper/linux.h
> index 01c348d..2e89833 100644
> --- a/helper/include/odp/helper/linux.h
> +++ b/helper/include/odp/helper/linux.h
> @@ -73,13 +73,13 @@ typedef struct {
>
> /** The odpthread starting arguments, used both in process or thread mode
> */
> typedef struct {
> - odph_odpthread_linuxtype_t linuxtype;
> - odph_odpthread_params_t thr_params; /*copy of thread start
> parameter*/
> + odph_odpthread_linuxtype_t linuxtype; /**< process or pthread */
>
ODP threads might not be pthreads. There are many implementations of
threads.
Aren't we trying to tell the application if we are using a single-process
(memory space) or multi-process model?
Let's report this and only this.
+ odph_odpthread_params_t thr_params; /**< odpthread start parameters
> */
> } odph_odpthread_start_args_t;
>
> /** Linux odpthread state information, used both in process or thread
> mode */
> typedef struct {
> - odph_odpthread_start_args_t start_args;
> + odph_odpthread_start_args_t start_args; /**< start arguments */
> int cpu; /**< CPU ID */
> int last; /**< true if last table
> entry */
> union {
> --
> 2.5.0
>
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> https://lists.linaro.org/mailman/listinfo/lng-odp
>
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
