[Beignet] [PATCH v2] Fix build with latest libdrm

Armin K. krejzi at email.com
Mon Oct 10 06:48:31 UTC 2016


Hm, I didn't see it. I'm fine with whatever one gets merged, as long as 
it's fixed.

On 10.10.2016 8:05, Igor Gnatenko wrote:
> I think I had more clear patch in bugzilla:
> https://bugs.freedesktop.org/show_bug.cgi?id=98090
>
> On Sun, Oct 9, 2016 at 6:54 PM Armin K <krejzi at email.com
> <mailto:krejzi at email.com>> wrote:
>
>     libdrm-2.4.71 contains drm_intel_get_pooled_eu and
>     drm_intel_get_min_eu_in_pool with different
>     signatures.
>
>     v2:
>
>      - Fix conditional not to include 0.
>
>     Signed-off-by: Armin K <krejzi at email.com <mailto:krejzi at email.com>>
>     ---
>      src/intel/intel_driver.c | 9 ++++++---
>      1 file changed, 6 insertions(+), 3 deletions(-)
>
>     diff --git a/src/intel/intel_driver.c b/src/intel/intel_driver.c
>     index f503b55..cf35f40 100644
>     --- a/src/intel/intel_driver.c
>     +++ b/src/intel/intel_driver.c
>     @@ -958,13 +958,16 @@ intel_update_device_info(cl_device_id device)
>      #ifdef HAS_POOLED_EU
>        /* BXT pooled eu, 3*6 to 2*9, like sub slice count is 2 */
>        unsigned int has_pooled_eu = 0;
>     -  if(!drm_intel_get_pooled_eu(driver->fd, &has_pooled_eu) &&
>     has_pooled_eu)
>     +  if (drm_intel_get_pooled_eu(driver->fd) > 0) {
>          device->sub_slice_count = 2;
>     +    has_pooled_eu = 1;
>     +  }
>
>      #ifdef HAS_MIN_EU_IN_POOL
>     -  unsigned int min_eu;
>     +  int min_eu;
>        /* for fused down 2x6 devices, beignet don't support. */
>     -  if (has_pooled_eu && !drm_intel_get_min_eu_in_pool(driver->fd,
>     &min_eu)) {
>     +  if (has_pooled_eu) {
>     +    min_eu = drm_intel_get_min_eu_in_pool(driver->fd);
>          assert(min_eu == 9); //don't support fuse down device.
>        }
>      #endif //HAS_MIN_EU_IN_POOL
>     --
>     2.10.1
>
>     _______________________________________________
>     Beignet mailing list
>     Beignet at lists.freedesktop.org <mailto:Beignet at lists.freedesktop.org>
>     https://lists.freedesktop.org/mailman/listinfo/beignet
>
> --
>
> -Igor Gnatenko
>


More information about the Beignet mailing list