Created attachment 32213 [details] [review] patch for master branch master branch require libdrm >= 2.4.17.
Created attachment 32214 [details] [review] patch for mesa_7_7_branch branch 7.7 require = 2.4.16 >= 2.4.16 due to: http://cgit.freedesktop.org/mesa/drm/commit/?id=b4312b639d56a6cad78953af0fd4f863182007e3 < 2.4.17 due to: http://cgit.freedesktop.org/mesa/drm/commit/?id=125994ab30d4f0f126c62fa741ec62a52d69d7a8
Created attachment 32215 [details] [review] patch for mesa_7_6_branch branch 7.6 should build with libdrm <= 2.4.16.
Can you use the LIBDRM_RADEON_REQUIRED in the 7.7 and 7.6 patches like you did for master? Hmm, why does libdrm_radeon have a separate version number in the .pc file, and why isn't it being changed when the API changes? That would straighten this all out and be a real LIBDRM_RADEON_REQUIRED.
libdrm_radeon is still experimental and hasn't been released yet. It will be released when radeon KMS comes out of staging.
Right, I forgot that you only get to install it with --enable-radeon-experimental-api. In that case, I'm not really worried about the breakage. But it would be nice to keep libdrm_radeon.pc updated once you guys put it out in the wild.
Alternatively the version of libdrm_radeon could be changed from 1.0.1 to @PACKAGE_VERSION@ like libdrm_intel and then check mesa for libdrm_radeon version rather than libdrm. The API has however been updated for final 7.6.1 and 7.7 and all of them need now libdrm_radeon 2.4.17. There are still two warnings that appears only when compiling without libdrm_radeon: In file included from ../radeon/radeon_cs_drm.h:38, from radeon_bocs_wrapper.h:29, from radeon_cs_space_drm.c:31: /usr/include/drm/radeon_drm.h:809:1: warning: "RADEON_TILING_SURFACE" redefined In file included from radeon_cs_space_drm.c:31: radeon_bocs_wrapper.h:23:1: warning: this is the location of the previous definition In file included from ./../radeon/radeon_cs_drm.h:38, from ./radeon_bocs_wrapper.h:29, from radeon_bo.c:1: /usr/include/drm/radeon_drm.h:809:1: warning: "RADEON_TILING_SURFACE" redefined In file included from radeon_bo.c:1: ./radeon_bocs_wrapper.h:23:1: warning: this is the location of the previous definition
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.