Both patches:

Reviewed-by: Petri Savolainen <[email protected]>


> -----Original Message-----
> From: EXT Anders Roxell [mailto:[email protected]]
> Sent: Wednesday, October 28, 2015 4:33 PM
> To: Orpana, Pasi (Nokia - FI/Espoo); Wallen, Carl (Nokia - FI/Espoo)
> Cc: [email protected]; Savolainen, Petri (Nokia - FI/Espoo); Anders
> Roxell
> Subject: [PATCH 1/2] api: cpumask: don't use platform internal defines
> 
> Applications that use an installed ODP will get the following
> compile error:
> .../odp_install/include/odp/plat/cpumask_types.h:26:33: fatal error:
> odp_config_internal.h: No such file or directory
>  #include <odp_config_internal.h>
> 
> Defines a new public API macro ODP_CPUMASK_SIZE that is the maximum
> number of CPUs that can be accessed in this system.
> 
> Fixes bug: https://bugs.linaro.org/show_bug.cgi?id=1864
> 
> Signed-off-by: Anders Roxell <[email protected]>
> ---
>  include/odp/api/cpumask.h                               | 6 ++++++
>  platform/linux-generic/include/odp/plat/cpumask_types.h | 5 +++--
>  2 files changed, 9 insertions(+), 2 deletions(-)
> 
> diff --git a/include/odp/api/cpumask.h b/include/odp/api/cpumask.h
> index 4835a6c..7480132 100644
> --- a/include/odp/api/cpumask.h
> +++ b/include/odp/api/cpumask.h
> @@ -26,6 +26,12 @@ extern "C" {
>   */
> 
>  /**
> + * @def ODP_CPUMASK_SIZE
> + * Maximum cpumask size, this definition limits the number of individual
> CPUs
> + * that can be accessed in this system.
> + */
> +
> +/**
>   * @def ODP_CPUMASK_STR_SIZE
>   * Minimum size of output buffer for odp_cpumask_to_str()
>   */
> diff --git a/platform/linux-generic/include/odp/plat/cpumask_types.h
> b/platform/linux-generic/include/odp/plat/cpumask_types.h
> index affe59b..1beaa1d 100644
> --- a/platform/linux-generic/include/odp/plat/cpumask_types.h
> +++ b/platform/linux-generic/include/odp/plat/cpumask_types.h
> @@ -23,12 +23,13 @@ extern "C" {
>   */
> 
>  #include <odp/std_types.h>
> -#include <odp_config_internal.h>
> +
> +#define ODP_CPUMASK_SIZE 1024
> 
>  /**
>   * Minimum size of output buffer for odp_cpumask_to_str()
>   */
> -#define ODP_CPUMASK_STR_SIZE ((_ODP_INTERNAL_MAX_THREADS + 3) / 4 + 3)
> +#define ODP_CPUMASK_STR_SIZE ((ODP_CPUMASK_SIZE + 3) / 4 + 3)
> 
>  /**
>   * CPU mask
> --
> 2.1.4

_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to