[Mesa-dev] [PATCH 00/28] Enable --with-platforms for all

Emil Velikov emil.l.velikov at gmail.com
Thu Dec 8 19:21:40 UTC 2016


Hi all,

As you may already know in order to build Vulkan+Wayland/X11 one needs
to select --enable-egl alongside --with-egl-platforms=...

Admittedly this is not an issue since both options have sane defaults.
At the same time, it is quite confusing to have such a dependency. Thus
this series aims to resolve that by renaming --with-{egl-}platforms and
honouring it in Vulkan, EGL and the VL targets.

So I've went ahead and:
 - cleaned-up some of the DRI3 confusion there is (patches 1-3)
 - added DRI3 sanity check for Vulkan (patch 8)
 - correctly managed X11 includes (15 & 17)
 - added DRI3 support for OMX and XVMC (trivial two-liners, 19, 20)
 - did some sanity checks when building Vulkan/OMX/... wo any platform 
(22-26, not 100% sure if building Vulkan w/o any platform is OK)
 - applied a couple of trivial configure cleanups on top (27,28)

Notes:
 - if people are really concerned with the removal of 
--with-egl-platforms, we can keep it around erroring as one tries to 
override the defaults.
 - there's a couple of "XXX" commits which will really appreciate
some input.

For anyone willing to give it a try:
https://github.com/evelikov/Mesa/tree/platforms


TL;DR; Renamed --with-{egl-,}platforms made sure everyone honours it,
such that we can have X11/Wayland/etc only builds ;-)


Testing will be greatly appreciated !

Thanks
Emil

Emil Velikov (28):
  configure: remove unneeded dri3/present proto requirements
  travis: remove unneeded dri3/present proto requirement
  configure: check once for DRI3 dependencies
  configure: remove egl platforms check
  configure: rename HAVE_{EGL_,}PLATFORM_foo
  configure: rename --with-{egl-,}platforms
  configure: move vulkan sha1 requirement to a separate function
  configure: error out when building X11 Vulkan without DRI3
  XXX:configure: set HAVE_PLATFORM_foo as applicable
  st/va: move variable declaration to where its used
  st/va: fix misplaced closing bracket
  st/va: honor the platform selection at configure time
  st/va: check if vl_*_screen_create has failed only once
  st/omx: remove unused drm_driver.h includes
  st/omx: remove unneeded X11 include
  XXX:st/omx: honour the configure --with-platform
  auxiliary/vl: attempt to fix X11-less builds
  auxiliary/vl: use stubs for vl_*_screen_create
  st/omx: add DRI3 support
  st/xvmc: add DRI3 support
  configure: move platform code further up
  configure: error out if building Vulkan w/o supported platform
  configure: error out if building VA w/o supported platform
  configure: error out if building OMX w/o supported platform
  configure: error out if building VDPAU w/o supported platform
  configure: error out if building XVMC w/o supported platform
  configure: trivial whitespace cleanup
  configure: drop trailing semicolon

 .travis.yml                                  |  12 +-
 Makefile.am                                  |   2 +-
 configure.ac                                 | 293 +++++++++++++++------------
 docs/egl.html                                |  12 +-
 src/amd/vulkan/Makefile.am                   |   3 +-
 src/egl/Makefile.am                          |   6 +-
 src/egl/main/egldisplay.c                    |   2 +-
 src/gallium/auxiliary/Makefile.am            |  15 ++
 src/gallium/auxiliary/Makefile.sources       |  10 +-
 src/gallium/auxiliary/vl/vl_winsys.h         |  19 +-
 src/gallium/state_trackers/omx/Makefile.am   |   1 +
 src/gallium/state_trackers/omx/entrypoint.c  |  14 +-
 src/gallium/state_trackers/omx/vid_dec.h     |   3 -
 src/gallium/state_trackers/omx/vid_enc.c     |   1 -
 src/gallium/state_trackers/va/Makefile.am    |   1 +
 src/gallium/state_trackers/va/context.c      |  22 +-
 src/gallium/state_trackers/vdpau/Makefile.am |   1 +
 src/gallium/state_trackers/vdpau/device.c    |   2 -
 src/gallium/state_trackers/xvmc/Makefile.am  |   3 +-
 src/gallium/state_trackers/xvmc/context.c    |   4 +-
 src/gallium/targets/omx/Makefile.am          |   1 +
 src/gallium/targets/va/Makefile.am           |   1 +
 src/gallium/targets/vdpau/Makefile.am        |   1 +
 src/gallium/targets/xvmc/Makefile.am         |   1 +
 src/intel/vulkan/Makefile.am                 |   3 +-
 src/vulkan/wsi/Makefile.am                   |   3 +-
 26 files changed, 254 insertions(+), 182 deletions(-)

-- 
2.10.2



More information about the mesa-dev mailing list