Re: [PATCH] intel: Export GT config attributes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 09, 2015 at 10:56:16AM +0800, Zhenyu Wang wrote:
> On 2014.12.18 12:12:33 -0600, jeff.mcgee@xxxxxxxxx wrote:
> > diff --git a/include/drm/i915_drm.h b/include/drm/i915_drm.h
> > index 15dd01d..be38adf 100644
> > --- a/include/drm/i915_drm.h
> > +++ b/include/drm/i915_drm.h
> > @@ -340,6 +340,10 @@ typedef struct drm_i915_irq_wait {
> >  #define I915_PARAM_HAS_EXEC_HANDLE_LUT   26
> >  #define I915_PARAM_HAS_WT     	 	 27
> >  #define I915_PARAM_CMD_PARSER_VERSION	 28
> > +#define I915_PARAM_SLICE_TOTAL		 30
> > +#define I915_PARAM_SUBSLICE_TOTAL	 31
> > +#define I915_PARAM_EU_TOTAL		 32
> > +#define I915_PARAM_THREADS_PER_EU	 33
> >  
> >  typedef struct drm_i915_getparam {
> >  	int param;
> > diff --git a/intel/intel_bufmgr.h b/intel/intel_bufmgr.h
> > index be83a56..90e535d 100644
> > --- a/intel/intel_bufmgr.h
> > +++ b/intel/intel_bufmgr.h
> > @@ -264,6 +264,11 @@ int drm_intel_get_reset_stats(drm_intel_context *ctx,
> >  			      uint32_t *active,
> >  			      uint32_t *pending);
> >  
> > +int drm_intel_get_slice_total(int fd, unsigned int *slice_total);
> > +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_threads_per_eu(int fd, unsigned int *threads_per_eu);
> > +
> 
> I think we normally use drm_intel_bufmgr in API instead of fd directly.
> And this config info can be initialized when bufmgr init, so later request
> can get info without calling ioctl everytime.
> 
> Also please cc intel-gfx list for any intel related patch.
> 
> thanks.
> 

The implementation doesn't require a drm_intel_bufmgr instance to
query and return these values, so I think it is best to ask for just
the drm file descriptor from client. I see this approach also in use
by drm_intel_get_aperture_sizes function. Let me know if there is a
strong reason from client perspective to use drm_intel_bufmgr instead
of fd.

I thought about caching these values during bufmgr init, but decided
there was no benefit. Some clients won't need these at all. And those
that do should be calling the query functions once - these values are
unchanging.

Jeff

> >  /** @{ 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 14e92c9..e0b13e7 100644
> > --- a/intel/intel_bufmgr_gem.c
> > +++ b/intel/intel_bufmgr_gem.c
> > @@ -3293,6 +3293,69 @@ drm_intel_reg_read(drm_intel_bufmgr *bufmgr,
> >  	return ret;
> >  }
> >  
> > +drm_public int
> > +drm_intel_get_slice_total(int fd, unsigned int *slice_total)
> > +{
> > +	drm_i915_getparam_t gp;
> > +	int ret;
> > +
> > +	VG_CLEAR(gp);
> > +	gp.value = (int*)slice_total;
> > +	gp.param = I915_PARAM_SLICE_TOTAL;
> > +	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> > +	if (ret)
> > +		return -errno;
> > +
> > +	return 0;
> > +}
> > +
> > +drm_public int
> > +drm_intel_get_subslice_total(int fd, unsigned int *subslice_total)
> > +{
> > +	drm_i915_getparam_t gp;
> > +	int ret;
> > +
> > +	VG_CLEAR(gp);
> > +	gp.value = (int*)subslice_total;
> > +	gp.param = I915_PARAM_SUBSLICE_TOTAL;
> > +	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> > +	if (ret)
> > +		return -errno;
> > +
> > +	return 0;
> > +}
> > +
> > +drm_public int
> > +drm_intel_get_eu_total(int fd, unsigned int *eu_total)
> > +{
> > +	drm_i915_getparam_t gp;
> > +	int ret;
> > +
> > +	VG_CLEAR(gp);
> > +	gp.value = (int*)eu_total;
> > +	gp.param = I915_PARAM_EU_TOTAL;
> > +	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> > +	if (ret)
> > +		return -errno;
> > +
> > +	return 0;
> > +}
> > +
> > +drm_public int
> > +drm_intel_get_threads_per_eu(int fd, unsigned int *threads_per_eu)
> > +{
> > +	drm_i915_getparam_t gp;
> > +	int ret;
> > +
> > +	VG_CLEAR(gp);
> > +	gp.value = (int*)threads_per_eu;
> > +	gp.param = I915_PARAM_THREADS_PER_EU;
> > +	ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> > +	if (ret)
> > +		return -errno;
> > +
> > +	return 0;
> > +}
> >  
> >  /**
> >   * Annotate the given bo for use in aub dumping.
> > -- 
> > 2.2.0
> > 
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@xxxxxxxxxxxxxxxxxxxxx
> > http://lists.freedesktop.org/mailman/listinfo/dri-devel
> 
> -- 
> Open Source Technology Center, Intel ltd.
> 
> $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827


_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux