[Intel-gfx] ?==?utf-8?q? [RFC i-g-t 2/9] benchmarks/Makefile: Don't build benchmarks that depend on libdrm_intel.

Emil Velikov emil.velikov at collabora.co.uk
Mon May 23 14:04:16 UTC 2016


On Friday, May 20, 2016 23:59 BST, robert.foss at collabora.com wrote: 
> From: Robert Foss <robert.foss at collabora.com>
> 
> Use the HAS_INTEL automake flag to avoid building benchmarks that won't
> compile unless libdrm_intel is available in the build system.
> 
> Signed-off-by: Robert Foss <robert.foss at collabora.com>
> ---
>  benchmarks/Makefile.sources | 15 ++++++++++-----
>  1 file changed, 10 insertions(+), 5 deletions(-)
> 
> diff --git a/benchmarks/Makefile.sources b/benchmarks/Makefile.sources
> index 81607a5..26ee3ea 100644
> --- a/benchmarks/Makefile.sources
> +++ b/benchmarks/Makefile.sources
> @@ -1,10 +1,6 @@
>  benchmarksdir=$(libexecdir)/intel-gpu-tools/benchmarks
>  
>  benchmarks_PROGRAMS =			\
> -	intel_upload_blit_large         \
> -	intel_upload_blit_large_gtt     \
> -	intel_upload_blit_large_map     \
> -	intel_upload_blit_small		\
>  	gem_blt				\
>  	gem_create			\
>  	gem_exec_ctx			\
> @@ -16,6 +12,15 @@ benchmarks_PROGRAMS =			\
>  	gem_prw				\
>  	gem_set_domain			\
>  	gem_syslatency			\
> -	gem_userptr_benchmark		\
>  	kms_vblank			\
>  	$(NULL)
> +
> +if HAVE_INTEL
> +	benchmarks_PROGRAMS +=			\
> +		intel_upload_blit_large		\
> +		intel_upload_blit_large_gtt	\
> +		intel_upload_blit_large_map	\
> +		intel_upload_blit_small		\
> +		gem_userptr_benchmark		\
> +		$(NULL)
> +endif
Some suggestions:
 - Please don't use conditionals in the Makefile.sources - you'll break the other build (Android)
 - instead of ^^ use separate variable and combine them in the Makefile.am/Android.mk
 - Don't double-indent, don't think any other place does so.

Thanks
Emil



More information about the Intel-gfx mailing list