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 <kre...@email.com
<mailto:kre...@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 <kre...@email.com <mailto:kre...@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@lists.freedesktop.org <mailto:Beignet@lists.freedesktop.org>
https://lists.freedesktop.org/mailman/listinfo/beignet
--
-Igor Gnatenko
_______________________________________________
Beignet mailing list
Beignet@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/beignet
