[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