[Mesa-dev] [PATCH] auxilary/os: Add Solaris support in os_get_total_physical_memory.

[Emil Velikov]

On 23/08/14 01:28, Vinson Lee wrote:
> The patch fixes the build on Oracle Solaris.
> 
>   CC     os/os_misc.lo
> "os/os_misc.c", line 59: #error: unexpected platform in os_sysinfo.c
> 
Does mesa even run on Solaris or is this just to silence the buildbot ?

FWIW Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>

> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>  src/gallium/auxiliary/os/os_misc.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/gallium/auxiliary/os/os_misc.c b/src/gallium/auxiliary/os/os_misc.c
> index ef84c79..4c5a22d 100644
> --- a/src/gallium/auxiliary/os/os_misc.c
> +++ b/src/gallium/auxiliary/os/os_misc.c
> @@ -47,7 +47,7 @@
>  #endif
>  
>  
> -#if defined(PIPE_OS_LINUX) || defined(PIPE_OS_CYGWIN)
> +#if defined(PIPE_OS_LINUX) || defined(PIPE_OS_CYGWIN) || defined(PIPE_OS_SOLARIS)
>  #  include <unistd.h>
>  #elif defined(PIPE_OS_APPLE) || defined(PIPE_OS_BSD)
>  #  include <sys/sysctl.h>
> @@ -111,7 +111,7 @@ os_get_option(const char *name)
>  bool
>  os_get_total_physical_memory(uint64_t *size)
>  {
> -#if defined(PIPE_OS_LINUX) || defined(PIPE_OS_CYGWIN)
> +#if defined(PIPE_OS_LINUX) || defined(PIPE_OS_CYGWIN) || defined(PIPE_OS_SOLARIS)
>     const long phys_pages = sysconf(_SC_PHYS_PAGES);
>     const long page_size = sysconf(_SC_PAGE_SIZE);
>  
>