[PATCH:intel-gpu-tools 1/7] Fix #ifdef check for _SC_AVPHYS_PAGES in intel_get_avail_ram_mb()

Daniel Vetter daniel at ffwll.ch
Mon Jan 5 08:11:25 PST 2015


On Tue, Dec 23, 2014 at 07:07:08PM -0800, Alan Coopersmith wrote:
> Check for the sysconf value used here, not the one used in the
> previous function.
>
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

Thanks for the patches, all merged. Aside: Do you really run all the
testcases on solaris or wouldn't it be better to just disable them?
And please cc intel-gfx for igt patches and Thomas Wood (who's doing
maintainer duties for it now) in the future.

Thanks, Daniel

> ---
>  lib/intel_os.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/intel_os.c b/lib/intel_os.c
> index db7889b..1badd3e 100644
> --- a/lib/intel_os.c
> +++ b/lib/intel_os.c
> @@ -112,7 +112,7 @@ intel_get_avail_ram_mb(void)
>
>   retval = sysinf.freeram;
>   retval *= sysinf.mem_unit;
> -#elif defined(_SC_PAGESIZE) && defined(_SC_PHYS_PAGES) /* Solaris */
> +#elif defined(_SC_PAGESIZE) && defined(_SC_AVPHYS_PAGES) /* Solaris */
>   long pagesize, npages;
>
>   pagesize = sysconf(_SC_PAGESIZE);
> --
> 1.7.9.2
>

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch


More information about the xorg-devel mailing list