All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] intel: Export pooled EU and min no. of eus in a pool.
@ 2016-07-06  4:51 Yang Rong
  2016-07-13 10:44 ` Arun Siluvery
  0 siblings, 1 reply; 5+ messages in thread
From: Yang Rong @ 2016-07-06  4:51 UTC (permalink / raw)
  To: beignet, arun.siluvery, dri-devel; +Cc: Yang Rong

Update kernel interface with new I915_GETPARAM ioctl entries for
pooled EU and min no. of eus in a pool. Add a wrapping function
for each parameter. Userspace drivers need these values when decide
the thread count. This kernel enabled pooled eu by default for BXT
and for fused down 2x6 parts it is advised to turn it off.

But there is another HW issue in these parts (fused
down 2x6 parts) before C0 that requires Pooled EU to be enabled as a
workaround. In this case the pool configuration changes depending upon
which subslice is disabled and the no. of eus in a pool is different,
So userspace need to know min no. of eus in a pool.

Signed-off-by: Yang Rong <rong.r.yang@intel.com>
---
 include/drm/i915_drm.h   |  2 ++
 intel/intel_bufmgr.h     |  3 +++
 intel/intel_bufmgr_gem.c | 32 ++++++++++++++++++++++++++++++++
 3 files changed, 37 insertions(+)

diff --git a/include/drm/i915_drm.h b/include/drm/i915_drm.h
index c4ce6b2..eb611a7 100644
--- a/include/drm/i915_drm.h
+++ b/include/drm/i915_drm.h
@@ -357,6 +357,8 @@ typedef struct drm_i915_irq_wait {
 #define I915_PARAM_HAS_GPU_RESET	 35
 #define I915_PARAM_HAS_RESOURCE_STREAMER 36
 #define I915_PARAM_HAS_EXEC_SOFTPIN	 37
+#define I915_PARAM_HAS_POOLED_EU         38
+#define I915_PARAM_MIN_EU_IN_POOL        39
 
 typedef struct drm_i915_getparam {
 	__s32 param;
diff --git a/intel/intel_bufmgr.h b/intel/intel_bufmgr.h
index a1abbcd..8370694 100644
--- a/intel/intel_bufmgr.h
+++ b/intel/intel_bufmgr.h
@@ -273,6 +273,9 @@ int drm_intel_get_reset_stats(drm_intel_context *ctx,
 int drm_intel_get_subslice_total(int fd, unsigned int *subslice_total);
 int drm_intel_get_eu_total(int fd, unsigned int *eu_total);
 
+int drm_intel_get_pooled_eu(int fd, unsigned int *has_pooled_eu);
+int drm_intel_get_min_eu_in_pool(int fd, unsigned int *min_eu);
+
 /** @{ Compatibility defines to keep old code building despite the symbol rename
  * from dri_* to drm_intel_*
  */
diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
index 0a4012b..b8bb654 100644
--- a/intel/intel_bufmgr_gem.c
+++ b/intel/intel_bufmgr_gem.c
@@ -3237,6 +3237,38 @@ drm_intel_get_eu_total(int fd, unsigned int *eu_total)
 	return 0;
 }
 
+int
+drm_intel_get_pooled_eu(int fd, unsigned int *has_pooled_eu)
+{
+	drm_i915_getparam_t gp;
+	int ret;
+
+	memclear(gp);
+	gp.value = (int*)has_pooled_eu;
+	gp.param = I915_PARAM_HAS_POOLED_EU;
+	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
+	if (ret)
+		return -errno;
+
+	return 0;
+}
+
+int
+drm_intel_get_min_eu_in_pool(int fd, unsigned int *min_eu)
+{
+	drm_i915_getparam_t gp;
+	int ret;
+
+	memclear(gp);
+	gp.value = (int*)min_eu;
+	gp.param = I915_PARAM_MIN_EU_IN_POOL;
+	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
+	if (ret)
+		return -errno;
+
+	return 0;
+}
+
 /**
  * Annotate the given bo for use in aub dumping.
  *
-- 
2.1.4

_______________________________________________
Beignet mailing list
Beignet@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/beignet

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] intel: Export pooled EU and min no. of eus in a pool.
  2016-07-06  4:51 [PATCH] intel: Export pooled EU and min no. of eus in a pool Yang Rong
@ 2016-07-13 10:44 ` Arun Siluvery
  2016-07-13 19:09   ` [Beignet] " Chris Wilson
  0 siblings, 1 reply; 5+ messages in thread
From: Arun Siluvery @ 2016-07-13 10:44 UTC (permalink / raw)
  To: Yang Rong, beignet, dri-devel, Daniel Vetter
  Cc: Tvrtko Ursulin, martin.peres, Chris Wilson

On 06/07/2016 05:51, Yang Rong wrote:
> Update kernel interface with new I915_GETPARAM ioctl entries for
> pooled EU and min no. of eus in a pool. Add a wrapping function
> for each parameter. Userspace drivers need these values when decide
> the thread count. This kernel enabled pooled eu by default for BXT
> and for fused down 2x6 parts it is advised to turn it off.
>
> But there is another HW issue in these parts (fused
> down 2x6 parts) before C0 that requires Pooled EU to be enabled as a
> workaround. In this case the pool configuration changes depending upon
> which subslice is disabled and the no. of eus in a pool is different,
> So userspace need to know min no. of eus in a pool.
>
> Signed-off-by: Yang Rong <rong.r.yang@intel.com>
> ---

Could you check this and give comments/ACK to merge this to libdrm?
Kernel changes are merged in drm-intel.

regards
Arun

>   include/drm/i915_drm.h   |  2 ++
>   intel/intel_bufmgr.h     |  3 +++
>   intel/intel_bufmgr_gem.c | 32 ++++++++++++++++++++++++++++++++
>   3 files changed, 37 insertions(+)
>
> diff --git a/include/drm/i915_drm.h b/include/drm/i915_drm.h
> index c4ce6b2..eb611a7 100644
> --- a/include/drm/i915_drm.h
> +++ b/include/drm/i915_drm.h
> @@ -357,6 +357,8 @@ typedef struct drm_i915_irq_wait {
>   #define I915_PARAM_HAS_GPU_RESET	 35
>   #define I915_PARAM_HAS_RESOURCE_STREAMER 36
>   #define I915_PARAM_HAS_EXEC_SOFTPIN	 37
> +#define I915_PARAM_HAS_POOLED_EU         38
> +#define I915_PARAM_MIN_EU_IN_POOL        39
>
>   typedef struct drm_i915_getparam {
>   	__s32 param;
> diff --git a/intel/intel_bufmgr.h b/intel/intel_bufmgr.h
> index a1abbcd..8370694 100644
> --- a/intel/intel_bufmgr.h
> +++ b/intel/intel_bufmgr.h
> @@ -273,6 +273,9 @@ int drm_intel_get_reset_stats(drm_intel_context *ctx,
>   int drm_intel_get_subslice_total(int fd, unsigned int *subslice_total);
>   int drm_intel_get_eu_total(int fd, unsigned int *eu_total);
>
> +int drm_intel_get_pooled_eu(int fd, unsigned int *has_pooled_eu);
> +int drm_intel_get_min_eu_in_pool(int fd, unsigned int *min_eu);
> +
>   /** @{ Compatibility defines to keep old code building despite the symbol rename
>    * from dri_* to drm_intel_*
>    */
> diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
> index 0a4012b..b8bb654 100644
> --- a/intel/intel_bufmgr_gem.c
> +++ b/intel/intel_bufmgr_gem.c
> @@ -3237,6 +3237,38 @@ drm_intel_get_eu_total(int fd, unsigned int *eu_total)
>   	return 0;
>   }
>
> +int
> +drm_intel_get_pooled_eu(int fd, unsigned int *has_pooled_eu)
> +{
> +	drm_i915_getparam_t gp;
> +	int ret;
> +
> +	memclear(gp);
> +	gp.value = (int*)has_pooled_eu;
> +	gp.param = I915_PARAM_HAS_POOLED_EU;
> +	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> +	if (ret)
> +		return -errno;
> +
> +	return 0;
> +}
> +
> +int
> +drm_intel_get_min_eu_in_pool(int fd, unsigned int *min_eu)
> +{
> +	drm_i915_getparam_t gp;
> +	int ret;
> +
> +	memclear(gp);
> +	gp.value = (int*)min_eu;
> +	gp.param = I915_PARAM_MIN_EU_IN_POOL;
> +	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> +	if (ret)
> +		return -errno;
> +
> +	return 0;
> +}
> +
>   /**
>    * Annotate the given bo for use in aub dumping.
>    *
>

_______________________________________________
Beignet mailing list
Beignet@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/beignet

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Beignet] [PATCH] intel: Export pooled EU and min no. of eus in a pool.
  2016-07-13 10:44 ` Arun Siluvery
