[Piglit] [PATCH] piglit framework: add gbm support

Chad Versace chad.versace at linux.intel.com
Thu Oct 4 10:05:15 PDT 2012


On 10/01/2012 03:47 PM, Jordan Justen wrote:
> Chad,
> 
> Does this look close to the correct way to add GBM support
> following your recent piglit framework refactor series?
> 
> -Jordan
> 
> --
> 
> Use gbm for testing if the PIGLIT_PLATFORM environment variable
> is set to gbm.

This patch looks correct to me. Singe piglit_gbm_framework doesn't use include
any gbm headers or link to libgbm, then it makes sense to enable PIGLIT_HAS_GBM
unconditionally on Linux for now.

Reviewed-by: Chad Versace <chad.versace at linux.intel.com>



More information about the Piglit mailing list