[Mesa-dev] [PATCH v4 4/4] meson: build "radv" vulkan driver for radeon hardware

Bas Nieuwenhuizen bas at basnieuwenhuizen.nl
Wed Sep 27 04:25:53 UTC 2017


It works now, thanks!

Also

Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>

for this patch.

On Tue, Sep 26, 2017 at 4:38 PM, Dylan Baker <dylan at pnwbakers.com> wrote:
> This builds, installs, and has been tested on a r290x (Hawaii) with the Vulkan
> CTS. It dies horribly in a fire at the same point for the meson build as the
> autotools build.
>
> v2: - enable radv by default
>     - add shader cache support and enforce that it's built for radv
> v3: - Fix typo in meson_options (Nicholas)
>     - strip trailing 'svn' from llvm version before setting the version
>       preprocessor flag (Bas)
>     - Check for LLVM module requirements
>
> Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
> ---
>  meson.build                       |  39 +++++++++++-
>  meson_options.txt                 |   4 +-
>  src/{ => amd/addrlib}/meson.build |  69 ++++++++++++---------
>  src/{ => amd/common}/meson.build  |  65 ++++++++++++--------
>  src/{ => amd}/meson.build         |  33 ++--------
>  src/amd/vulkan/meson.build        | 124 ++++++++++++++++++++++++++++++++++++++
>  src/meson.build                   |   2 +-
>  7 files changed, 252 insertions(+), 84 deletions(-)
>  copy src/{ => amd/addrlib}/meson.build (51%)
>  copy src/{ => amd/common}/meson.build (51%)
>  copy src/{ => amd}/meson.build (65%)
>  create mode 100644 src/amd/vulkan/meson.build
>
> diff --git a/meson.build b/meson.build
> index 0f5198bac72..5353a417484 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -71,6 +71,12 @@ if get_option('buildtype').startswith('debug')
>    pre_args += '-DDEBUG'
>  endif
>
> +if get_option('shader-cache')
> +  pre_args += '-DENABLE_SHADER_CACHE'
> +elif with_amd_vk
> +  error('Radv requires shader cache support')
> +endif
> +
>  # Check for GCC style builtins
>  foreach b : ['bswap32', 'bswap64', 'clz', 'clzll', 'ctz', 'expect', 'ffs',
>               'ffsll', 'popcount', 'popcountll', 'unreachable']
> @@ -79,7 +85,7 @@ foreach b : ['bswap32', 'bswap64', 'clz', 'clzll', 'ctz', 'expect', 'ffs',
>    endif
>  endforeach
>
> -# check for GCC __attribute__ s
> +# check for GCC __attribute__
>  foreach a : ['const', 'flatten', 'malloc', 'pure', 'unused',
>               'warn_unused_result', 'weak',]
>    if cc.compiles('int foo(void) __attribute__((@0@));'.format(a),
> @@ -286,6 +292,35 @@ dep_m = cc.find_library('m', required : false)
>  # TODO: conditionalize libdrm requirement
>  dep_libdrm = dependency('libdrm', version : '>= 2.4.75')
>  pre_args += '-DHAVE_LIBDRM'
> +dep_libdrm_amdgpu = []
> +if with_amd_vk
> +  dep_libdrm_amdgpu = dependency('libdrm_amdgpu', version : '>= 2.4.82')
> +endif
> +
> +llvm_modules = ['bitwriter', 'engine', 'mcdisassembler', 'mcjit']
> +if with_amd_vk
> +  llvm_modules += ['amdgpu', 'bitreader', 'ipo']
> +endif
> +dep_llvm = dependency(
> +  'llvm', version : '>= 3.9.0', required : false, modules : llvm_modules,
> +)
> +if not dep_llvm.found()
> +  if with_amd_vk
> +    error('Radv requires llvm.')
> +  endif
> +else
> +  _llvm_version = dep_llvm.version().split('.')
> +  # Development versions of LLVM have an 'svn' suffix, we don't want that for
> +  # our version checks.
> +  _llvm_patch = _llvm_version[2]
> +  if _llvm_patch.endswith('svn')
> +    _llvm_patch = _llvm_patch.split('s')[0]
> +  endif
> +  pre_args += [
> +    '-DHAVE_LLVM=0x0 at 0@@1@@2@'.format(_llvm_version[0], _llvm_version[1], _llvm_patch),
> +    '-DMESA_LLVM_VERSION_PATCH=@0@'.format(_llvm_patch),
> +  ]
> +endif
>
>  # TODO: make this conditional
>  dep_valgrind = dependency('valgrind', required : false)
> @@ -299,8 +334,6 @@ endif
>
>  # TODO: llvm-prefix and llvm-shared-libs
>
> -# TODO: llvm dependency (that's all native now, yay!)
> -
>  # TODO: unwind (llvm [radeon, gallivm] and gallium)
>
>  # TODO: flags for opengl, gles, dri
> diff --git a/meson_options.txt b/meson_options.txt
> index 0e0d04a0f8f..082ade7f480 100644
> --- a/meson_options.txt
> +++ b/meson_options.txt
> @@ -20,8 +20,10 @@
>
>  option('platforms',      type : 'string',  value : 'x11,wayland',
>         description : 'comma separated list of window systems to support. wayland, x11, surfaceless, drm, etc.')
> -option('vulkan-drivers', type : 'string',  value : 'intel',
> +option('vulkan-drivers', type : 'string',  value : 'intel,amd',
>         description : 'comma separated list of vulkan drivers to build.')
> +option('shader-cache',    type : 'boolean', value : true,
> +       description : 'Build with on-disk shader cache support')
>  option('vulkan_icd_dir', type : 'string',  value : '',
>         description : 'Location relative to prefix to put vulkan icds on install. Default: $datadir/vulkan/icd.d')
>  option('valgrind',       type : 'boolean', value : true,
> diff --git a/src/meson.build b/src/amd/addrlib/meson.build
> similarity index 51%
> copy from src/meson.build
> copy to src/amd/addrlib/meson.build
> index 4c82eec70f1..a6cad1207b0 100644
> --- a/src/meson.build
> +++ b/src/amd/addrlib/meson.build
> @@ -18,31 +18,46 @@
>  # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
>  # SOFTWARE.
>
> -# TODO: libglsl_util
> +files_addrlib = files(
> +  'addrinterface.cpp',
> +  'addrinterface.h',
> +  'addrtypes.h',
> +  'core/addrcommon.h',
> +  'core/addrelemlib.cpp',
> +  'core/addrelemlib.h',
> +  'core/addrlib.cpp',
> +  'core/addrlib.h',
> +  'core/addrlib1.cpp',
> +  'core/addrlib1.h',
> +  'core/addrlib2.cpp',
> +  'core/addrlib2.h',
> +  'core/addrobject.cpp',
> +  'core/addrobject.h',
> +  'gfx9/chip/gfx9_enum.h',
> +  'gfx9/coord.cpp',
> +  'gfx9/coord.h',
> +  'gfx9/gfx9addrlib.cpp',
> +  'gfx9/gfx9addrlib.h',
> +  'gfx9/rbmap.cpp',
> +  'gfx9/rbmap.h',
> +  'inc/chip/gfx9/gfx9_gb_reg.h',
> +  'inc/chip/r800/si_gb_reg.h',
> +  'inc/lnx_common_defs.h',
> +  'r800/chip/si_ci_vi_merged_enum.h',
> +  'r800/ciaddrlib.cpp',
> +  'r800/ciaddrlib.h',
> +  'r800/egbaddrlib.cpp',
> +  'r800/egbaddrlib.h',
> +  'r800/siaddrlib.cpp',
> +  'r800/siaddrlib.h',
> +)
>
> -# TODO: git_sha. There's a meson builtin for this
> -
> -inc_common = include_directories(
> -  '../include', '.', 'mapi', 'mesa', 'gallium/include', 'gallium/auxiliary')
> -inc_mesa = include_directories('mesa')
> -inc_mapi = include_directories('mapi')
> -
> -subdir('gtest')
> -subdir('util')
> -#subdir('mapi/glapi/gen')
> -# TODO: mapi
> -# TODO: opengl
> -# TODO: glx
> -# TODO: osmesa
> -subdir('compiler')
> -subdir('egl/wayland/wayland-drm')
> -subdir('vulkan')
> -# TODO: amd
> -subdir('intel')
> -# TODO: vc4
> -# TODO: opengl_common
> -# TODO: dri_glx
> -# TODO: gbm
> -# TODO: egl
> -# TODO: radv
> -# TODO: gallium
> +libamdgpu_addrlib = static_library(
> +  'addrlib',
> +  files_addrlib,
> +  include_directories : include_directories(
> +    'core', 'inc/chip/gfx9', 'inc/chip/r800', 'gfx9/chip', 'r800/chip',
> +    '../common', '../../'),
> +  cpp_args : [cpp_vis_args, '-DBRAHMA_BUILD=1'],
> +  build_by_default : false,
> +)
> diff --git a/src/meson.build b/src/amd/common/meson.build
> similarity index 51%
> copy from src/meson.build
> copy to src/amd/common/meson.build
> index 4c82eec70f1..842b42f897e 100644
> --- a/src/meson.build
> +++ b/src/amd/common/meson.build
> @@ -18,31 +18,46 @@
>  # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
>  # SOFTWARE.
>
> -# TODO: libglsl_util
> +inc_amd_common = include_directories('.')
>
> -# TODO: git_sha. There's a meson builtin for this
> +sid_tables_h = custom_target(
> +  'sid_tables_h',
> +  input : ['sid_tables.py', 'sid.h', 'gfx9d.h'],
> +  output : 'sid_tables.h',
> +  command : [prog_python2, '@INPUT@'],
> +  capture : true,
> +)
>
> -inc_common = include_directories(
> -  '../include', '.', 'mapi', 'mesa', 'gallium/include', 'gallium/auxiliary')
> -inc_mesa = include_directories('mesa')
> -inc_mapi = include_directories('mapi')
> +amd_common_files = files(
> +  'ac_binary.c',
> +  'ac_binary.h',
> +  'ac_exp_param.h',
> +  'ac_llvm_build.c',
> +  'ac_llvm_build.h',
> +  'ac_llvm_helper.cpp',
> +  'ac_llvm_util.c',
> +  'ac_llvm_util.h',
> +  'ac_shader_abi.h',
> +  'ac_shader_info.c',
> +  'ac_shader_info.h',
> +  'ac_nir_to_llvm.c',
> +  'ac_nir_to_llvm.h',
> +  'ac_gpu_info.c',
> +  'ac_gpu_info.h',
> +  'ac_surface.c',
> +  'ac_surface.h',
> +  'ac_debug.c',
> +  'ac_debug.h',
> +)
>
> -subdir('gtest')
> -subdir('util')
> -#subdir('mapi/glapi/gen')
> -# TODO: mapi
> -# TODO: opengl
> -# TODO: glx
> -# TODO: osmesa
> -subdir('compiler')
> -subdir('egl/wayland/wayland-drm')
> -subdir('vulkan')
> -# TODO: amd
> -subdir('intel')
> -# TODO: vc4
> -# TODO: opengl_common
> -# TODO: dri_glx
> -# TODO: gbm
> -# TODO: egl
> -# TODO: radv
> -# TODO: gallium
> +libamd_common = static_library(
> +  'amd_common',
> +  [amd_common_files, sid_tables_h, nir_opcodes_h],
> +  include_directories : [inc_common, inc_compiler, inc_nir, inc_mesa, inc_mapi,
> +                         inc_amd],
> +  dependencies : [dep_llvm, dep_thread, dep_elf, dep_libdrm_amdgpu,
> +                  dep_valgrind],
> +  c_args : [c_vis_args],
> +  cpp_args : [cpp_vis_args],
> +  build_by_default : false,
> +)
> diff --git a/src/meson.build b/src/amd/meson.build
> similarity index 65%
> copy from src/meson.build
> copy to src/amd/meson.build
> index 4c82eec70f1..f96a9aac095 100644
> --- a/src/meson.build
> +++ b/src/amd/meson.build
> @@ -18,31 +18,10 @@
>  # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
>  # SOFTWARE.
>
> -# TODO: libglsl_util
> +inc_amd = include_directories('.')
>
> -# TODO: git_sha. There's a meson builtin for this
> -
> -inc_common = include_directories(
> -  '../include', '.', 'mapi', 'mesa', 'gallium/include', 'gallium/auxiliary')
> -inc_mesa = include_directories('mesa')
> -inc_mapi = include_directories('mapi')
> -
> -subdir('gtest')
> -subdir('util')
> -#subdir('mapi/glapi/gen')
> -# TODO: mapi
> -# TODO: opengl
> -# TODO: glx
> -# TODO: osmesa
> -subdir('compiler')
> -subdir('egl/wayland/wayland-drm')
> -subdir('vulkan')
> -# TODO: amd
> -subdir('intel')
> -# TODO: vc4
> -# TODO: opengl_common
> -# TODO: dri_glx
> -# TODO: gbm
> -# TODO: egl
> -# TODO: radv
> -# TODO: gallium
> +subdir('addrlib')
> +subdir('common')
> +if with_amd_vk
> +  subdir('vulkan')
> +endif
> diff --git a/src/amd/vulkan/meson.build b/src/amd/vulkan/meson.build
> new file mode 100644
> index 00000000000..02a2703a5c1
> --- /dev/null
> +++ b/src/amd/vulkan/meson.build
> @@ -0,0 +1,124 @@
> +# Copyright © 2017 Intel Corporation
> +
> +# Permission is hereby granted, free of charge, to any person obtaining a copy
> +# of this software and associated documentation files (the "Software"), to deal
> +# in the Software without restriction, including without limitation the rights
> +# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
> +# copies of the Software, and to permit persons to whom the Software is
> +# furnished to do so, subject to the following conditions:
> +
> +# The above copyright notice and this permission notice shall be included in
> +# all copies or substantial portions of the Software.
> +
> +# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> +# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> +# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
> +# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> +# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
> +# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
> +# SOFTWARE.
> +
> +radv_entrypoints = custom_target(
> +  'radv_entrypoints.[ch]',
> +  input : ['radv_entrypoints_gen.py', vk_api_xml],
> +  output : ['radv_entrypoints.h', 'radv_entrypoints.c'],
> +  command : [prog_python2, '@INPUT0@', '--xml', '@INPUT1@',
> +             '--outdir', meson.current_build_dir()],
> +)
> +
> +vk_format_table_c = custom_target(
> +  'vk_format_table.c',
> +  input : ['vk_format_table.py', 'vk_format_layout.csv'],
> +  output : 'vk_format_table.c',
> +  command : [prog_python2, '@INPUT@'],
> +  depend_files : files('vk_format_parse.py'),
> +  capture : true,
> +)
> +
> +libradv_files = files(
> +  'winsys/amdgpu/radv_amdgpu_bo.c',
> +  'winsys/amdgpu/radv_amdgpu_bo.h',
> +  'winsys/amdgpu/radv_amdgpu_cs.c',
> +  'winsys/amdgpu/radv_amdgpu_cs.h',
> +  'winsys/amdgpu/radv_amdgpu_surface.c',
> +  'winsys/amdgpu/radv_amdgpu_surface.h',
> +  'winsys/amdgpu/radv_amdgpu_winsys.c',
> +  'winsys/amdgpu/radv_amdgpu_winsys.h',
> +  'winsys/amdgpu/radv_amdgpu_winsys_public.h',
> +  'radv_cmd_buffer.c',
> +  'radv_cs.h',
> +  'radv_debug.c',
> +  'radv_debug.h',
> +  'radv_device.c',
> +  'radv_descriptor_set.c',
> +  'radv_descriptor_set.h',
> +  'radv_formats.c',
> +  'radv_image.c',
> +  'radv_meta.c',
> +  'radv_meta.h',
> +  'radv_meta_blit.c',
> +  'radv_meta_blit2d.c',
> +  'radv_meta_buffer.c',
> +  'radv_meta_bufimage.c',
> +  'radv_meta_clear.c',
> +  'radv_meta_copy.c',
> +  'radv_meta_decompress.c',
> +  'radv_meta_fast_clear.c',
> +  'radv_meta_resolve.c',
> +  'radv_meta_resolve_cs.c',
> +  'radv_meta_resolve_fs.c',
> +  'radv_pass.c',
> +  'radv_pipeline.c',
> +  'radv_pipeline_cache.c',
> +  'radv_private.h',
> +  'radv_radeon_winsys.h',
> +  'radv_shader.c',
> +  'radv_shader.h',
> +  'radv_query.c',
> +  'radv_util.c',
> +  'radv_util.h',
> +  'radv_wsi.c',
> +  'si_cmd_buffer.c',
> +  'vk_format.h',
> +)
> +
> +radv_deps = []
> +radv_flags = []
> +
> +if with_platform_x11
> +  radv_deps += dep_xcb_dri3
> +  radv_flags += [
> +    '-DVK_USE_PLATFORM_XCB_KHR',
> +    '-DVK_USE_PLATFORM_XLIB_KHR',
> +  ]
> +  libradv_files += files('radv_wsi_x11.c')
> +endif
> +
> +if with_platform_wayland
> +  radv_deps += dep_wayland_client
> +  radv_flags += '-DVK_USE_PLATFORM_WAYLAND_KHR'
> +  libradv_files += files('radv_wsi_wayland.c')
> +endif
> +
> +libvulkan_radeon = shared_library(
> +  'vulkan_radeon',
> +  [libradv_files, radv_entrypoints, nir_opcodes_h, vk_format_table_c],
> +  include_directories : [inc_common, inc_amd, inc_amd_common, inc_compiler,
> +                         inc_vulkan_util, inc_vulkan_wsi],
> +  link_with : [libamd_common, libamdgpu_addrlib, libvulkan_util,
> +               libvulkan_wsi, libnir, libmesa_util],
> +  dependencies : [dep_llvm, dep_libdrm_amdgpu, dep_thread, dep_elf, dep_dl,
> +                  dep_m, dep_valgrind],
> +  c_args : [c_vis_args, no_override_init_args, radv_flags],
> +  link_args : [ld_args_bsymbolic, ld_args_gc_sections],
> +  install : true,
> +)
> +
> +radv_data = configuration_data()
> +radv_data.set('install_libdir', join_paths(get_option('prefix'), get_option('libdir')))
> +configure_file(
> +  configuration : radv_data,
> +  input : 'radeon_icd.json.in',
> +  output : 'radeon_icd. at 0@.json'.format(target_machine.cpu()),
> +  install_dir : with_vulkan_icd_dir,
> +)
> diff --git a/src/meson.build b/src/meson.build
> index 4c82eec70f1..76ff497e565 100644
> --- a/src/meson.build
> +++ b/src/meson.build
> @@ -37,7 +37,7 @@ subdir('util')
>  subdir('compiler')
>  subdir('egl/wayland/wayland-drm')
>  subdir('vulkan')
> -# TODO: amd
> +subdir('amd')
>  subdir('intel')
>  # TODO: vc4
>  # TODO: opengl_common
> --
> 2.14.1
>
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>


More information about the mesa-dev mailing list