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

[Vinson Lee]

On Fri, Aug 22, 2014 at 5:44 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> 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>
>

This is just to fix a build. I do not know if it runs or not. It
probably doesn't.

>> 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);
>>
>>
>