@ 2016-07-13 19:09   ` Chris Wilson
  2016-07-15  5:36     ` Yang, Rong R
  0 siblings, 1 reply; 5+ messages in thread
From: Chris Wilson @ 2016-07-13 19:09 UTC (permalink / raw)
  To: Arun Siluvery; +Cc: Tvrtko Ursulin, Yang Rong, dri-devel, beignet

On Wed, Jul 13, 2016 at 11:44:09AM +0100, Arun Siluvery wrote:
> On 06/07/2016 05:51, Yang Rong wrote:
> >Update kernel interface with new I915_GETPARAM ioctl entries for
> >pooled EU and min no. of eus in a pool. Add a wrapping function
> >for each parameter. Userspace drivers need these values when decide
> >the thread count. This kernel enabled pooled eu by default for BXT
> >and for fused down 2x6 parts it is advised to turn it off.
> >
> >But there is another HW issue in these parts (fused
> >down 2x6 parts) before C0 that requires Pooled EU to be enabled as a
> >workaround. In this case the pool configuration changes depending upon
> >which subslice is disabled and the no. of eus in a pool is different,
> >So userspace need to know min no. of eus in a pool.
> >
> >Signed-off-by: Yang Rong <rong.r.yang@intel.com>
> >---
> 
> Could you check this and give comments/ACK to merge this to libdrm?
> Kernel changes are merged in drm-intel.
> 
> regards
> Arun
> 
> >  include/drm/i915_drm.h   |  2 ++
> >  intel/intel_bufmgr.h     |  3 +++
> >  intel/intel_bufmgr_gem.c | 32 ++++++++++++++++++++++++++++++++
> >  3 files changed, 37 insertions(+)
> >
> >diff --git a/include/drm/i915_drm.h b/include/drm/i915_drm.h
> >index c4ce6b2..eb611a7 100644
> >--- a/include/drm/i915_drm.h
> >+++ b/include/drm/i915_drm.h
> >@@ -357,6 +357,8 @@ typedef struct drm_i915_irq_wait {
> >  #define I915_PARAM_HAS_GPU_RESET	 35
> >  #define I915_PARAM_HAS_RESOURCE_STREAMER 36
> >  #define I915_PARAM_HAS_EXEC_SOFTPIN	 37
> >+#define I915_PARAM_HAS_POOLED_EU         38
> >+#define I915_PARAM_MIN_EU_IN_POOL        39
> >
> >  typedef struct drm_i915_getparam {
> >  	__s32 param;
> >diff --git a/intel/intel_bufmgr.h b/intel/intel_bufmgr.h
> >index a1abbcd..8370694 100644
> >--- a/intel/intel_bufmgr.h
> >+++ b/intel/intel_bufmgr.h
> >@@ -273,6 +273,9 @@ int drm_intel_get_reset_stats(drm_intel_context *ctx,
> >  int drm_intel_get_subslice_total(int fd, unsigned int *subslice_total);
> >  int drm_intel_get_eu_total(int fd, unsigned int *eu_total);
> >
> >+int drm_intel_get_pooled_eu(int fd, unsigned int *has_pooled_eu);
> >+int drm_intel_get_min_eu_in_pool(int fd, unsigned int *min_eu);

Do we export more than 2 billion eu? Or more than 2 billion states for
the bool has_pooled_eu?

Could we not just use

int drm_intel_get_pooled_eu(int fd);

ret < 0 => error occurred / unknown
ret == 0 => no
ret > 0 => yes

and same for get_min_eu.

> >+
> >  /** @{ Compatibility defines to keep old code building despite the symbol rename
> >   * from dri_* to drm_intel_*
> >   */
> >diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
> >index 0a4012b..b8bb654 100644
> >--- a/intel/intel_bufmgr_gem.c
> >+++ b/intel/intel_bufmgr_gem.c
> >@@ -3237,6 +3237,38 @@ drm_intel_get_eu_total(int fd, unsigned int *eu_total)
> >  	return 0;
> >  }
> >
> >+int
> >+drm_intel_get_pooled_eu(int fd, unsigned int *has_pooled_eu)
> >+{
> >+	drm_i915_getparam_t gp;
> >+	int ret;
> >+
> >+	memclear(gp);
> >+	gp.value = (int*)has_pooled_eu;
> >+	gp.param = I915_PARAM_HAS_POOLED_EU;
> >+	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> >+	if (ret)

save a few electrons....

int drm_intel_get_pooled_eu(int fd)
{
	drm_i915_getparam_t gp;
	int ret;

	memclear(gp);
	gp.param = I915_PARAM_HAS_POOLED_EU;
	gp.value = &ret;
	if (drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp))
		return -errno;

	return ret;
}

(save a few more by calling this int __getparam(int fd, unsigned name))
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] intel: Export pooled EU and min no. of eus in a pool.
  2016-07-13 19:09   ` [Beignet] " Chris Wilson
@ 2016-07-15  5:36     ` Yang, Rong R
  0 siblings, 0 replies; 5+ messages in thread
From: Yang, Rong R @ 2016-07-15  5:36 UTC (permalink / raw)
  To: Chris Wilson, Arun Siluvery
  Cc: Daniel Vetter, martin.peres, dri-devel, beignet, Tvrtko Ursulin

OK, I have send a new version patch.

> -----Original Message-----
> From: Chris Wilson [mailto:chris@chris-wilson.co.uk]
> Sent: Thursday, July 14, 2016 3:10
> To: Arun Siluvery <arun.siluvery@linux.intel.com>
> Cc: Yang, Rong R <rong.r.yang@intel.com>; beignet@lists.freedesktop.org;
> dri-devel@lists.freedesktop.org; Daniel Vetter <daniel@ffwll.ch>; Tvrtko
> Ursulin <tvrtko.ursulin@linux.intel.com>; martin.peres@linux.intel.com
> Subject: Re: [Beignet] [PATCH] intel: Export pooled EU and min no. of eus in a
> pool.
> 
> On Wed, Jul 13, 2016 at 11:44:09AM +0100, Arun Siluvery wrote:
> > On 06/07/2016 05:51, Yang Rong wrote:
> > >Update kernel interface with new I915_GETPARAM ioctl entries for
> > >pooled EU and min no. of eus in a pool. Add a wrapping function for
> > >each parameter. Userspace drivers need these values when decide the
> > >thread count. This kernel enabled pooled eu by default for BXT and
> > >for fused down 2x6 parts it is advised to turn it off.
> > >
> > >But there is another HW issue in these parts (fused down 2x6 parts)
> > >before C0 that requires Pooled EU to be enabled as a workaround. In
> > >this case the pool configuration changes depending upon which
> > >subslice is disabled and the no. of eus in a pool is different, So
> > >userspace need to know min no. of eus in a pool.
> > >
> > >Signed-off-by: Yang Rong <rong.r.yang@intel.com>
> > >---
> >
> > Could you check this and give comments/ACK to merge this to libdrm?
> > Kernel changes are merged in drm-intel.
> >
> > regards
> > Arun
> >
> > >  include/drm/i915_drm.h   |  2 ++
> > >  intel/intel_bufmgr.h     |  3 +++
> > >  intel/intel_bufmgr_gem.c | 32 ++++++++++++++++++++++++++++++++
> > >  3 files changed, 37 insertions(+)
> > >
> > >diff --git a/include/drm/i915_drm.h b/include/drm/i915_drm.h index
> > >c4ce6b2..eb611a7 100644
> > >--- a/include/drm/i915_drm.h
> > >+++ b/include/drm/i915_drm.h
> > >@@ -357,6 +357,8 @@ typedef struct drm_i915_irq_wait {
> > >  #define I915_PARAM_HAS_GPU_RESET	 35
> > >  #define I915_PARAM_HAS_RESOURCE_STREAMER 36
> > >  #define I915_PARAM_HAS_EXEC_SOFTPIN	 37
> > >+#define I915_PARAM_HAS_POOLED_EU         38
> > >+#define I915_PARAM_MIN_EU_IN_POOL        39
> > >
> > >  typedef struct drm_i915_getparam {
> > >  	__s32 param;
> > >diff --git a/intel/intel_bufmgr.h b/intel/intel_bufmgr.h index
> > >a1abbcd..8370694 100644
> > >--- a/intel/intel_bufmgr.h
> > >+++ b/intel/intel_bufmgr.h
> > >@@ -273,6 +273,9 @@ int drm_intel_get_reset_stats(drm_intel_context
> > >*ctx,
> > >  int drm_intel_get_subslice_total(int fd, unsigned int
> > >*subslice_total);
> > >  int drm_intel_get_eu_total(int fd, unsigned int *eu_total);
> > >
> > >+int drm_intel_get_pooled_eu(int fd, unsigned int *has_pooled_eu);
> > >+int drm_intel_get_min_eu_in_pool(int fd, unsigned int *min_eu);
> 
> Do we export more than 2 billion eu? Or more than 2 billion states for the
> bool has_pooled_eu?
> 
> Could we not just use
> 
> int drm_intel_get_pooled_eu(int fd);
> 
> ret < 0 => error occurred / unknown
> ret == 0 => no
> ret > 0 => yes
> 
> and same for get_min_eu.
> 
> > >+
> > >  /** @{ Compatibility defines to keep old code building despite the
> symbol rename
> > >   * from dri_* to drm_intel_*
> > >   */
> > >diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
> > >index 0a4012b..b8bb654 100644
> > >--- a/intel/intel_bufmgr_gem.c
> > >+++ b/intel/intel_bufmgr_gem.c
> > >@@ -3237,6 +3237,38 @@ drm_intel_get_eu_total(int fd, unsigned int
> *eu_total)
> > >  	return 0;
> > >  }
> > >
> > >+int
> > >+drm_intel_get_pooled_eu(int fd, unsigned int *has_pooled_eu) {
> > >+	drm_i915_getparam_t gp;
> > >+	int ret;
> > >+
> > >+	memclear(gp);
> > >+	gp.value = (int*)has_pooled_eu;
> > >+	gp.param = I915_PARAM_HAS_POOLED_EU;
> > >+	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> > >+	if (ret)
> 
> save a few electrons....
> 
> int drm_intel_get_pooled_eu(int fd)
> {
> 	drm_i915_getparam_t gp;
> 	int ret;
> 
> 	memclear(gp);
> 	gp.param = I915_PARAM_HAS_POOLED_EU;
> 	gp.value = &ret;
> 	if (drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp))
> 		return -errno;
> 
> 	return ret;
> }
> 
> (save a few more by calling this int __getparam(int fd, unsigned name)) -
> Chris
> 
> --
> Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Beignet mailing list
Beignet@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/beignet

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Beignet] [PATCH] intel: Export pooled EU and min no. of eus in a pool.
  2016-06-30  8:58     ` Yang, Rong R
@ 2016-06-30 10:30       ` Arun Siluvery
  0 siblings, 0 replies; 5+ messages in thread
From: Arun Siluvery @ 2016-06-30 10:30 UTC (permalink / raw)
  To: Yang, Rong R, beignet, intel-gfx

On 30/06/2016 09:58, Yang, Rong R wrote:
> Hi, Arun,
>
>      Beignet patch is reviewed by ruiling, can you have to export them?
>

Thanks for the update, I will send kernel patches to export them.

regards
Arun

> Thanks,
> Yang Rong
>
>> -----Original Message-----
>> From: Beignet [mailto:beignet-bounces@lists.freedesktop.org] On Behalf Of
>> Arun Siluvery
>> Sent: Wednesday, June 15, 2016 16:17
>> To: Yang, Rong R <rong.r.yang@intel.com>; beignet@lists.freedesktop.org;
>> intel-gfx@lists.freedesktop.org
>> Subject: Re: [Beignet] [PATCH] intel: Export pooled EU and min no. of eus in a
>> pool.
>>
>> On 15/06/2016 13:49, Yang Rong wrote:
>>> Update kernel interface with new I915_GETPARAM ioctl entries for
>>> pooled EU and min no. of eus in a pool. Add a wrapping function for
>>> each parameter. Userspace drivers need these values when decide the
>>> thread count. This kernel enabled pooled eu by default for BXT and for
>>> fused down 2x6 parts it is advised to turn it off.
>>>
>>> But there is another HW issue in these parts (fused down 2x6 parts)
>>> before C0 that requires Pooled EU to be enabled as a workaround. In
>>> this case the pool configuration changes depending upon which subslice
>>> is disabled and the no. of eus in a pool is different, So userspace
>>> need to know min no. of eus in a pool.
>>>
>>> Signed-off-by: Yang Rong <rong.r.yang@intel.com>
>>> ---
>>>    include/drm/i915_drm.h   |  2 ++
>>>    intel/intel_bufmgr.h     |  3 +++
>>>    intel/intel_bufmgr_gem.c | 32 ++++++++++++++++++++++++++++++++
>>>    3 files changed, 37 insertions(+)
>>>
>>> diff --git a/include/drm/i915_drm.h b/include/drm/i915_drm.h index
>>> c4ce6b2..eb611a7 100644
>>> --- a/include/drm/i915_drm.h
>>> +++ b/include/drm/i915_drm.h
>>> @@ -357,6 +357,8 @@ typedef struct drm_i915_irq_wait {
>>>    #define I915_PARAM_HAS_GPU_RESET	 35
>>>    #define I915_PARAM_HAS_RESOURCE_STREAMER 36
>>>    #define I915_PARAM_HAS_EXEC_SOFTPIN	 37
>>> +#define I915_PARAM_HAS_POOLED_EU         38
>>> +#define I915_PARAM_MIN_EU_IN_POOL        39
>>>
>>
>> Please note that these are not yet added in kernel because opensource user
>> is required to merge kernel support.
>>
>> At the moment kernel bits are separated and are merged, only thing
>> remaining is to export these getparam ioctls which can be done once
>> userspace is available which is the current set of patches.
>>
>> Once these patches are reviewed then I can export them from kernel side
>> also.
>>
>> regards
>> Arun
>>
>>>    typedef struct drm_i915_getparam {
>>>    	__s32 param;
>>> diff --git a/intel/intel_bufmgr.h b/intel/intel_bufmgr.h index
>>> a1abbcd..8370694 100644
>>> --- a/intel/intel_bufmgr.h
>>> +++ b/intel/intel_bufmgr.h
>>> @@ -273,6 +273,9 @@ int drm_intel_get_reset_stats(drm_intel_context
>> *ctx,
>>>    int drm_intel_get_subslice_total(int fd, unsigned int *subslice_total);
>>>    int drm_intel_get_eu_total(int fd, unsigned int *eu_total);
>>>
>>> +int drm_intel_get_pooled_eu(int fd, unsigned int *has_pooled_eu); int
>>> +drm_intel_get_min_eu_in_pool(int fd, unsigned int *min_eu);
>>> +
>>>    /** @{ Compatibility defines to keep old code building despite the symbol
>> rename
>>>     * from dri_* to drm_intel_*
>>>     */
>>> diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c index
>>> 0a4012b..b8bb654 100644
>>> --- a/intel/intel_bufmgr_gem.c
>>> +++ b/intel/intel_bufmgr_gem.c
>>> @@ -3237,6 +3237,38 @@ drm_intel_get_eu_total(int fd, unsigned int
>> *eu_total)
>>>    	return 0;
>>>    }
>>>
>>> +int
>>> +drm_intel_get_pooled_eu(int fd, unsigned int *has_pooled_eu) {
>>> +	drm_i915_getparam_t gp;
>>> +	int ret;
>>> +
>>> +	memclear(gp);
>>> +	gp.value = (int*)has_pooled_eu;
>>> +	gp.param = I915_PARAM_HAS_POOLED_EU;
>>> +	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
>>> +	if (ret)
>>> +		return -errno;
>>> +
>>> +	return 0;
>>> +}
>>> +
>>> +int
>>> +drm_intel_get_min_eu_in_pool(int fd, unsigned int *min_eu) {
>>> +	drm_i915_getparam_t gp;
>>> +	int ret;
>>> +
>>> +	memclear(gp);
>>> +	gp.value = (int*)min_eu;
>>> +	gp.param = I915_PARAM_MIN_EU_IN_POOL;
>>> +	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
>>> +	if (ret)
>>> +		return -errno;
>>> +
>>> +	return 0;
>>> +}
>>> +
>>>    /**
>>>     * Annotate the given bo for use in aub dumping.
>>>     *
>>>
>>
>> _______________________________________________
>> Beignet mailing list
>> Beignet@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/beignet
> _______________________________________________
> Beignet mailing list
> Beignet@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/beignet
>

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-07-15  5:36 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-06  4:51 [PATCH] intel: Export pooled EU and min no. of eus in a pool Yang Rong
2016-07-13 10:44 ` Arun Siluvery
2016-07-13 19:09   ` [Beignet] " Chris Wilson
2016-07-15  5:36     ` Yang, Rong R
  -- strict thread matches above, loose matches on Subject: below --
2016-06-15  8:19 [PATCH] Runtime: set the sub slice according to kernel pooled EU configure Yang Rong
2016-06-15  8:19 ` [PATCH] intel: Export pooled EU and min no. of eus in a pool Yang Rong
2016-06-15  8:17   ` Arun Siluvery
2016-06-30  8:58     ` Yang, Rong R
2016-06-30 10:30       ` [Beignet] " Arun Siluvery

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.