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

[randyf at sibernet.com]

On Mon, 5 Jan 2015, Alan Coopersmith wrote:

> On 01/ 5/15 08:11 AM, Daniel Vetter wrote:
>> 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 - and I'm not actually sure which of the tests our intel driver
> porting team run on Solaris, I was just trying to reduce the number of
> bits that failed to build altogether.  (Though I'm sure some that build
> will fail at runtime since they won't be able to open files in debugfs
> on Solaris.)
>


   We don't run any of the tests in intel-gpu-tools due to various 
incompatibilities, and currently opt for a small set of "sanity" tests 
that are specific to Solaris.

   There was an intent to investigate these tests more closely once we
had a stable environment, but haven't yet undertaken that task (mostly
due to the limited resources available to do the work).  A couple of
the tools themselves have been quite useful, though.


         ---- Randy